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

glyf: only recalcBounds once in setCoordinates

Browse Source 
glyph.recalcBounds is called unconditionally a few lines below within the same
setCoordinates method, just after setting the new glyph's coordinates.
We don't need to call recalcBounds twice.
Only empty glyphs with numberOfContours == 0 may not have xMin set.
recalcBounds ensure it's set to 0 for those.
This commit is contained in:
Cosimo Lupo 2019-04-20 15:16:02 +01:00
parent 1e6f8bc39b
commit 7b5202cd79
No known key found for this signature in database
GPG Key ID: 179A8F0895A02F4F
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Lib/fontTools/ttLib/tables/_g_l_y_f.py
View File

@ -372,8 +372,6 @@ class table__g_l_y_f(DefaultTable.DefaultTable):
# Handle phantom points for (left, right, top, bottom) positions. # Handle phantom points for (left, right, top, bottom) positions.
assert len(coord) >= 4 assert len(coord) >= 4
if not hasattr(glyph, 'xMin'):
glyph.recalcBounds(self)
leftSideX = coord[-4][0] leftSideX = coord[-4][0]
rightSideX = coord[-3][0] rightSideX = coord[-3][0]
topSideY = coord[-2][1] topSideY = coord[-2][1]