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 bindings to hb_ft_font_create_referenced. #69

Merged
merged 1 commit into from May 7, 2017
Merged

Conversation

@glennw
Copy link
Member

glennw commented May 7, 2017

This allows creating a harfbuzz font directly from a FreeType
face. Internally it increments the ref count of the FT face
so that there are no lifetime issues.

This allows much simpler use of the harfbuzz API rather than
having to implement all the custom callback types, when
using FT.

It's exposed on all non-mac UNIX platforms (Linux, BSD etc)
that are not Mac, and also on Android.


This change is Reviewable

This allows creating a harfbuzz font directly from a FreeType
face. Internally it increments the ref count of the FT face
so that there are no lifetime issues.

This allows much simpler use of the harfbuzz API rather than
having to implement all the custom callback types, when
using FT.

It's exposed on all non-mac UNIX platforms (Linux, BSD etc)
that are not Mac, and also on Android.
@glennw
Copy link
Member Author

glennw commented May 7, 2017

@emilio
Copy link
Member

emilio commented May 7, 2017

@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2017

📌 Commit dd3434d has been approved by emilio

@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2017

Testing commit dd3434d with merge 1f7f339...

bors-servo added a commit that referenced this pull request May 7, 2017
Add bindings to hb_ft_font_create_referenced.

This allows creating a harfbuzz font directly from a FreeType
face. Internally it increments the ref count of the FT face
so that there are no lifetime issues.

This allows much simpler use of the harfbuzz API rather than
having to implement all the custom callback types, when
using FT.

It's exposed on all non-mac UNIX platforms (Linux, BSD etc)
that are not Mac, and also on Android.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-harfbuzz/69)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 7, 2017

☀️ Test successful - status-travis
Approved by: emilio
Pushing 1f7f339 to master...

@bors-servo bors-servo merged commit dd3434d into servo:master May 7, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
MortimerGoro added a commit to MortimerGoro/rust-harfbuzz that referenced this pull request May 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.