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

[opentype/tables] parse postscript names #161

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benoitkugler
Copy link
Contributor

I'm working on a font sub-setter to trim font files bundled in PDF. One heavy table is the 'post' one : it contains a (string) name for every glyph. The wire format is a byte slice where strings are Pascal encoded : [length, data, length, data, etc...]
This format does not allow direct access at a given index, and also makes it harder to remove the unused names.

We already process theses names in the font.NewFont constructor, and I would like to reuse the logic (currently hidden). To do so, this PR moves the parsing to the opentype/tables package, so that we can directly use the tables.PostNames20.Strings field.

There is no change for end users (only the consumers of the low level tables package will be impacted).

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.

None yet

1 participant