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

Finish syncing to 4.2.1 #101

Merged
merged 19 commits into from
Apr 13, 2024
Merged

Finish syncing to 4.2.1 #101

merged 19 commits into from
Apr 13, 2024

Conversation

Pi-Cla
Copy link

@Pi-Cla Pi-Cla commented Apr 13, 2024

Big thank you to @LaurenzV for starting this round of backporting!
This continues the effort started at #100

(Idk how to make or copy the fancy table you used so pretend it is here...)

LaurenzV and others added 19 commits February 27, 2024 22:17
This allows mix-and-matching per-syllable and other lookups.
In fact, removes the clear-syllables call completely.
harfbuzz/harfbuzz@044d7a06d
Test for the fix in 7bdc20e for the
regression from 43be5ba.
harfbuzz/harfbuzz@383c11d75
@Pi-Cla
Copy link
Author

Pi-Cla commented Apr 13, 2024

There was a typo at 767ff47 which was what stopped the tests from working earlier. So I git amended the fix onto that commit

@LaurenzV
Copy link
Collaborator

Oh cool! Thanks for doing this, I'll close my old PR then, not sure when I will have time to work on it again anyway...

I used this Google Sheet document to keep track of it, and I basically copy the relevant commits into a Markdown converter.

@RazrFalcon
Copy link
Collaborator

Thanks! Much appreciated!

Don't worry, the changelog table isn't mandatory 😉

@RazrFalcon RazrFalcon merged commit 6781f6f into harfbuzz:master Apr 13, 2024
1 check passed
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