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

Relax some warnings

Browse Source
This commit is contained in:
Behdad Esfahbod 2013-12-04 22:51:27 -05:00
parent e5bee3716e
commit 650a38fffc
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Lib/fontTools/ttLib/tables/otTables.py
View File

@ -62,7 +62,11 @@ class Coverage(FormatSwitchingBaseTable):
try:
endID = font.getGlyphID(end, requireReal=True) + 1
except KeyError:
warnings.warn("Coverage table has end glyph ID out of range: %s." % end)
# Apparently some tools use 65535 to "match all" the range
if end != 'glyph65535':
warnings.warn("Coverage table has end glyph ID out of range: %s." % end)
# NOTE: We clobber out-of-range things here. There are legit uses for those,
# but none that we have seen in the wild.
endID = len(glyphOrder)
glyphs.extend(glyphOrder[glyphID] for glyphID in range(startID, endID))
else:
@ -224,6 +228,8 @@ class ClassDef(FormatSwitchingBaseTable):
endID = startID + len(classList)
if endID > len(glyphOrder):
warnings.warn("ClassDef table has entries for out of range glyph IDs: %s,%s." % (start, len(classList)))
# NOTE: We clobber out-of-range things here. There are legit uses for those,
# but none that we have seen in the wild.
endID = len(glyphOrder)
for glyphID, cls in zip(range(startID, endID), classList):
@ -243,7 +249,11 @@ class ClassDef(FormatSwitchingBaseTable):
try:
endID = font.getGlyphID(end, requireReal=True)
except KeyError:
warnings.warn("ClassDef table has end glyph ID out of range: %s." % end)
# Apparently some tools use 65535 to "match all" the range
if end != 'glyph65535':
warnings.warn("ClassDef table has end glyph ID out of range: %s." % end)
# NOTE: We clobber out-of-range things here. There are legit uses for those,
# but none that we have seen in the wild.
endID = len(glyphOrder)
for glyphID in range(startID, endID):
classDefs[glyphOrder[glyphID]] = cls