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

use a binary search for pathfinder_renderer::SortedVector #245

Merged
merged 1 commit into from Dec 29, 2019

Conversation

@s3bk
Copy link
Contributor

s3bk commented Dec 28, 2019

During profiling of the PDF viewer, SortedVector::push showed up at 44% when displaying the Inkscape about screen (converted to PDF).
Switching to slice::binary_search_by made it disappear.

@pcwalton pcwalton merged commit 7f6374f into servo:master Dec 29, 2019
2 checks passed
2 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@s3bk s3bk deleted the s3bk:sorted_vector_binary_search branch Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.