<div dir="ltr">Here is my +1<br><div class="gmail_extra"><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">Regards,<br>Konstantin</div></div>
<br><div class="gmail_quote">2015-09-30 14:29 GMT+04:00 Behdad Esfahbod <span dir="ltr"><<a href="mailto:behdad@behdad.org" target="_blank">behdad@behdad.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Since there's been no objections, I'll go ahead and make this change.<br>
<div class="HOEnZb"><div class="h5"><br>
On 15-09-02 09:12 AM, Behdad Esfahbod wrote:<br>
> Right now default scale is 0,0 when a new font is created.  Should I change<br>
> that to upem,upem?  That sounds like a more useful default.<br>
><br>
> Specifically, this pattern is becoming more common:<br>
><br>
> parent_font = hb_font_create (face);<br>
> hb_ot_font_set_funcs (parent_font);<br>
><br>
> font = hb_font_create_sub_font (parent_font);<br>
> hb_font_set_funcs(font, custom_funcs...);<br>
><br>
> hb_font_set_scale(font, ...);<br>
><br>
><br>
> This way custom funcs can provide only a few (say, hinted h_advance) function<br>
> and everything else comes from parent_font.  For size-related things (like<br>
> glyph_extents()) that do come from parent_font, they are scaled properly to<br>
> match the scale of the new font, but that will only work if the scale of<br>
> parent_font is NOT zero, otherwise the extents will be converted to a scale of<br>
> zero (ie, they all become zero), then go through a division by zero!<br>
><br>
> Changing the default scale addresses that, and plus, make "shape at font<br>
> units" the default behavior.<br>
><br>
> Objections?<br>
><br>
> Cheers,<br>
><br>
<br>
--<br>
behdad<br>
<a href="http://behdad.org/" rel="noreferrer" 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" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/harfbuzz</a><br>
</div></div></blockquote></div><br></div></div>