[HarfBuzz] an optimization for complex fonts

[Jonathan Kew]

On 28/10/13 14:47, Behdad Esfahbod wrote:
> 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?
>

Won't this cause us to swap buffers for *all* GSUB lookups, whereas 
currently we execute many of them (e.g. single substitutions) without 
doing a swap?

JK