<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Sep 15, 2017 at 4:35 AM, Matthias Clasen <span dir="ltr"><<a href="mailto:matthias.clasen@gmail.com" target="_blank">matthias.clasen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">On Fri, Sep 15, 2017 at 6:51 AM, Adrian Johnson<span dir="ltr"></span><span class="gmail-m_1860329818731201292gmail-"></span><br><span class="gmail-m_1860329818731201292gmail-"></span><br><div class="gmail_extra"><div class="gmail_quote"><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Is the text string supplied to cairo_font_options_set_variati<wbr>ons()<br>
likely to be generated by sprintf or is it always read from an external<br>
source in C locale format?<br></blockquote><div><br></div></span><div> You have a good point here. The string is expected to be generated<br></div><div>programmatically, as in this example here:<br><br><a href="https://git.gnome.org/browse/gtk+/tree/demos/gtk-demo/font_features.c?h=wip/matthiasc/font-variations#n454" target="_blank">https://git.gnome.org/browse/<wbr>gtk+/tree/demos/gtk-demo/font_<wbr>features.c?h=wip/matthiasc/<wbr>font-variations#n454</a><br><br></div><div>I need to fix that code up to use C locale as well.<br><br></div><span class="gmail-"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
We really need a test case for this. It is hard to review code I can't test.<br>
</blockquote></span></div><br></div><div class="gmail_extra">I don't disagree. As always with testing font-related things, the problem is to find a suitable<br></div><div class="gmail_extra">font that can be shipped with the tests...<br></div></div></blockquote><div><br></div><div>We have many many fonts:</div><div><br></div><div>  <a href="https://github.com/googlei18n/noto-fonts-alpha/tree/master/from-pipeline/unhinted/variable-ttf">https://github.com/googlei18n/noto-fonts-alpha/tree/master/from-pipeline/unhinted/variable-ttf</a></div><div><br></div><div>Voto is nice because it has all main three registered axes.  Our FontTools subsetter can subset varfonts.  Here:</div><div><br></div><div>behdad:fonttools 0 (master)$ ./fonttools subset VotoSerifGX.ttf --text=cairo<br>behdad:fonttools 0 (master)$ ll VotoSerifGX.*<br>-rw-r--r-- 1 behdad eng   30848 Sep 15 12:08 VotoSerifGX.subset.ttf<br>-rw-r--r-- 1 behdad eng 2182012 Sep 15 12:07 VotoSerifGX.ttf</div><div><br></div><div>Attached.</div><div><br></div><div>b<br></div></div><br clear="all"><br>-- <br><div class="gmail_signature">behdad<br><a href="http://behdad.org/" target="_blank">http://behdad.org/</a></div>
</div></div>