shithub: freetype+ttf2subf

Download patch

ref: 61d50756b6ddc6cce459f64445abf19f09d18253
parent: 339c451ad0e8a743b59fb2bd0589fe9362b45024
author: Alexei Podtelezhnikov <[email protected]>
date: Fri Nov 2 16:42:25 EDT 2018

* src/truetype/ttgxvar.c (ft_var_get_item_delta): Fixed logic.

Reported and tested by Behdad.

git/fs: mount .git/fs: mount/attach disallowed
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2018-11-02  Alexei Podtelezhnikov  <[email protected]>
+
+	* src/truetype/ttgxvar.c (ft_var_get_item_delta): Fixed logic.
+
+	Reported and tested by Behdad.
+
 2018-11-02  Shailesh Mistry  <[email protected]>
 
 	[autofit] Prevent SEGV.
--- a/src/truetype/ttgxvar.c
+++ b/src/truetype/ttgxvar.c
@@ -68,8 +68,6 @@
 
 
   /* some macros we need */
-#define FT_FIXED_ONE  ( (FT_Fixed)0x10000 )
-
 #define FT_fdot14ToFixed( x )                \
         ( (FT_Fixed)( (FT_ULong)(x) << 2 ) )
 #define FT_intToFixed( i )                    \
@@ -884,7 +882,7 @@
     /* outer loop steps through master designs to be blended */
     for ( master = 0; master < varData->regionIdxCount; master++ )
     {
-      FT_Fixed  scalar      = FT_FIXED_ONE;
+      FT_Fixed  scalar      = 0x10000L;
       FT_UInt   regionIndex = varData->regionIndices[master];
 
       GX_AxisCoords  axis = itemStore->varRegionList[regionIndex].axisList;
@@ -908,6 +906,9 @@
         else if ( axis->peakCoord == 0 )
           continue;
 
+        else if ( face->blend->normalizedcoords[j] == axis->peakCoord )
+          continue;
+
         /* ignore this region if coords are out of range */
         else if ( face->blend->normalizedcoords[j] <= axis->startCoord ||
                   face->blend->normalizedcoords[j] >= axis->endCoord   )
@@ -915,9 +916,6 @@
           scalar = 0;
           break;
         }
-
-        else if ( face->blend->normalizedcoords[j] == axis->peakCoord )
-          continue;
 
         /* cumulative product of all the axis scalars */
         else if ( face->blend->normalizedcoords[j] < axis->peakCoord )