Merge pull request #3202 from fonttools/cff-latin1-strings

CFF: use latin1 so we can roundtrip non-ASCII in {Full,Font,Family}Name
fonttools · Jul 10, 2023 · 5ac1e5b · 5ac1e5b
2 parents 4660b8c + a8bd0ef
commit 5ac1e5b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Lib/fontTools/cffLib/__init__.py b/Lib/fontTools/cffLib/__init__.py
@@ -2172,9 +2172,9 @@ def xmlRead(self, name, attrs, content, parent):
     (0, "version", "SID", None, None),
     (1, "Notice", "SID", None, Latin1Converter()),
     ((12, 0), "Copyright", "SID", None, Latin1Converter()),
-    (2, "FullName", "SID", None, None),
-    ((12, 38), "FontName", "SID", None, None),
-    (3, "FamilyName", "SID", None, None),
+    (2, "FullName", "SID", None, Latin1Converter()),
+    ((12, 38), "FontName", "SID", None, Latin1Converter()),
+    (3, "FamilyName", "SID", None, Latin1Converter()),
     (4, "Weight", "SID", None, None),
     ((12, 1), "isFixedPitch", "number", 0, None),
     ((12, 2), "ItalicAngle", "number", 0, None),