shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[OS/2] don't error getting/setting codepages if version=0

Browse Source 
return empty set, or set the bits and bump the version to match
This commit is contained in:
Cosimo Lupo 2023-11-15 12:20:24 +00:00
parent 49bd8adbc4
commit f518e5a704
No known key found for this signature in database
GPG Key ID: DF65A8A5A119C9A8
2 changed files with 21 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
Lib/fontTools/ttLib/tables/O_S_2f_2.py
View File

@ -343,6 +343,8 @@ class table_O_S_2f_2(DefaultTable.DefaultTable):
def getCodePageRanges(self):
"""Return the set of 'ulCodePageRange*' bits currently enabled."""
bits = set()
if self.version < 1:
return bits
ul1, ul2 = self.ulCodePageRange1, self.ulCodePageRange2
for i in range(32):
if ul1 & (1 << i):
@ -361,6 +363,8 @@ class table_O_S_2f_2(DefaultTable.DefaultTable):
ul2 |= 1 << (bit - 32)
else:
raise ValueError(f"expected 0 <= int <= 63, found: {bit:r}")
if self.version < 1:
self.version = 1
self.ulCodePageRange1, self.ulCodePageRange2 = ul1, ul2
def recalcCodePageRanges(self, ttFont, pruneOnly=False):

17
Tests/ttLib/tables/O_S_2f_2_test.py
View File

@ -8,6 +8,7 @@ class OS2TableTest(unittest.TestCase):
def makeOS2_cmap(mapping):
font = TTFont()
font["OS/2"] = os2 = newTable("OS/2")
os2.version = 4
font["cmap"] = cmap = newTable("cmap")
st = getTableModule("cmap").CmapSubtable.newSubtable(4)
st.platformID, st.platEncID, st.language = 3, 1, 0
@ -64,6 +65,11 @@ class OS2TableTest(unittest.TestCase):
def test_getCodePageRanges(self):
table = table_O_S_2f_2()
# version 0 doesn't define these fields so by definition defines no cp ranges
table.version = 0
self.assertEqual(table.getCodePageRanges(), set())
# version 1 and above do contain ulCodePageRange1 and 2 fields
table.version = 1
table.ulCodePageRange1 = 0xFFFFFFFF
table.ulCodePageRange2 = 0xFFFFFFFF
bits = table.getCodePageRanges()
@ -72,6 +78,7 @@ class OS2TableTest(unittest.TestCase):
def test_setCodePageRanges(self):
table = table_O_S_2f_2()
table.version = 4
table.ulCodePageRange1 = 0
table.ulCodePageRange2 = 0
bits = set(range(64))
@ -84,6 +91,16 @@ class OS2TableTest(unittest.TestCase):
with self.assertRaises(ValueError):
table.setCodePageRanges([255])
def test_setCodePageRanges_bump_version(self):
# Setting codepage ranges on a OS/2 table version 0 automatically makes it
# a version 1 table
table = table_O_S_2f_2()
table.version = 0
self.assertEqual(table.getCodePageRanges(), set())
table.setCodePageRanges({0, 1, 2})
self.assertEqual(table.getCodePageRanges(), {0, 1, 2})
self.assertEqual(table.version, 1)
def test_recalcCodePageRanges(self):
font, os2, cmap = self.makeOS2_cmap(
{ord("A"): "A", ord("Ά"): "Alphatonos", ord("Б"): "Be"}