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

[glyf] fix rounding of bbox for nested transformed components

Browse Source 
when getCoordinates is asked to round, e.g. from recalcBounds, only round the simple glyphs' coordinates; don't do any rounding for the coordinates of component glyphs otherwise rounding errors would accummulate if a composite glyph contains transformed/nested components
This commit is contained in:
Cosimo Lupo 2025-01-24 15:10:18 +00:00
parent e02349c7ce
commit 4ea10eb88f
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -1258,7 +1258,11 @@ class Glyph(object):
""" """
if self.numberOfContours > 0: if self.numberOfContours > 0:
return self.coordinates, self.endPtsOfContours, self.flags coordinates = self.coordinates
if round is not noRound:
coordinates = GlyphCoordinates(coordinates)
coordinates.toInt(round=round)
return coordinates, self.endPtsOfContours, self.flags
elif self.isComposite(): elif self.isComposite():
# it's a composite # it's a composite
allCoords = GlyphCoordinates() allCoords = GlyphCoordinates()
@ -1276,7 +1280,6 @@ class Glyph(object):
% compo.glyphName % compo.glyphName
) )
coordinates = GlyphCoordinates(coordinates) coordinates = GlyphCoordinates(coordinates)
coordinates.toInt(round=round)
if hasattr(compo, "firstPt"): if hasattr(compo, "firstPt"):
# component uses two reference points: we apply the transform _before_ # component uses two reference points: we apply the transform _before_
# computing the offset between the points # computing the offset between the points