On Thu, Jan 24, 2013 at 2:23 AM, Wojciech Mamrak <wmam...@gmail.com> wrote:
> 2013/1/23 Alexei Podtelezhnikov <apodt...@gmail.com>:
>> On Wed, Jan 23, 2013 at 10:50 AM, Wojciech Mamrak <wmam...@gmail.com> wrote:
>>> What is the largest possible "height" change between the old and new
>>> implementation? Can it exceed one pixel?
>>
>> It should not exceed 1 pixel unless the height specified in the font
>> differs from (acsender - descender) by more than 1 pixel.
>
> And why would anyone make it differ from that? :)

By design or by mistake...
I think that David's idea to make the change conditional can work
except we should use a softer condition:

  if ( FT_ABS( height_26.6 - (ascender_26.6 - descender_26.6) ) < 64 ) {
    height_int = ascender_int - descender_int;
  } else {
    height_int = FT_ROUND( height_26.6);
  }

This way we respect height that is very different from  (ascender - descender).

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

Reply via email to