From freetype-devel  Mon Dec 18 21:39:24 2017
From: Matthias Clasen <matthias.clasen () gmail ! com>
Date: Mon, 18 Dec 2017 21:39:24 +0000
To: freetype-devel
Subject: Re: [ft-devel] Another font variation bug fix
Message-Id: <CAFwd_vBjCKEbwXwtpduuANQmXuxGhWsnR=bxD0gnXOpZRDvsGA () mail ! gmail ! com>
X-MARC-Message: https://marc.info/?l=freetype-devel&m=151363318401807
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--===============5725181607180274617=="

--===============5725181607180274617==
Content-Type: multipart/alternative; boundary="001a11455eac1b92150560a42f8e"

--001a11455eac1b92150560a42f8e
Content-Type: text/plain; charset="UTF-8"

On Mon, Dec 18, 2017 at 4:37 PM, Matthias Clasen <matthias.clasen@gmail.com>
wrote:

> the important change is the last hunk.
>
> We store coords[i] in the local variable coord, then we apply the clamping
> to coord, but then ...
> we use coords[i] as input for the normalization, so the clamping is lost.
> That is what my patch changes.
>
>
Oh, and I just notice now that the patch I sent included my debug changes
to replace FT_TRACE with printf.
That was of course not meant to be applied. Sorry for not noticing that
earlier.

--001a11455eac1b92150560a42f8e
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">On Mon, Dec 18, 2017 at 4:37 PM, Matthias Clasen <span dir=
=3D"ltr">&lt;<a href=3D"mailto:matthias.clasen@gmail.com" target=3D"_blank"=
>matthias.clasen@gmail.com</a>&gt;</span> wrote:<br><div class=3D"gmail_ext=
ra"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style=3D"m=
argin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir=3D"l=
tr"><div><div>the important change is the last hunk.<br><br></div>We store =
coords[i] in the local variable coord, then we apply the clamping to coord,=
 but then ...</div><div>we use coords[i] as input for the normalization, so=
 the clamping is lost.<br></div>That is what my patch changes.<br></div><br=
></blockquote><div><br></div><div>Oh, and I just notice now that the patch =
I sent included my debug changes to replace FT_TRACE with printf.</div><div=
>That was of course not meant to be applied. Sorry for not noticing that ea=
rlier.<br></div></div></div></div>

--001a11455eac1b92150560a42f8e--


--===============5725181607180274617==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel

--===============5725181607180274617==--