-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix incorrect warning when converting to CFF a CFF2 variable font with non-varying glyphs #476
Conversation
@readroberts you can take care of this branch now |
Will do. I need to add the fix into 'tx' on this branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test pass on my local system on he build servers.
@@ -986,6 +986,14 @@ static float var_applyDeltasForIndexPair(ctlSharedStmCallbacks *sscb, var_itemVa | |||
} | |||
|
|||
subtable = &dataList->ivdSubtables.array[pair->outerIndex]; | |||
|
|||
/* If specific glyphs do not have any variation, may will be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: may --> they
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@msousa Done - please check.
…e non-varying glyphs A glyph which is non-varying in a variable font may be referenced by a VariationStore data item subtable which has a region count of 0. The VF support code assumed that this was an error, and issued a false warning.
No description provided.