[fonttools/fonttools] 97929b: [feaLib] try reuse existing inline chained multipl...

[Cosimo Lupo]

Branch: refs/heads/reuse-chain-multi-subst
Home: https://github.com/fonttools/fonttools
Commit: 97929b32369514642280be56c75d4e3c563efe09
https://github.com/fonttools/fonttools/commit/97929b32369514642280be56c75d4e3c563efe09
Author: Cosimo Lupo <cl...@google.com>
Date: 2024-06-03 (Mon, 03 Jun 2024)

Changed paths:
M Lib/fontTools/feaLib/builder.py
M Lib/fontTools/otlLib/builder.py
M Tests/feaLib/builder_test.py
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.fea
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.ttx

Log Message:
-----------
[feaLib] try reuse existing inline chained multiple subst lookups when possible

We already do this for inline single substitutions in chained contextual lookups, this PR extends this for multiple substitutions as well.

Fixes https://github.com/fonttools/fonttools/issues/3551



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: 97929b32369514642280be56c75d4e3c563efe09
https://github.com/fonttools/fonttools/commit/97929b32369514642280be56c75d4e3c563efe09
Author: Cosimo Lupo <cl...@google.com>
Date: 2024-06-03 (Mon, 03 Jun 2024)

Changed paths:
M Lib/fontTools/feaLib/builder.py
M Lib/fontTools/otlLib/builder.py
M Tests/feaLib/builder_test.py
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.fea
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.ttx

Log Message:
-----------
[feaLib] try reuse existing inline chained multiple subst lookups when possible

We already do this for inline single substitutions in chained contextual lookups, this PR extends this for multiple substitutions as well.

Fixes https://github.com/fonttools/fonttools/issues/3551

Commit: 4b030d2db80335ae981204578a1b0c8a2151510f
https://github.com/fonttools/fonttools/commit/4b030d2db80335ae981204578a1b0c8a2151510f
Author: Cosimo Lupo <cos...@anthrotype.com>
Date: 2024-06-11 (Tue, 11 Jun 2024)

Changed paths:
M Lib/fontTools/feaLib/builder.py
M Lib/fontTools/otlLib/builder.py
M Tests/feaLib/builder_test.py
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.fea
A Tests/feaLib/data/contextual_inline_multi_sub_format_2.ttx

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

Merge pull request #3559 from fonttools/reuse-chain-multi-subst

[feaLib] try reuse existing inline chained multiple subst lookups when possible

Compare: https://github.com/fonttools/fonttools/compare/5e6b12d12fa0...4b030d2db803