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

[ttFont] Remove requireReal

Browse Source 
To my knowledge this is not used by anyone.

See https://github.com/fonttools/fonttools/issues/2334#issuecomment-902789001
This commit is contained in:
Behdad Esfahbod 2021-08-20 09:53:15 -06:00
parent d0d082e76e
commit 9988054f3c
2 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Lib/fontTools/misc/testTools.py
View File

@ -136,7 +136,7 @@ class MockFont(object):
self._reverseGlyphOrder = AllocatingDict({'.notdef': 0})
self.lazy = False
def getGlyphID(self, glyph, requireReal=None):
def getGlyphID(self, glyph):
gid = self._reverseGlyphOrder[glyph]
return gid

10
Lib/fontTools/ttLib/ttFont.py
View File

@ -544,11 +544,11 @@ class TTFont(object):
from fontTools.misc import textTools
return textTools.caselessSort(self.getGlyphOrder())
def getGlyphName(self, glyphID, requireReal=False):
def getGlyphName(self, glyphID):
try:
return self.getGlyphOrder()[glyphID]
except IndexError:
if requireReal or not self.allowVID:
if not self.allowVID:
# XXX The ??.W8.otf font that ships with OSX uses higher glyphIDs in
# the cmap table than there are glyphs. I don't think it's legal...
return "glyph%.5d" % glyphID
@ -563,7 +563,7 @@ class TTFont(object):
self.VIDDict[glyphID] = glyphName
return glyphName
def getGlyphID(self, glyphName, requireReal=False):
def getGlyphID(self, glyphName):
if not hasattr(self, "_reverseGlyphOrderDict"):
self._buildReverseGlyphOrderDict()
glyphOrder = self.getGlyphOrder()
@ -573,9 +573,7 @@ class TTFont(object):
self._buildReverseGlyphOrderDict()
return self.getGlyphID(glyphName)
else:
if requireReal:
raise KeyError(glyphName)
elif not self.allowVID:
if not self.allowVID:
# Handle glyphXXX only
if glyphName[:5] == "glyph":
try: