-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add Ada to Node.js #142
Comments
Current benchmark results:
|
This comment was marked as outdated.
This comment was marked as outdated.
Currently, unless you changed something, ada will not use ICU under Windows. |
Yes, I'm running the tests on macOS for now. When all tests pass, I'll move over to Windows. |
@anonrig Do you have any figures on how Ada does against the existing Node.js WHATWG URL parser? |
@TimothyGu This Comment is the initial benchmark result. With the current changes we did on |
Ah, I misread. Thank you! Great results indeed. |
Current work in progress: nodejs/node#46410
The text was updated successfully, but these errors were encountered: