<div dir="ltr">I added this enhancement to Bugzilla to keep track of it.<div><a href="https://bugs.freedesktop.org/show_bug.cgi?id=72805">https://bugs.freedesktop.org/show_bug.cgi?id=72805</a><br></div><div><br></div><div>
Also, so Behdad can assign it to me if he thinks fit.</div><div><br></div><div>Thanks!</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 17 December 2013 05:21, Behdad Esfahbod <span dir="ltr"><<a href="mailto:behdad@behdad.org" target="_blank">behdad@behdad.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 13-12-17 03:13 AM, Konstantin Ritt wrote:<br>
> Hi Behdad,<br>
><br>
> Since the common case of initializing hb_feature_t is something like<br>
><br>
> { HB_TAG('k','e','r','n'), !!kerningEnabled, 0, -1 }<br>
><br>
><br>
> , where "0, -1" means "the entire segment" and start/end fields are of unsigned int type,<br>
> it probably worth of introducing some special constant/enum to avoid compilation warnings and to improve the code readability a bit;<br>
><br>
><br>
> i.e. EntireSegment = (unsigned int)-1 // ### some better name needed<br>
<br>
</div></div>Hi Konstantin,<br>
<br>
Makes sense.  Will do.<br>
<br>
Cheers,<br>
<span class="HOEnZb"><font color="#888888">--<br>
behdad<br>
<a href="http://behdad.org/" target="_blank">http://behdad.org/</a><br>
_______________________________________________<br>
HarfBuzz mailing list<br>
<a href="mailto:HarfBuzz@lists.freedesktop.org">HarfBuzz@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/harfbuzz" target="_blank">http://lists.freedesktop.org/mailman/listinfo/harfbuzz</a><br>
</font></span></blockquote></div><br></div>