[fonttools/fonttools] 117123: [feaLib] raise FeatureLibError for a non-ASCII OS/...

0 views
Skip to first unread message

Cosimo Lupo

unread,
Oct 9, 2026, 11:48:48 AM (17 hours ago) Oct 9
to fontto...@googlegroups.com
Branch: refs/heads/main
Home: https://github.com/fonttools/fonttools
Commit: 11712326e91696a0bfd16838f45cf5e86eebe1e4
https://github.com/fonttools/fonttools/commit/11712326e91696a0bfd16838f45cf5e86eebe1e4
Author: RavSinghChandan <ravcha...@gmail.com>
Date: 2026-10-09 (Fri, 09 Oct 2026)

Changed paths:
M Lib/fontTools/feaLib/parser.py
M Tests/feaLib/parser_test.py

Log Message:
-----------
[feaLib] raise FeatureLibError for a non-ASCII OS/2 Vendor

table OS/2 { Vendor "éé"; } OS/2; compiled, then saving the font failed
with a bare UnicodeEncodeError from packing achVendID, with no line number.


Commit: 08374512b269a9b7797ae1185beb6568403eab49
https://github.com/fonttools/fonttools/commit/08374512b269a9b7797ae1185beb6568403eab49
Author: Cosimo Lupo <cos...@anthrotype.com>
Date: 2026-10-09 (Fri, 09 Oct 2026)

Changed paths:
M Lib/fontTools/feaLib/parser.py
M Tests/feaLib/parser_test.py

Log Message:
-----------
Merge pull request #4269 from RavSinghChandan/fealib-vendor-ascii

[feaLib] raise FeatureLibError for a non-ASCII OS/2 Vendor


Compare: https://github.com/fonttools/fonttools/compare/97dace246f11...08374512b269

To unsubscribe from these emails, change your notification settings at https://github.com/fonttools/fonttools/settings/notifications
Reply all
Reply to author
Forward
0 new messages