[fonttools/fonttools] 7e73fc: [feaLib] write 0xFFFF instead of '0' for missing n...

0 views
Skip to first unread message

Cosimo Lupo

unread,
Dec 18, 2025, 6:24:50 AM (2 days ago) Dec 18
to fontto...@googlegroups.com
Branch: refs/heads/main
Home: https://github.com/fonttools/fonttools
Commit: 7e73fc3c6dc9b04e9f6807cdd39e464f9ddf63c7
https://github.com/fonttools/fonttools/commit/7e73fc3c6dc9b04e9f6807cdd39e464f9ddf63c7
Author: Colin Rofls <co...@cmyr.net>
Date: 2025-12-16 (Tue, 16 Dec 2025)

Changed paths:
M Lib/fontTools/feaLib/builder.py
M Lib/fontTools/ttLib/tables/otConverters.py
M Tests/feaLib/builder_test.py
A Tests/feaLib/data/cvparam_null.fea
A Tests/feaLib/data/cvparam_null.ttx

Log Message:
-----------
[feaLib] write 0xFFFF instead of '0' for missing nameIDs

Specifically in FeatureParams for the cv features; the spec is poorly
worded but a 0 value here may be misinterpreted by clients.


Commit: 764681404e8a511b48af42320eadf5b4ac344179
https://github.com/fonttools/fonttools/commit/764681404e8a511b48af42320eadf5b4ac344179
Author: Colin Rofls <co...@cmyr.net>
Date: 2025-12-17 (Wed, 17 Dec 2025)

Changed paths:
M Tests/feaLib/data/cvparam_null.fea

Log Message:
-----------
Use 0xFFFF instead of 0 in comment

Co-authored-by: Cosimo Lupo <cos...@anthrotype.com>


Commit: ffeaa5180c638630e1b47309aaa817fb5c26f4f1
https://github.com/fonttools/fonttools/commit/ffeaa5180c638630e1b47309aaa817fb5c26f4f1
Author: Cosimo Lupo <cos...@anthrotype.com>
Date: 2025-12-18 (Thu, 18 Dec 2025)

Changed paths:
M Lib/fontTools/feaLib/builder.py
M Lib/fontTools/ttLib/tables/otConverters.py
M Tests/feaLib/builder_test.py
A Tests/feaLib/data/cvparam_null.fea
A Tests/feaLib/data/cvparam_null.ttx

Log Message:
-----------
Merge pull request #4012 from cmyr/cv-feature-null

[feaLib] write 0xFFFF instead of '0' for missing nameIDs


Compare: https://github.com/fonttools/fonttools/compare/ff07f6be86c1...ffeaa5180c63

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