[HarfBuzz] harfbuzz-ng: Branch 'master'

[Behdad Esfahbod]

Hi Jonathan,

Committed.  Thanks.

behdad

On 06/01/2010 12:13 PM, Jonathan Kew wrote:
> Hi Behdad,
> 
> I updated my Mozilla tree to use your latest code, with the allocator and feature compilation. Looks good in general, but I had one problem: disabling a default feature (using liga=0) didn't work.
> 
> Tracked this down to the sorting of feature_info_t records in hb_mask_allocator_t::compile(); although the liga=0 record originally followed the liga=1 (default) record, the qsort operation ended up reversing them, so then the default overrides the user feature. This seems to indicate that the feature_info_t::cmp() function is not doing what you want, in the case where the tags are the same.
> 
> I fixed this by adding an explicit "sequence" field to the feature_info_t records, so that the cmp() function can reliably maintain the original ordering for features with the same tag. This avoids the need to rely on the (perhaps implementation-dependent?) behavior of a pointer ordering comparison.
> 
> JK
> 
> 
> 
> 
> 
>