Re: [PATCH 0/6] cover glyph support
Chris Wilson <[email protected]>
| Newsgroups | gmane.comp.lib.cairo |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Apr 08, 2016 at 11:22:27PM -0400, [email protected] wrote: > From: Matthias Clasen <[email protected]> > > Its been almost three years since Behdad posted a quick sketch > of how support for colored glyphs could look in cairo > (https://github.com/behdad/cairo/commits/color-emoji). > Over the last few days, I've spent some time on turning this > into a more complete solution. > > The patches in this series provide fully working support for > colored glyphs in OpenType fonts for the image compositor and > the xlib compositor, and should be mergable. It's not mergeable since you break cairo_show_glyphs(), which now randomly operates differently based on whether the system loaded a "color" glyph - your implementation completely discards the source and changes the definition of the glyph operator. Looking at the implmentation, there is no difference between the RGBA color glyph and the RGBA non-color glyphs, just that you imposed a WHITE source. i.e. your color-glyph OP dst is equivalent to the current (white IN color-glyph) OP dst. Just as color-filtered glyphs are designed to work correctly with only a white source, so it appears are color glyphs. -Chris -- Chris Wilson, Intel Open Source Technology Centre -- cairo mailing list [email protected] https://lists.cairographics.org/mailman/listinfo/cairo