[varStore] Another bugfix in optimize()

fonttools · May 24, 2023 · 0677c0e · 0677c0e
1 parent e4281b0
commit 0677c0e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/Lib/fontTools/varLib/varStore.py b/Lib/fontTools/varLib/varStore.py
@@ -422,6 +422,8 @@ def _find_yourself_best_new_encoding(self, done_by_width):
             else:
                 new_encoding = None
             self.best_new_encoding = new_encoding
+            if new_encoding:
+                break
 
 
 class _EncodingDict(dict):

diff --git a/Tests/varLib/varStore_test.py b/Tests/varLib/varStore_test.py
@@ -108,6 +108,19 @@ def buildAxis(axisTag):
             2,
             186,
         ),
+        (
+            5,
+            [
+                [10, 11, 0, 0, 20],
+                [10, 300, 0, 0, 20],
+                [10, 301, 0, 0, 20],
+                [10, 302, 0, 0, 20],
+                [10, 303, 0, 0, 20],
+                [10, 304, 0, 0, 20],
+            ],
+            1,
+            180,
+        ),
     ],
 )
 def test_optimize(numRegions, varData, expectedNumVarData, expectedBytes):