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

Update to HarfBuzz 4.0.0 #110

Merged
merged 2 commits into from
Mar 11, 2022
Merged

Update to HarfBuzz 4.0.0 #110

merged 2 commits into from
Mar 11, 2022

Conversation

khaledhosny
Copy link
Collaborator

@khaledhosny khaledhosny commented Feb 15, 2022

In preparation for the stabilization hb-draw API.

TODO:

  • Review the Python API and align it with the new HarfBuzz API
  • Deprecate old Python API if needed

@khaledhosny khaledhosny force-pushed the hb-draw-update branch 2 times, most recently from e81bf8b to f9bb97e Compare February 16, 2022 06:30
@justvanrossum
Copy link
Collaborator

Blackrenderer uses the draw API, and may be a good test case. Let me know if/when you want me to try out this PR.

The user_data object should be passed to the callback setter. The old
Python way of keeping user_data in the DrawFuncs object still works but
is deprecated.
@khaledhosny khaledhosny marked this pull request as ready for review March 2, 2022 19:33
@khaledhosny
Copy link
Collaborator Author

Blackrenderer uses the draw API, and may be a good test case. Let me know if/when you want me to try out this PR.

It should be ready for testing now.

@khaledhosny khaledhosny changed the title Update to final HarfBuzz draw API Update to HarfBuzz 4.0.0 Mar 2, 2022
@garretrieger
Copy link
Contributor

Is this able to be merged now? We're planning on PR'ing the addition of the repacker API but it's blocked on updating to 4.0.0.

@khaledhosny khaledhosny merged commit 35512b9 into main Mar 11, 2022
@khaledhosny khaledhosny deleted the hb-draw-update branch March 11, 2022 02:33
@khaledhosny
Copy link
Collaborator Author

Merged, we can fix any issues before next release.

@justvanrossum
Copy link
Collaborator

I just ran my blackrenderer tests with this version, and it works perfectly.

I do see differences between the previous int-only output and the current float coordinates, but that's obviously to be expected. I'll just have to adapt my expected results in those few cases.

@khaledhosny
Copy link
Collaborator Author

Excellent !

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

3 participants