Add comment

iorsh · iorsh · commit e9fa9a3de7e1 · 2025-06-13T00:45:48.000+03:00
diff --git a/fontforge/parsettf.c b/fontforge/parsettf.c
@@ -6305,6 +6305,10 @@ static SplineFont *SFFillFromTTF(struct ttfinfo *info) {
 			info->bad_cff = true;
 		    }
 		    if ( info->isFixedPitch ) {
+			/* Modern Unicode fonts tend to have lots of zero-width
+			   characters like diacritics or control chars, e.g. LRM.
+			   These characters don't have an advance width and
+			   shouldn't affect pitch computation. */
 			bool valid = (sc->width==0 || sc->width==info->advanceWidthMax || sc->width==info->advanceWidthMax / 2);
 			if (!valid) {
 			    LogError(_("The advance width of %s (%d) does not match the font's advanceWidthMax (%d) and this is a fixed pitch font\n"),
