<div dir="ltr"><div><div><div><div><div><div>Thanks Matthias.  I took a look at the patch.  Looks good!  Minor comments:<br><br></div>- Moving the format towards CSS by default (no comma, no equal sign; just spaces!) while accepting both is what I do in HarfBuzz. Maybe advertise the same here,  UPDATE: I was wrong.  CSS uses comma as well.  It's the equal sign that they don't use, they use space.  We should accept both.<br><br></div>- Maybe make the variations member be "const char *" instead of "char *".<br><br></div>- In +cairo_font_options_set_variations(), typically in these kinds of functions I like copying then freeing the old value, such that the input can be the return value of get_variations() on the same object.<br><br>- (((uint8_t)p[0])<<24)|(((uint8_t)p[1])<<16)|(((uint8_t)p[2])<<8)|((uint8_t)p[3]); </div><div>FreeType has a macro for that: FT_MAKE_TAG()</div><div><br></div><div>Looks really good otherwise.  Let's get this in quickly.<br></div><div><br></div>Cheers,<br></div>b<br><br></div>(Generally I find it easier pushing a tree on github and reviewing there; also because I can just fix stuff and push my own tree.)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 13, 2017 at 2:41 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"><div><div>Over the last few days, Behdad and I worked together on supporting OpenType font variations in the Linux text rendering stack. This requires small additions to pango, fontconfig, and ... cairo.<br><br></div>The cairo patch is simple and ready (i'm still debugging some issues on the pango side).<br><br></div><div>It adds a new member to the font options struct, a string which contains the font variation settings.<br></div></div>
</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>