freetype-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [ft-devel] [freetype2] master ce36777: [sfnt, truetype] Minor adjust


From: Werner LEMBERG
Subject: Re: [ft-devel] [freetype2] master ce36777: [sfnt, truetype] Minor adjustments for OpenType 1.8.2.
Date: Tue, 01 Aug 2017 08:41:43 +0200 (CEST)

>> +    if ( face->header.Units_Per_EM <    16 ||
>> +         face->header.Units_Per_EM > 16384 )
>>
> 
> Humm.  I'm pretty sure those limits have been there for many
> years... I've had them in HarfBuzz since 2010.

Well, such limits are sensible, but they were not documented.

>> -    else if ( loader->n_contours == -1 )
>> +    else if ( loader->n_contours < 0 )
> 
> Ouch.  Not sure if this was intentional... I was hoping we can use
> other numbers for future extensions of components.

Interesting.  What do you have in mind?


    Werner



reply via email to

[Prev in Thread] Current Thread [Next in Thread]