<div dir="ltr">On Thu, Sep 28, 2017 at 7:03 PM, Behdad Esfahbod <span dir="ltr"><<a href="mailto:behdad.esfahbod@gmail.com" target="_blank">behdad.esfahbod@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>+Khaled,Werner<br><br></div>Two things:<br><br></div>  * Werner, please please please remove code that change face->face_index.  It's bad bad bad.<br><br></div>  * With Matthias's branch, we break Khaled's code, ie. if user creates FT_Face, sets variations on it, and creates cairo_face_t from, we reset / ignore those variations.  The fix would require savings the variations that were on the face when face was created, and use those instead of instance-defaults in apply_variations().  Matthias, any chance you can do this?<br></div></div></blockquote><div><br></div><div>I'll have a look. <br></div></div><br></div></div>