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

[subset] actually, ignore the width of emptied charstrings

Browse Source 
basically, implies setting them to their defaultWidthX, which is the most efficient
way to store these (unnecessary) piece of data.
This commit is contained in:
Cosimo Lupo 2019-01-16 16:01:12 +00:00
parent 3e400c8828
commit 6b4474b2c4
No known key found for this signature in database
GPG Key ID: 59D54DB0C9976482
1 changed files with 14 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
Lib/fontTools/subset/cff.py
View File

@ -66,29 +66,25 @@ def closure_glyphs(self, s):
s.glyphs.update(components)
decompose = components
def _empty_charstring(font, glyphName, isCFF2, width=None):
def _empty_charstring(font, glyphName, isCFF2, ignoreWidth=False):
c, fdSelectIndex = font.CharStrings.getItemAndSelector(glyphName)
if isCFF2:
if isCFF2 or ignoreWidth:
# CFF2 charstrings have no widths nor 'endchar' operators
c.decompile()
c.program = []
return
if hasattr(font, 'FDArray') and font.FDArray is not None:
private = font.FDArray[fdSelectIndex].Private
c.program = [] if isCFF2 else ['endchar']
else:
private = font.Private
dfltWdX = private.defaultWidthX
nmnlWdX = private.nominalWidthX
if width is None:
if hasattr(font, 'FDArray') and font.FDArray is not None:
private = font.FDArray[fdSelectIndex].Private
else:
private = font.Private
dfltWdX = private.defaultWidthX
nmnlWdX = private.nominalWidthX
pen = NullPen()
c.draw(pen) # this will set the charstring's width
width = c.width
else:
c.decompile()
if width != dfltWdX:
c.program = [width - nmnlWdX, 'endchar']
else:
c.program = ['endchar']
if c.width != dfltWdX:
c.program = [c.width - nmnlWdX, 'endchar']
else:
c.program = ['endchar']
@_add_method(ttLib.getTableClass('CFF '))
def prune_pre_subset(self, font, options):
@ -118,7 +114,7 @@ def subset_glyphs(self, s):
if s.options.retain_gids:
for g in s.glyphs_emptied:
_empty_charstring(font, g, isCFF2=cff.major > 1, width=0)
_empty_charstring(font, g, isCFF2=cff.major > 1, ignoreWidth=True)
else:
# Load all glyphs
for g in font.charset: