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.

Sign up for GitHub

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

Jump to bottom

feat: use smartstring for handlebars path segments #567

Open

sunng87 wants to merge 2 commits into master from feature/smartstring-2

Owner

sunng87 commented Mar 14, 2023

We see 10% performance improvement in our nested-loop benchmark after switching to LazyCompactString

sunng87 added 2 commits

March 14, 2023 20:56


          feat: use smartstring for internal path string

9c0cd68


          feat: switch to lazy-compact string for better performance

d74f966

coveralls commented Mar 14, 2023

Coverage: 80.047% (-0.2%) from 80.2% when pulling d74f966 on feature/smartstring-2 into d3caa6d on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment