[PATCH v2 xserver] EXA: Honour op parameter to exaGlyphs even if maskFormat == NULL

[Uli Schlachter]

Am 01.04.2016 um 11:24 schrieb Michel Dänzer:
> From: Michel Dänzer <michel.daenzer at amd.com>
> 
> Reported-by: Uli Schlachter <psychon at znc.in>
> 
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94775
> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>

Ping? Does anyone with lots of clue about EXA want to take a look at this?

If it helps (since it's a step in the right direction and it becomes harder to
get wrong rendering after this patch):

Reviewed-by: Uli Schlachter <psychon at znc.in>

> ---
> 
> v2: Add proper Reported-by: and Bugzilla: tags.
> 
>  exa/exa_glyphs.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/exa/exa_glyphs.c b/exa/exa_glyphs.c
> index cf21ea9..192a643 100644
> --- a/exa/exa_glyphs.c
> +++ b/exa/exa_glyphs.c
> @@ -618,9 +618,9 @@ exaGlyphsToMask(PicturePtr pMask, ExaGlyphBufferPtr buffer)
>  }
>  
>  static void
> -exaGlyphsToDst(PicturePtr pSrc, PicturePtr pDst, ExaGlyphBufferPtr buffer)
> +exaGlyphsToDst(CARD8 op, PicturePtr pSrc, PicturePtr pDst, ExaGlyphBufferPtr buffer)
>  {
> -    exaCompositeRects(PictOpOver, pSrc, buffer->mask, pDst, buffer->count,
> +    exaCompositeRects(op, pSrc, buffer->mask, pDst, buffer->count,
>                        buffer->rects);
>  
>      buffer->count = 0;
> @@ -801,7 +801,7 @@ exaGlyphs(CARD8 op,
>                                         0, 0, x - glyph->info.x,
>                                         y - glyph->info.y)
>                          == ExaGlyphNeedFlush) {
> -                        exaGlyphsToDst(pSrc, pDst, &buffer);
> +                        exaGlyphsToDst(op, pSrc, pDst, &buffer);
>                          exaBufferGlyph(pScreen, &buffer, glyph, pSrc, pDst,
>                                         xSrc + (x - glyph->info.x) - first_xOff,
>                                         ySrc + (y - glyph->info.y) - first_yOff,
> @@ -821,7 +821,7 @@ exaGlyphs(CARD8 op,
>          if (maskFormat)
>              exaGlyphsToMask(pMask, &buffer);
>          else
> -            exaGlyphsToDst(pSrc, pDst, &buffer);
> +            exaGlyphsToDst(op, pSrc, pDst, &buffer);
>      }
>  
>      if (maskFormat) {
> 


-- 
Homophobia - The fear that another man will treat you the way you treat women.