[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cairo
Subject:    Re: [cairo] OpenType font variations and cairo
From:       Matthias Clasen <matthias.clasen () gmail ! com>
Date:       2017-09-21 11:22:36
Message-ID: CAFwd_vDQhhny+w8CSyrkGvdeDQWCOFQq+j-PSufJdSwKmJDqjg () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


On Thu, Sep 21, 2017 at 5:42 AM, Adrian Johnson <ajohnson@redneon.com>
wrote:

> On 21/09/17 12:03, Matthias Clasen wrote:
> > On Wed, Sep 20, 2017 at 6:15 PM, Behdad Esfahbod
> > <behdad.esfahbod@gmail.com <mailto:behdad.esfahbod@gmail.com>> wrote:
> >
> >     The test was really broken...
> >
> >     Ok, I've pushed some things there that should help Matthias move
> foward:
> >
> >       https://github.com/behdad/cairo/commits/font-variations
> >     <https://github.com/behdad/cairo/commits/font-variations>
> >
> >
> > Thanks for those fixes, Behdad. Taking another look at the test, I'm not
> > sure what I was thinking - it was checking the width axis, but the font
> > we use only has weight and contrast as axes.
> > i've pushed all the fixes to
> >
> > https://github.com/matthiasclasen/cairo/tree/wip/
> matthiasc/font-variations
> >
> > Does this work for you now, Adrian ?
>
> It works with freetype 2.8.0, but not 2.8.1. Here are the results of
> testing with different combinations of freetype and fontconfig.
>
> freetype 2.8.0, fontconfig 2.11.0 : pass
>
> freetype 2.8.0, fontconfig 2.12.5 : pass
>
> freetype 2.8.0, Behdad's fontconfig (2d0063948a44) : pass
>
> freetype 2.8.1, fontconfig 2.11.0 : fail
>
>   axis Weight, value 300
>   axis CNTR, value 0
>   axis Weight, value 0
>   axis CNTR, value 0
>   Axis Weight: not expected value (0 != 300.5)
>
> freetype 2.8.1, fontconfig 2.12.5 : fail
>
>   axis Weight, value 300
>   axis CNTR, value 0
>   axis Weight, value 0
>   axis CNTR, value 0
>   Axis Weight: not expected value (0 != 300.5)
>
> freetype 2.8.1, Behdad's fontconfig (2d0063948a44) : fail
>
>   axis Weight, value 300
>   axis CNTR, value 0
>   axis Weight, value 0
>   axis CNTR, value 0
>   Axis Weight: not expected value (0 != 300.5)
>
>

I'm seeing those failures with freetype 2.8.1 too. They are somewhat
mysterious - if I reorder the tests different ones fail. The fact that they
sometimes report values that are nowhere to be found in the input, and
sometime reports 0, which is not in the allowed range for the axis
makes me think that something goes wrong in freetype.

[Attachment #5 (text/html)]

<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep \
21, 2017 at 5:42 AM, Adrian Johnson <span dir="ltr">&lt;<a \
href="mailto:ajohnson@redneon.com" \
target="_blank">ajohnson@redneon.com</a>&gt;</span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><span class="">On 21/09/17 12:03, Matthias Clasen wrote:<br> \
&gt; On Wed, Sep 20, 2017 at 6:15 PM, Behdad Esfahbod<br> </span><span class="">&gt; \
&lt;<a href="mailto:behdad.esfahbod@gmail.com">behdad.esfahbod@gmail.com</a> \
&lt;mailto:<a href="mailto:behdad.esfahbod@gmail.com">behdad.esfahbod@gmail.<wbr>com</a>&gt;&gt; \
wrote:<br> &gt;<br>
&gt;        The test was really broken...<br>
&gt;<br>
&gt;        Ok, I&#39;ve pushed some things there that should help Matthias move \
foward:<br> &gt;<br>
&gt;           <a href="https://github.com/behdad/cairo/commits/font-variations" \
rel="noreferrer" target="_blank">https://github.com/behdad/<wbr>cairo/commits/font-variations</a><br>
 &gt;        &lt;<a href="https://github.com/behdad/cairo/commits/font-variations" \
rel="noreferrer" target="_blank">https://github.com/behdad/<wbr>cairo/commits/font-variations</a>&gt;<br>
 &gt;<br>
&gt;<br>
&gt; Thanks for those fixes, Behdad. Taking another look at the test, I&#39;m not<br>
&gt; sure what I was thinking - it was checking the width axis, but the font<br>
&gt; we use only has weight and contrast as axes.<br>
&gt; i&#39;ve pushed all the fixes to<br>
&gt;<br>
&gt; <a href="https://github.com/matthiasclasen/cairo/tree/wip/matthiasc/font-variations" \
rel="noreferrer" target="_blank">https://github.com/<wbr>matthiasclasen/cairo/tree/wip/<wbr>matthiasc/font-variations</a><br>
 &gt;<br>
&gt; Does this work for you now, Adrian ?<br>
<br>
</span>It works with freetype 2.8.0, but not 2.8.1. Here are the results of<br>
testing with different combinations of freetype and fontconfig.<br>
<br>
freetype 2.8.0, fontconfig 2.11.0 : pass<br>
<br>
freetype 2.8.0, fontconfig 2.12.5 : pass<br>
<br>
freetype 2.8.0, Behdad&#39;s fontconfig (2d0063948a44) : pass<br>
<br>
freetype 2.8.1, fontconfig 2.11.0 : fail<br>
<br>
   axis Weight, value 300<br>
   axis CNTR, value 0<br>
   axis Weight, value 0<br>
   axis CNTR, value 0<br>
   Axis Weight: not expected value (0 != 300.5)<br>
<br>
freetype 2.8.1, fontconfig 2.12.5 : fail<br>
<br>
   axis Weight, value 300<br>
   axis CNTR, value 0<br>
   axis Weight, value 0<br>
   axis CNTR, value 0<br>
   Axis Weight: not expected value (0 != 300.5)<br>
<br>
freetype 2.8.1, Behdad&#39;s fontconfig (2d0063948a44) : fail<br>
<br>
   axis Weight, value 300<br>
   axis CNTR, value 0<br>
   axis Weight, value 0<br>
   axis CNTR, value 0<br>
   Axis Weight: not expected value (0 != 300.5)<br>
<br>
</blockquote></div></div><div class="gmail_extra"><br></div><div \
class="gmail_extra"><br></div><div class="gmail_extra">I&#39;m seeing those failures \
with freetype 2.8.1 too. They are somewhat mysterious - if I reorder the tests \
different ones fail. The fact that they</div><div class="gmail_extra">sometimes \
report values that are nowhere to be found in the input, and sometime reports 0, \
which is not in the allowed range for the axis</div><div class="gmail_extra">makes me \
think that something goes wrong in freetype.<br></div></div>


[Attachment #6 (text/plain)]

-- 
cairo mailing list
cairo@cairographics.org
https://lists.cairographics.org/mailman/listinfo/cairo

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic