-
Notifications
You must be signed in to change notification settings - Fork 167
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
Tx fixes #397
Conversation
… variable in tx.varread.varread.c This update makes no change in function: it simply renames variables and fields to avoid confusion by developers.
@readroberts Please add an integration test that can validate this change. |
I will create a small CFF2 font and test snapshotting it. |
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.
Integration test needed
Noted. Will complete this on Tues. |
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.
looks fine to me
* [tx][varread] Fix issue adobe-type-tools#272: Fix inconsistency of use of regionCount variable in tx.varread.varread.c This update makes no change in function: it simply renames variables and fields to avoid confusion by developers. * [tx][tx_test.py] Add test for reading and snapshotting simple CFF2 VF font to test fix for issue adobe-type-tools#272
[tx] Fix for issue #272. Fix inconsistency of use of regionCount variable in tx.varread.varread.c