Re: [ft-devel] Another font variation bug fix

[Matthias Clasen]

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.

On Mon, Dec 18, 2017 at 2:48 PM, Werner LEMBERG <address@hidden> wrote:

Thanks for the patch, which I've applied with slight modifications.

> Subject: [PATCH] Fix clamping of font variation axis values
>
> We were clamping the value correctly here, but then used the
> original value for normalization anyway.

This subject line and the omment I don't understand.  Your related
code is just a minor beautification that doesn't change any
functionality.  Am I missing something?


    Werner