Skip to content
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 Glyphs 2 sources with implicit axes always getting indirect HVAR #1263

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

anthrotype
Copy link
Member

Fixes #1256

The first commit reproduces the bug, followed by actual fix

@anthrotype anthrotype force-pushed the glyphs2-implicit-axes-direct-hvar branch from ab1c3bf to b3a20e9 Compare February 13, 2025 13:27
@anthrotype anthrotype force-pushed the glyphs2-implicit-axes-direct-hvar branch from b3a20e9 to a0d4725 Compare February 13, 2025 13:28
@anthrotype
Copy link
Member Author

anthrotype commented Feb 13, 2025

ci failed as expected, the HVAR got an advance width mapping
a0d4725

assertion failed: hvar.advance_width_mapping().is_none()

Update: after 7716fb6 now it passes, and HVAR has no mapping.

Copy link
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

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

looks good, one Q inline :)

"-",
"!"
);
.appVersion = "3340";
Copy link
Member

Choose a reason for hiding this comment

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

doesn't this mean glyphs3?

Copy link
Member Author

Choose a reason for hiding this comment

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

no, that's simply saying this particular glyphs2 source file was serialized by Glyphs 3 (3340). you can 'save as'. I don't have a Glyphs 2.app any more.
The .formatVersion key stores the source format version, it's omitted in Glyphs 2 (defaults to 2 in fact).

@anthrotype anthrotype added this pull request to the merge queue Feb 13, 2025
Merged via the queue into main with commit 39a7f6c Feb 13, 2025
11 checks passed
@anthrotype anthrotype deleted the glyphs2-implicit-axes-direct-hvar branch February 13, 2025 15:48
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.

Indirect HVAR is sometimes built where one without AdvWidthMap would be more efficient
2 participants