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: Behdad Esfahbod
Subject: Re: [ft-devel] [freetype2] master ce36777: [sfnt, truetype] Minor adjustments for OpenType 1.8.2.
Date: Tue, 1 Aug 2017 07:29:28 +0100


-    if ( face->header.Units_Per_EM == 0 )
+    /* OpenType 1.8.2 introduced limits to this value;    */
+    /* however, they make sense for older SFNT fonts also */
+    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.
 

-    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.


reply via email to

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