[compile] Support for beyond-64k fonts

Lib/ufo2ft/_compilers/baseCompiler.py

@@ -105,6 +105,13 @@ def postprocess(self, ttf, ufo, glyphSet, info=None):
             )
             kwargs = prune_unknown_kwargs(self.__dict__, postProcessor.process)
             ttf = postProcessor.process(**kwargs)
+            # Upgrade to the beyond-64k companion tables (GLYF/LOCA/MAXP/...)
+            # only on final outputs. While building a variable font,
+            # postProcessorClass is None for the interpolation masters (see
+            # _compileNeededSources), so they keep their lowercase glyf/loca/maxp
+            # tables and varLib sees a consistent table family; the merged VF is
+            # uppercased here when its own postprocess runs.
+            _maybe_uppercase_beyond64k(ttf)
         return ttf
 
     def compileFeatures(
@@ -481,3 +488,27 @@ def compile_variable_features(self, designSpaceDoc, ttFont, glyphSet):
 
         # Add back feature variations, as the code above would overwrite them.
         varLib.addGSUBFeatureVariations(ttFont, designSpaceDoc)
+
+
+def _maybe_uppercase_beyond64k(ttFont):
+    # maxp.numGlyphs is uint16: a count of 65536 overflows it even though every
+    # gid (0..0xFFFF) still fits. Guard on count > 0xFFFF, not gid width.
+    if len(ttFont.getGlyphOrder()) <= 0xFFFF and not _has_cubic_glyf(ttFont):
+        return
+
+    from fontTools.ttLib.beyond64k import upper_tables
+
+    upper_tables(ttFont)
+
+
+def _has_cubic_glyf(ttFont):
+    if "glyf" not in ttFont:
+        return False
+
+    from fontTools.ttLib.tables._g_l_y_f import flagCubic
+
+    glyf = ttFont["glyf"]
+    return any(
+        any(flag & flagCubic for flag in getattr(glyph, "flags", ()))
+        for glyph in glyf.glyphs.values()
+    )

Lib/ufo2ft/_compilers/interpolatableTTFCompiler.py

@@ -28,7 +28,7 @@ class InterpolatableTTFCompiler(BaseInterpolatableCompiler):
 
     def compileOutlines(self, ufo, glyphSet, layerName=None):
         kwargs = prune_unknown_kwargs(self.__dict__, self.outlineCompilerClass)
-        kwargs["glyphDataFormat"] = 0 if self.allQuadratic else 1
+        kwargs["allowCubic"] = not self.allQuadratic
         kwargs["tables"] = SPARSE_TTF_MASTER_TABLES if layerName else None
         # we want to keep coordinates as floats in glyf masters so that fonttools
         # can compute impliable on-curve points from unrounded coordinates before

Lib/ufo2ft/_compilers/ttfCompiler.py

@@ -23,6 +23,6 @@ class TTFCompiler(BaseCompiler):
 
     def compileOutlines(self, ufo, glyphSet):
         kwargs = prune_unknown_kwargs(self.__dict__, self.outlineCompilerClass)
-        kwargs["glyphDataFormat"] = 0 if self.allQuadratic else 1
+        kwargs["allowCubic"] = not self.allQuadratic
         outlineCompiler = self.outlineCompilerClass(ufo, glyphSet=glyphSet, **kwargs)
         return outlineCompiler.compile()

Lib/ufo2ft/outlineCompiler.py

@@ -1673,7 +1673,7 @@ def __init__(
         dropImpliedOnCurves=False,
         autoUseMyMetrics=True,
         roundCoordinates=True,
-        glyphDataFormat=0,
+        allowCubic=False,
         ftConfig=None,
         *,
         compilingVFDefaultSource=True,
@@ -1693,7 +1693,7 @@ def __init__(
         self.autoUseMyMetrics = autoUseMyMetrics
         self.dropImpliedOnCurves = dropImpliedOnCurves
         self.roundCoordinates = roundCoordinates
-        self.glyphDataFormat = glyphDataFormat
+        self.allowCubic = allowCubic
 
     def makeMissingRequiredGlyphs(self, font, glyphSet, sfntVersion, notdefGlyph=None):
         """
@@ -1728,7 +1728,6 @@ def compileGlyphs(self):
         allGlyphs = self.allGlyphs
         ttGlyphs = {}
         round = otRound if self.roundCoordinates else noRound
-        glyphDataFormat = self.glyphDataFormat
         for name in self.glyphOrder:
             glyph = allGlyphs[name]
             pen = TTGlyphPointPen(allGlyphs)
@@ -1743,14 +1742,13 @@ def compileGlyphs(self):
                     round=round,
                 )
                 if (
-                    glyphDataFormat == 0
+                    not self.allowCubic
                     and ttGlyph.numberOfContours > 0
                     and any(f & flagCubic for f in ttGlyph.flags)
                 ):
                     raise ValueError(
-                        f"{name!r} has cubic Bezier curves, but glyphDataFormat=0; "
-                        "either convert to quadratic (convertCubics=True) or use "
-                        "allQuadratic=False so that glyphDataFormat=1."
+                        f"{name!r} has cubic Bezier curves; either convert to "
+                        "quadratic (convertCubics=True) or use allQuadratic=False."
                     )
             ttGlyphs[name] = ttGlyph
         return ttGlyphs