Re: Identifying the face between STRETCH and right fringe.

[Eli Zaretskii]

> From: Robert Pluim <address@hidden>
> Cc: address@hidden
> Date: Tue, 27 Nov 2018 20:14:14 +0100
> 
> >> > Why does NS use indices here, and not RGBA values?
> >> 
> >> Probably because NS doesnʼt really use the RGBA values directly at
> >> all, but uses the indices all the time.
> >
> > So where does the RGBA value in FRAME_BACKGROUND_PIXEL come from?

Can you answer this question?

> I love those rare moments when I implement something one way, and then
> Eli tells me to do it that way :-)
> 
> In theory there are a couple of other fields in the face structure
> that might need the same type of treatment, but I didnʼt see any code
> comparing them to RGBA pixel values.
> 
> I suck at naming things, comments welcome.

LGTM, thanks.  Let's wait for comments from NS people.

> -      && face->background == FRAME_BACKGROUND_PIXEL (f)
> +      && FACE_COLOR_TO_PIXEL(face->background, f) == FRAME_BACKGROUND_PIXEL 
> (f)
                               ^
Please leave one space between the macro name and the opening paren.