[VARC] Update subsetting

Lib/fontTools/subset/__init__.py

@@ -2633,58 +2633,68 @@ def closure_glyphs(self, s):
 @_add_method(ttLib.getTableClass("VARC"))
 def subset_glyphs(self, s):
     indices = self.table.Coverage.subset(s.glyphs)
-    # TODO Subset MultiVarStore
+    self.table.VarCompositeGlyphs.VarCompositeGlyph = _list_subset(
-    self.table.VarCompositeGlyphs.glyphs = _list_subset(
+        self.table.VarCompositeGlyphs.VarCompositeGlyph, indices
-        self.table.VarCompositeGlyphs.glyphs, indices
     )
-    return bool(self.table.VarCompositeGlyphs.glyphs)
+    return bool(self.table.VarCompositeGlyphs.VarCompositeGlyph)
 
 
 @_add_method(ttLib.getTableClass("VARC"))
 def closure_glyphs(self, s):
-    if self.table.VarCompositeGlyphs:
+    if self.table.VarCompositeGlyphs is None:
-        allGlyphs = {
+        return
-            glyphName: glyph
-            for glyphName, glyph in zip(
-                self.table.Coverage.glyphs, self.table.VarCompositeGlyphs.glyphs
-            )
-        }
 
-        glyphs = s.glyphs
+    allGlyphs = {
-        new = set(s.glyphs)
+        glyphName: glyph
-        while new:
+        for glyphName, glyph in zip(
-            oldNew = new
+            self.table.Coverage.glyphs, self.table.VarCompositeGlyphs.VarCompositeGlyph
-            new = set()
+        )
-            for glyphName in oldNew:
+    }
-                glyph = allGlyphs.get(glyphName)
+
-                if glyph is None:
+    glyphs = s.glyphs
-                    continue
+    covered = set()
-                for comp in glyph.components:
+    new = set(glyphs)
-                    name = comp.glyphName
+    while new:
-                    if name not in glyphs:
+        oldNew = new
-                        glyphs.add(name)
+        new = set()
-                        new.add(name)
+        for glyphName in oldNew:
+            if glyphName in covered:
+                continue
+            glyph = allGlyphs.get(glyphName)
+            if glyph is None:
+                continue
+            for comp in glyph.components:
+                name = comp.glyphName
+                glyphs.add(name)
+                if name not in covered:
+                    new.add(name)
 
 
 @_add_method(ttLib.getTableClass("VARC"))
 def prune_post_subset(self, font, options):
     table = self.table
 
-    if not hasattr(table, "MultiVarStore"):
-        return True
-
     store = table.MultiVarStore
+    if store is not None:
+        usedVarIdxes = set()
+        table.collect_varidxes(usedVarIdxes)
+        varidx_map = store.subset_varidxes(usedVarIdxes)
+        table.remap_varidxes(varidx_map)
 
-    usedVarIdxes = set()
+    axisIndicesList = table.AxisIndicesList.Item
-
+    if axisIndicesList is not None:
-    # Collect.
+        usedIndices = set()
-    table.collect_varidxes(usedVarIdxes)
+        for glyph in table.VarCompositeGlyphs.VarCompositeGlyph:
-
+            for comp in glyph.components:
-    # Subset.
+                if comp.axisIndicesIndex is not None:
-    varidx_map = store.subset_varidxes(usedVarIdxes)
+                    usedIndices.add(comp.axisIndicesIndex)
-
+        usedIndices = sorted(usedIndices)
-    # Map.
+        table.AxisIndicesList.Item = _list_subset(axisIndicesList, usedIndices)
-    table.remap_varidxes(varidx_map)
+        mapping = {old: new for new, old in enumerate(usedIndices)}
+        for glyph in table.VarCompositeGlyphs.VarCompositeGlyph:
+            for comp in glyph.components:
+                if comp.axisIndicesIndex is not None:
+                    comp.axisIndicesIndex = mapping[comp.axisIndicesIndex]
 
     return True
 

Lib/fontTools/varLib/multiVarStore.py

@@ -236,16 +236,16 @@ ot.MultiVarStore.get_supports = MultiVarStore_get_supports
 
 
 def VARC_collect_varidxes(self, varidxes):
-    for glyph in self.VarCompositeGlyphs.glyphs:
+    for glyph in self.VarCompositeGlyphs.VarCompositeGlyph:
         for component in glyph.components:
-            varidxes.add(component.locationVarIndex)
+            varidxes.add(component.axisValuesVarIndex)
             varidxes.add(component.transformVarIndex)
 
 
 def VARC_remap_varidxes(self, varidxes_map):
-    for glyph in self.VarCompositeGlyphs.glyphs:
+    for glyph in self.VarCompositeGlyphs.VarCompositeGlyph:
         for component in glyph.components:
-            component.locationVarIndex = varidxes_map[component.locationVarIndex]
+            component.axisValuesVarIndex = varidxes_map[component.axisValuesVarIndex]
             component.transformVarIndex = varidxes_map[component.transformVarIndex]