[HarfBuzz] an optimization for complex fonts

Behdad Esfahbod behdad at behdad.org
Mon Oct 28 07:47:15 PDT 2013


On 13-10-28 02:47 PM, Jonathan Kew wrote:

> See what you think. An alternative implementation might be to initialize the
> is_inplace flag during hb_ot_layout_lookup_accelerator_t::init() (then no need
> for the _initialized flag, hence taking an if() out of the hot path, and the
> hb_ot_layout_lookup_accelerator_t references could remain const), but it
> looked like this would involve rather more rearrangement of code, so I took
> the simplest approach for now.

Actually, looks like we can safely just replace the inplace variable with "are
we in GPOS".  Can you try the attached patch?

-- 
behdad
http://behdad.org/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: inplace.patch
Type: text/x-patch
Size: 1151 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/harfbuzz/attachments/20131028/0ef436dd/attachment.bin>


More information about the HarfBuzz mailing list