<div dir="ltr">Yeah, I think a <a href="http://configure.ac">configure.ac</a> check and strtod_l() is best.  I'll do that in HarfBuzz.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 14, 2017 at 4:59 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">On Thu, Sep 14, 2017 at 5:54 PM, Adrian Johnson <span dir="ltr"><<a href="mailto:ajohnson@redneon.com" target="_blank">ajohnson@redneon.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 15/09/17 05:52, Matthias Clasen wrote:<br>
> On Thu, Sep 14, 2017 at 12:37 PM, Behdad Esfahbod<br>
</span><span>> <<a href="mailto:behdad.esfahbod@gmail.com" target="_blank">behdad.esfahbod@gmail.com</a> <mailto:<a href="mailto:behdad.esfahbod@gmail.com" target="_blank">behdad.esfahbod@gmail.<wbr>com</a>>> wrote:<br>
><br>
>     Thanks Matthias.  I took a look at the patch.  Looks good!  Minor<br>
>     comments:<br>
><br>
>     - Moving the format towards CSS by default (no comma, no equal sign;<br>
>     just spaces!) while accepting both is what I do in HarfBuzz. Maybe<br>
>     advertise the same here,  UPDATE: I was wrong.  CSS uses comma as<br>
>     well.  It's the equal sign that they don't use, they use space.  We<br>
>     should accept both.<br>
><br>
><br>
> I've made the parser a bit more flexible. It now accepts variations like<br>
><br>
> wdth=200,wght=300<br>
> wdth 200, wght=300<br>
> wdth 200 , wght 300<br>
<br>
</span>Does it work with decimal commas?<br>
<div class="m_-5154014040981926892HOEnZb"><div></div></div></blockquote><div><br></div></span><div>The patch currently uses strtod, so yes, it will parse floating point numbers, but decimal separators spell trouble.<br></div></div>I could rewrite it to use strtod_l, if that is ok to use in cairo.<br></div></div>
<br>--<br>
cairo mailing list<br>
<a href="mailto:cairo@cairographics.org">cairo@cairographics.org</a><br>
<a href="https://lists.cairographics.org/mailman/listinfo/cairo" rel="noreferrer" target="_blank">https://lists.cairographics.<wbr>org/mailman/listinfo/cairo</a><br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">behdad<br><a href="http://behdad.org/" target="_blank">http://behdad.org/</a></div>
</div>