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 variable font support to ttread #957

Merged
merged 4 commits into from
Sep 7, 2019
Merged

Add variable font support to ttread #957

merged 4 commits into from
Sep 7, 2019

Conversation

blueshade7
Copy link
Contributor

Added an optional UDV parameter to ttrBegFont() to specify a VF instance.
Use varread library to calculate glyph widths from HVAR table.
Added a bunch of code to calculate glyph coordinates' variations using the gvar table.
Added AdobeVFPrototype.ttf as a test font along with a tx test case to instantiate & convert to tx -3 output.

c/public/lib/api/ttrerr.h Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
c/public/lib/source/ttread/ttread.c Outdated Show resolved Hide resolved
@cjchapman
Copy link
Contributor

The The scope of the variable ... can be reduced. messages from Codacy don't particularly concern me, but it's easy enough to clean them up, so you might as well do that while you're in there.

@cjchapman
Copy link
Contributor

👍

@miguelsousa miguelsousa merged commit 767aece into develop Sep 7, 2019
@miguelsousa miguelsousa deleted the ttread-var-stg branch September 7, 2019 04:39
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.

3 participants