<div dir="ltr">On Fri, Sep 15, 2017 at 6:51 AM, Adrian Johnson<span dir="ltr"></span><span class="gmail-"></span><br><span class="gmail-"></span><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Is the text string supplied to cairo_font_options_set_<wbr>variations()<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><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">https://git.gnome.org/browse/gtk+/tree/demos/gtk-demo/font_features.c?h=wip/matthiasc/font-variations#n454</a><br><br></div><div>I need to fix that code up to use C locale as well.<br><br></div><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></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 class="gmail_extra"><br></div></div>