Skip to content

Conversation

behdad
Copy link
Member

@behdad behdad commented May 23, 2025

Fixes #3839

@behdad behdad requested a review from justvanrossum May 23, 2025 21:08
Copy link
Collaborator

@justvanrossum justvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that the fix works, thanks! I'm not familiar enough with the code for a proper review, though.

@behdad
Copy link
Member Author

behdad commented May 24, 2025

I can confirm that the fix works, thanks! I'm not familiar enough with the code for a proper review, though.

Basically, the items in the operands are either a number, or a list if they are a variable number. To add two operands we need to call _addArgs instead of using the + operator.

@behdad behdad merged commit b2cb06f into main May 24, 2025
11 checks passed
@behdad behdad deleted the specializer-rmoveto-blends branch May 24, 2025 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CFF specializer error with fontmake variable-cff2 output
2 participants