Ensure opentype/monospace works with monospace OTF fonts

PeterDekkers · PeterDekkers · commit 5fa3df9c8465 · 2025-10-07T10:42:34.000+13:00
diff --git a/Lib/fontbakery/checks/opentype/monospace.py b/Lib/fontbakery/checks/opentype/monospace.py
@@ -166,10 +166,11 @@ def check_monospace(ttFont, glyph_metrics_stats):
                 f"{PANOSE_expected(family_type)}",
             )
 
-        num_glyphs = len(ttFont["glyf"].glyphs)
+        glyph_names = ttFont.getGlyphNames()
+        num_glyphs = len(glyph_names)
         unusually_spaced_glyphs = [
             g
-            for g in ttFont["glyf"].glyphs
+            for g in glyph_names
             if g not in [".notdef", ".null", "NULL"]
             and ttFont["hmtx"].metrics[g][0] != 0
             and ttFont["hmtx"].metrics[g][0] != most_common_width

Original file line number	Diff line number	Diff line change
`@@ -166,10 +166,11 @@ def check_monospace(ttFont, glyph_metrics_stats):`
`166`	`166`	`f"{PANOSE_expected(family_type)}",`
`167`	`167`	`)`
`168`	`168`
`169`		`- num_glyphs = len(ttFont["glyf"].glyphs)`
	`169`	`+ glyph_names = ttFont.getGlyphNames()`
	`170`	`+ num_glyphs = len(glyph_names)`
`170`	`171`	`unusually_spaced_glyphs = [`
`171`	`172`	`g`
`172`		`- for g in ttFont["glyf"].glyphs`
	`173`	`+ for g in glyph_names`
`173`	`174`	`if g not in [".notdef", ".null", "NULL"]`
`174`	`175`	`and ttFont["hmtx"].metrics[g][0] != 0`
`175`	`176`	`and ttFont["hmtx"].metrics[g][0] != most_common_width`