Re: [Devel] FIX? [was: Non-integer bbox for T1 font?]

[Werner LEMBERG]

> I just looked at the use of T1_CoordArray: it was only used in
> parse_font_bbox.  The same function (under a different name:
> CID_ToFixedArray) is used in only parse_font_bbox from the cid
> module.  So I think these can be replaced to calls of
> {T1,CID}_ToFixedArray.

Without looking into the code: Can this be unified and moved to PSaux?

> These values are FT_Shorts, and as type1.font_bbox.yMax is an
> FT_Fixed, the cast is *bad* thing to do.  The least significant byte
> (the fractional part) is used, when the most significant byte (the
> integer part) should be used instead.  Perhaps root->ascender should
> be assigned as follows:
> 
>   root->ascender     = face->type1.font_bbox.yMax >> 16;
> 
> etc...

Looks better.  But shouldn't bbox values always be rounded to the
larger integer?


    Werner