Hi Behdad,

Since the common case of initializing hb_feature_t is something like

{ HB_TAG('k','e','r','n'), !!kerningEnabled, 0, -1 }

, where "0, -1" means "the entire segment" and start/end fields are of
unsigned int type,
it probably worth of introducing some special constant/enum to avoid
compilation warnings and to improve the code readability a bit;

i.e. EntireSegment = (unsigned int)-1 // ### some better name needed


Regards,
Konstantin
_______________________________________________
HarfBuzz mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/harfbuzz

Reply via email to