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

add --ttf-curves to support building glyf v1 with cubics #1021

Merged
merged 1 commit into from
Jul 31, 2023

Conversation

anthrotype
Copy link
Member

The new option allows to pass 'mixed' (to selectively convert cubics to quad only when more economical), 'keep-cubic' / 'keep-quad' (to skip conversion altogether and keep the desired input curve format).
The default value 'cu2qu' keeps the current behavior unchanged (i.e. all cubics converted to quad and glyf v0).

Requires ufo2ft >= 2.33.2

@anthrotype
Copy link
Member Author

I added a note to both --help and docstrings that support for glyf v1 is experimental and spec proposal (https://github.com/harfbuzz/boring-expansion-spec/blob/main/glyf1.md) not finalized, and as such not widely implemented.
It's also impossible to accidentally produce a glyf v1 table, because even if cu2qu is skipped with --keep-quad, ufo2ft will return an error if any of the glyphs do contain cubics but the head.glyphDataFormat = 0

@anthrotype anthrotype force-pushed the not-all-quadratic branch 3 times, most recently from 76846fe to 0102e07 Compare July 31, 2023 12:32
default behaviour is unchanged (all cubics converted to quad and glyf v0)
@anthrotype anthrotype merged commit 26642dd into main Jul 31, 2023
12 checks passed
@anthrotype anthrotype deleted the not-all-quadratic branch July 31, 2023 15:37
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.

1 participant