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

List:       freetype-devel
Subject:    Re: [ft-devel] Another font variation bug fix
From:       Matthias Clasen <matthias.clasen () gmail ! com>
Date:       2017-12-18 21:39:24
Message-ID: CAFwd_vBjCKEbwXwtpduuANQmXuxGhWsnR=bxD0gnXOpZRDvsGA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


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.

[Attachment #5 (text/html)]

<div dir="ltr">On Mon, Dec 18, 2017 at 4:37 PM, Matthias Clasen <span \
dir="ltr">&lt;<a href="mailto:matthias.clasen@gmail.com" \
target="_blank">matthias.clasen@gmail.com</a>&gt;</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>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 earlier.<br></div></div></div></div>



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


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