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

[glyf_test] Test that composite referencing empty glyph also gets zero bounds

Browse Source 
Add reproducer for possible regression in Glyph.recalcBounds introduced with https://github.com/fonttools/fonttools/pull/3301
This commit is contained in:
Cosimo Lupo 2023-11-14 16:30:26 +00:00
parent 3d6a55d536
commit aaed718e41
No known key found for this signature in database
GPG Key ID: DF65A8A5A119C9A8
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Tests/ttLib/tables/_g_l_y_f_test.py
View File

@ -562,6 +562,23 @@ class GlyphTest:
assert glyphSet["percent"].getCompositeMaxpValues(glyphSet)[2] == 2
assert glyphSet["perthousand"].getCompositeMaxpValues(glyphSet)[2] == 2
def test_recalcBounds_empty_components(self):
glyphSet = {}
pen = TTGlyphPen(glyphSet)
# empty simple glyph
foo = glyphSet["foo"] = pen.glyph()
# use the empty 'foo' glyph as a component in 'bar' with some x/y offsets
pen.addComponent("foo", (1, 0, 0, 1, -80, 50))
bar = glyphSet["bar"] = pen.glyph()
foo.recalcBounds(glyphSet)
bar.recalcBounds(glyphSet)
# we expect both the empty simple glyph and the composite referencing it
# to have empty bounding boxes (0, 0, 0, 0) no matter the component's shift
assert (foo.xMin, foo.yMin, foo.xMax, foo.yMax) == (0, 0, 0, 0)
assert (bar.xMin, bar.yMin, bar.xMax, bar.yMax) == (0, 0, 0, 0)
class GlyphComponentTest:
def test_toXML_no_transform(self):