>> This has been replaced with
>>
>>   ascender_int = round(ascender_26.6);
>>   descender_int = round(descender_26.6);
>>   height_int = ascender_int - descender_int;
>>
>
> This seems to assume that height_26.6 was always ascender_26.6 + -
> descender_26.6.  I'm not sure this is always true.  IIRC, there are
> fonts where the line height is actually different from ascender -
> descender, they would be affected by this patch in surprising ways.

*blush* Uh, oh, I misunderstood what FreeType's `height' value is.
How embarassing.  As you correctly state it's *not* `ascender -
descender' but the baseline-to-baseline distance.

I'll undo the change (and add a comment).  Sorry for the noise.


    Werner

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

Reply via email to