[fonttools/fonttools] cb8d62: Add regression test for reading glyph with glyphOb...

[Cosimo Lupo]

Branch: refs/heads/fix-gliflib-no-glyph-object
Home: https://github.com/fonttools/fonttools
Commit: cb8d62200afed262f25682a2a30465a182e8029a
https://github.com/fonttools/fonttools/commit/cb8d62200afed262f25682a2a30465a182e8029a
Author: Cosimo Lupo <cl...@google.com>
Date: 2026-01-15 (Thu, 15 Jan 2026)

Changed paths:
M Tests/ufoLib/glifLib_test.py

Log Message:
-----------
Add regression test for reading glyph with glyphObject=None

Test that pointPen still receives drawing commands when reading a glyph
with glyphObject=None. This is a valid use case for drawing outlines
without populating glyph attributes.

Reproduces #4030



To unsubscribe from these emails, change your notification settings at https://github.com/fonttools/fonttools/settings/notifications

[Cosimo Lupo]

Branch: refs/heads/main

Home: https://github.com/fonttools/fonttools
Commit: cb8d62200afed262f25682a2a30465a182e8029a
https://github.com/fonttools/fonttools/commit/cb8d62200afed262f25682a2a30465a182e8029a
Author: Cosimo Lupo <cl...@google.com>
Date: 2026-01-15 (Thu, 15 Jan 2026)

Changed paths:
M Tests/ufoLib/glifLib_test.py

Log Message:
-----------
Add regression test for reading glyph with glyphObject=None

Test that pointPen still receives drawing commands when reading a glyph
with glyphObject=None. This is a valid use case for drawing outlines
without populating glyph attributes.

Reproduces #4030

Commit: 1d9551817aa4f324b9c6ea368c83c4243e929567
https://github.com/fonttools/fonttools/commit/1d9551817aa4f324b9c6ea368c83c4243e929567

Author: Cosimo Lupo <cl...@google.com>
Date: 2026-01-15 (Thu, 15 Jan 2026)

Changed paths:

M Lib/fontTools/ufoLib/glifLib.py

Log Message:
-----------
Fix reading glyph outline when glyphObject is None

Restore the original placement of `elif glyphObject is None: continue`
after the outline processing block. This allows drawing via pointPen
even when no glyphObject is provided.

The check was incorrectly moved before all element processing in
commit 7f5abe600 (PR #3875), which caused outlines to be silently
skipped when glyphObject=None.

Fixes #4030


Commit: e2fa0c82d1ce4ce63ae7f63325036c43ceb30803
https://github.com/fonttools/fonttools/commit/e2fa0c82d1ce4ce63ae7f63325036c43ceb30803
Author: Cosimo Lupo <cos...@anthrotype.com>

Date: 2026-01-15 (Thu, 15 Jan 2026)

Changed paths:

M Lib/fontTools/ufoLib/glifLib.py

M Tests/ufoLib/glifLib_test.py

Log Message:
-----------

Merge pull request #4031 from fonttools/fix-gliflib-no-glyph-object

[glifLib] Fix regression reading glyph outlines with glyphObject=None


Compare: https://github.com/fonttools/fonttools/compare/5716f2c41ad1...e2fa0c82d1ce