<div dir="ltr"><div>Hi Behdad,<br clear="all"></div><div><br></div><div>Since the common case of initializing hb_feature_t is something like<span style="color:#800080"></span>
<pre style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><span style="color:#000000">{</span><span style="color:#c0c0c0"> </span>HB_TAG<span style="color:#000000">(</span><span style="color:#008000">'k'</span><span style="color:#000000">,</span><span style="color:#008000">'e'</span><span style="color:#000000">,</span><span style="color:#008000">'r'</span><span style="color:#000000">,</span><span style="color:#008000">'n'</span><span style="color:#000000">),</span><span style="color:#c0c0c0"> </span>!!kerningEnabled<span style="color:#000000">,</span><span style="color:#c0c0c0"> </span><span style="color:#000080">0</span><span style="color:#000000">,</span><span style="color:#c0c0c0"> </span><span style="color:#000000">-</span><span style="color:#000080">1</span><span style="color:#c0c0c0"> </span><span style="color:#000000">}<br>

, where "0, -1" means "the entire segment" and start/end fields are of unsigned int type,<br></span><span style="color:#000000">it probably worth of introducing some special constant/enum to avoid compilation warnings and to improve the code readability a bit;<br>

i.e. EntireSegment = (unsigned int)-1 // ### some better name needed<br></span></pre></div><div><br></div><div>Regards,<br>Konstantin</div>
</div>