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

AutoLayout → LayoutVisitor #434

Merged
merged 13 commits into from
Jan 16, 2024
Merged

AutoLayout → LayoutVisitor #434

merged 13 commits into from
Jan 16, 2024

Conversation

dhardy
Copy link
Collaborator

@dhardy dhardy commented Jan 15, 2024

Replace trait AutoLayout with LayoutVisitor using RPITIT to move the temporary layout visitor into a method.

Code should be more inlinable for better optimisation and has reduced size in optimised builds, but debug builds are larger. Size of gallery (bytes):

Commit Debug Stripped Release Stripped
16bd205 248853056 37325408 23188160 14148312
c851799 248793400 37321136 23172256 14143248
c792102 248971880 37350112 23158136 14135344
f40e17d 248971224 37350112 23158088 14135344
1706ddc 249231608 37392720 23117136 14085992

Also fix docs.rs builds for kas-dylib, kas-resvg. See rust-lang/docs.rs#2359.

@dhardy dhardy merged commit 2f6b417 into master Jan 16, 2024
5 checks 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.

1 participant