<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Apr 9, 2016 at 7:39 AM, Matthias Clasen <span dir="ltr"><<a href="mailto:matthias.clasen@gmail.com" target="_blank">matthias.clasen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, Apr 9, 2016 at 3:54 AM, Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>> wrote:<br>
><br>
</span><span class="">> It's not mergeable since you break cairo_show_glyphs(), which now<br>
> randomly operates differently based on whether the system loaded a "color"<br>
> glyph - your implementation completely discards the source and changes<br>
> the definition of the glyph operator.<br>
><br>
<br>
</span>To put this another way, there are several places in cairo (and<br>
pixman) which make the assumption that "fonts are collections of<br>
masks".<br>
Fonts with color glyphs invalidate this assumption, which is why these<br>
patches are a bit ugly.<br>
<br>
Do you have advice on how to handle this nicely in the cairo architecture ?<br><div class="HOEnZb"><div class="h5"></div></div></blockquote><div><br></div><div>It was part of cairo's original imaging model that show_glyphs is one of the various methods to specify an area within which a Porter-Duff operator happens.  It. it was defined as a mask.  Color fonts change all of that; I remember, I think it was Owen, who was opposed to making such changes to show_glyphs when I first brought up color fonts years ago.  But the world has moved in this direction, and I don't see any other option.  Asking the cairo client to separate color glyphs and draw them using a different operator is not an option IMO.<br><br></div><div>That said, perhaps we should add a flag to scaled_font_options as well.<br></div></div><br>-- <br><div class="gmail_signature">behdad<br><a href="http://behdad.org/" target="_blank">http://behdad.org/</a></div>
</div></div>