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

[varLib.mutator] Drop GDEF is nothing left in it

Browse Source 
Part of fixing https://github.com/fonttools/fonttools/pull/1079#issuecomment-338710803
This commit is contained in:
Behdad Esfahbod 2017-10-24 17:31:23 -04:00
parent 0b5a923ef8
commit 3fbc4688eb
3 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Lib/fontTools/varLib/merger.py
View File

@ -713,6 +713,12 @@ class MutatorMerger(AligningMerger):
if gdef.MarkGlyphSetsDef is None: if gdef.MarkGlyphSetsDef is None:
del gdef.MarkGlyphSetsDef del gdef.MarkGlyphSetsDef
gdef.Version = 0x00010000 gdef.Version = 0x00010000
if not (gdef.LigCaretList or
gdef.MarkAttachClassDef or
gdef.GlyphClassDef or
gdef.AttachList or
(gdef.Version >= 0x00010002 and gdef.MarkGlyphSetsDef)):
del font['GDEF']
@MutatorMerger.merger(ot.Anchor) @MutatorMerger.merger(ot.Anchor)
def merge(merger, self, lst): def merge(merger, self, lst):

4
Tests/varLib/data/test_results/Mutator.ttx
View File

@ -604,8 +604,4 @@
</extraNames> </extraNames>
</post> </post>
<GDEF>
<Version value="0x00010000"/>
</GDEF>
</ttFont> </ttFont>

4
Tests/varLib/data/test_results/Mutator_IUP-instance.ttx
View File

@ -297,8 +297,4 @@
</extraNames> </extraNames>
</post> </post>
<GDEF>
<Version value="0x00010000"/>
</GDEF>
</ttFont> </ttFont>