I added this enhancement to Bugzilla to keep track of it. https://bugs.freedesktop.org/show_bug.cgi?id=72805
Also, so Behdad can assign it to me if he thinks fit. Thanks! On 17 December 2013 05:21, Behdad Esfahbod <[email protected]> wrote: > On 13-12-17 03:13 AM, Konstantin Ritt wrote: > > 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 > > Hi Konstantin, > > Makes sense. Will do. > > Cheers, > -- > behdad > http://behdad.org/ > _______________________________________________ > HarfBuzz mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/harfbuzz >
_______________________________________________ HarfBuzz mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/harfbuzz
