shithub: freetype+ttf2subf

Download patch

ref: 559727d5196633c735298fe16f7f1e835fe94ef7
parent: 3d3523f721591bc53851e42e8096615c71ab234e
author: Werner Lemberg <[email protected]>
date: Wed Feb 25 01:55:00 EST 2015

Change dimension fields in `FTC_ImageTypeRec' to unsigned type.

This doesn't break ABI.

* include/ftcache.h (FTC_ImageTypeRec): Use unsigned types for
`width' and `height'.

* docs/CHANGES: Document it.

git/fs: mount .git/fs: mount/attach disallowed
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,16 @@
 2015-02-25  Werner Lemberg  <[email protected]>
 
+	Change dimension fields in `FTC_ImageTypeRec' to unsigned type.
+
+	This doesn't break ABI.
+
+	* include/ftcache.h (FTC_ImageTypeRec): Use unsigned types for
+	`width' and `height'.
+
+	* docs/CHANGES: Document it.
+
+2015-02-25  Werner Lemberg  <[email protected]>
+
 	[cache] Don't use `labs'.
 
 	This is the only place in FreeType where this function was used.
--- a/docs/CHANGES
+++ b/docs/CHANGES
@@ -1,5 +1,5 @@
 
-CHANGES BETWEEN 2.5.5 and 2.5.6
+CHANGES BETWEEN 2.5.5 and 2.6
 
   II. IMPORTANT CHANGES
 
@@ -20,6 +20,17 @@
       Any calls to `FT_Load_Glyph' and similar API are safe and do not
       need the lock  to be held as  long as the same  `FT_Face' is not
       used from multiple threads at the same time.
+
+
+  III. MISCELLANEOUS
+
+    - Some  fields  in  the  `FTC_ImageTypeRec'  structure  have  been
+      changed from signed to unsigned  type, which better reflects the
+      actual usage.  It is also an additional means to protect against
+      malformed input.
+
+      This  change doesn't  break  the ABI;  however,  it might  cause
+      compiler warnings.
 
 
 ======================================================================
--- a/include/ftcache.h
+++ b/include/ftcache.h
@@ -667,8 +667,8 @@
   typedef struct  FTC_ImageTypeRec_
   {
     FTC_FaceID  face_id;
-    FT_Int      width;
-    FT_Int      height;
+    FT_UInt     width;
+    FT_UInt     height;
     FT_Int32    flags;
 
   } FTC_ImageTypeRec;