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

update servo-url and fix detection #789

Merged
merged 1 commit into from
Dec 1, 2024
Merged

update servo-url and fix detection #789

merged 1 commit into from
Dec 1, 2024

Conversation

anonrig
Copy link
Member

@anonrig anonrig commented Dec 1, 2024

We are still the fastest URL parser on both RFC-3986 and WHATWG.

--------------------------------------------------------------------------------------------
Benchmark                                  Time             CPU   Iterations UserCounters...
--------------------------------------------------------------------------------------------
BasicBench_AdaURL_href                  3411 ns         3404 ns       204433 speed=253.506M/s time/byte=3.94468ns time/url=378.251ns url/s=2.64375M/s
BasicBench_AdaURL_aggregator_href       2051 ns         2046 ns       338881 speed=421.819M/s time/byte=2.37068ns time/url=227.322ns url/s=4.39904M/s
BasicBench_AdaURL_CanParse              1431 ns         1425 ns       486000 speed=605.437M/s time/byte=1.6517ns time/url=158.38ns url/s=6.31394M/s
BasicBench_whatwg                       6840 ns         6789 ns       103240 speed=127.114M/s time/byte=7.86693ns time/url=754.351ns url/s=1.32564M/s
BasicBench_CURL                        17156 ns        17093 ns        41000 speed=50.4882M/s time/byte=19.8066ns time/url=1.89923us url/s=526.528k/s
BasicBench_uriparser_just_parse         4627 ns         4595 ns       151417 speed=187.82M/s time/byte=5.32424ns time/url=510.536ns url/s=1.95873M/s
BasicBench_http_parser_just_parse       4273 ns         4262 ns       164956 speed=202.481M/s time/byte=4.93873ns time/url=473.569ns url/s=2.11162M/s
BasicBench_ServoUrl                     9441 ns         9401 ns        75232 speed=91.7962M/s time/byte=10.8937ns time/url=1.04459us url/s=957.318k/s

@anonrig anonrig requested a review from lemire December 1, 2024 00:05
Copy link
Member

@lemire lemire left a comment

Choose a reason for hiding this comment

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

!!!

@anonrig anonrig merged commit 4096858 into main Dec 1, 2024
28 checks passed
@anonrig anonrig deleted the yagiz/update-rust-impl branch December 1, 2024 01:47
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.

2 participants