On 13-11-29 05:10 PM, Khaled Hosny wrote: > We should compare the CGFont's then. Updated patch attached.
Ok, I just studied this. I think the work should be done at the very beginning of the look (line 653), and produce one notdef per input character. The number of glyphs in the current run is dependent on the chosen fallback font and not what we should be using. I can rework this myself eventually, but don't have my Mac around right now. Feel free to send an updated patch though. Thanks, behdad > Regards, > Khaled > > On Fri, Nov 29, 2013 at 03:13:19PM -0500, Behdad Esfahbod wrote: >> Your patch doesn't work with user features. For each user feature we use a >> sub-font of ct_font: >> >> range->font = CTFontCreateCopyWithAttributes (font_data->ct_font, 0.0, >> NULL, >> font_desc); >> >> >> On 13-11-25 08:42 AM, Khaled Hosny wrote: >>> Resending a patch that actually applies! >>> >>> >>> >>> _______________________________________________ >>> HarfBuzz mailing list >>> [email protected] >>> http://lists.freedesktop.org/mailman/listinfo/harfbuzz >>> >> >> -- >> behdad >> http://behdad.org/ -- behdad http://behdad.org/ _______________________________________________ HarfBuzz mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/harfbuzz
