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

Populate defaults even for tables that have postRead

Browse Source 
Implement custom populateDefaults for them.
This commit is contained in:
Behdad Esfahbod 2017-02-03 14:33:57 -08:00
parent b887860b65
commit e45297bf4b
2 changed files with 31 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Lib/fontTools/ttLib/tables/otConverters.py
View File

@ -361,8 +361,9 @@ class Struct(BaseConverter):
else:
pass
table.populateDefaults(propagator=getattr(font, '_propagator', None))
if noPostRead:
table.populateDefaults(propagator=getattr(font, '_propagator', None))
if cleanPropagation:
for conv in table.getConverters():
if conv.isPropagated:

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

@ -37,6 +37,10 @@ class Coverage(FormatSwitchingBaseTable):
# manual implementation to get rid of glyphID dependencies
def populateDefaults(self, propagator=None):
if not hasattr(self, 'glyphs'):
self.glyphs = []
def postRead(self, rawTable, font):
if self.Format == 1:
# TODO only allow glyphs that are valid?
@ -136,6 +140,10 @@ class Coverage(FormatSwitchingBaseTable):
class VarIdxMap(BaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'mapping'):
self.mapping = []
def postRead(self, rawTable, font):
assert (rawTable['EntryFormat'] & 0xFFC0) == 0
self.mapping = rawTable['mapping']
@ -221,6 +229,10 @@ class VarData(BaseTable):
class SingleSubst(FormatSwitchingBaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'mapping'):
self.mapping = {}
def postRead(self, rawTable, font):
mapping = {}
input = _getGlyphsFromCoverageTable(rawTable["Coverage"])
@ -295,6 +307,11 @@ class SingleSubst(FormatSwitchingBaseTable):
class MultipleSubst(FormatSwitchingBaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'mapping'):
self.mapping = {}
def postRead(self, rawTable, font):
mapping = {}
if self.Format == 1:
@ -368,6 +385,10 @@ class MultipleSubst(FormatSwitchingBaseTable):
class ClassDef(FormatSwitchingBaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'classDefs'):
self.classDefs = {}
def postRead(self, rawTable, font):
classDefs = {}
glyphOrder = font.getGlyphOrder()
@ -485,6 +506,10 @@ class ClassDef(FormatSwitchingBaseTable):
class AlternateSubst(FormatSwitchingBaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'alternates'):
self.alternates = {}
def postRead(self, rawTable, font):
alternates = {}
if self.Format == 1:
@ -551,6 +576,10 @@ class AlternateSubst(FormatSwitchingBaseTable):
class LigatureSubst(FormatSwitchingBaseTable):
def populateDefaults(self, propagator=None):
if not hasattr(self, 'ligatures'):
self.ligatures = {}
def postRead(self, rawTable, font):
ligatures = {}
if self.Format == 1: