Skip to content

[klippa-repacker] split ligature subst table#1710

Merged
qxliu76 merged 1 commit intomainfrom
split_ligature
Jan 7, 2026
Merged

[klippa-repacker] split ligature subst table#1710
qxliu76 merged 1 commit intomainfrom
split_ligature

Conversation

@qxliu76
Copy link
Copy Markdown
Contributor

@qxliu76 qxliu76 commented Jan 5, 2026

plus change RepackerErrorFlags from bitflags to enum

Copy link
Copy Markdown
Member

@cmyr cmyr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay some notes inline but this seems reasonable.

Is there any reason for including the error change in with this PR? it seems like those should be pretty independent, and having them be separate would have made this a bit easier to review. :)

Comment thread klippa/src/graph/coverage_graph.rs
Comment thread klippa/src/graph.rs Outdated
Comment thread klippa/src/graph.rs Outdated
Comment thread klippa/src/graph/layout.rs Outdated
Comment thread klippa/src/graph/layout.rs Outdated
Comment thread klippa/src/graph.rs Outdated
Comment thread klippa/src/graph.rs Outdated
Comment thread klippa/src/repack.rs Outdated
Comment thread klippa/src/graph/ligature_graph.rs
Comment thread klippa/src/graph/ligature_graph.rs
@qxliu76 qxliu76 force-pushed the split_ligature branch 2 times, most recently from bc8d530 to 430300d Compare January 7, 2026 17:54
plus change RepackerErrorFlags from bitflags to enum
@qxliu76 qxliu76 merged commit d89060e into main Jan 7, 2026
13 checks passed
@qxliu76 qxliu76 deleted the split_ligature branch January 7, 2026 18:15
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