We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
@bnoordhuis posted some excellent ideas here:
For example, .toASCII() calls String#split() twice and is heavy on the map-over-substrings idiom. It would probably be faster to: Scan for the split character and only split when found; saves allocating an array in the common case. Iterate rather than map; V8 is not great at lowering/inlining callbacks. Try to avoid concatenating strings too much; cons strings eventually have to be flattened. There is probably more but that is what stood out from a quick look.
For example, .toASCII() calls String#split() twice and is heavy on the map-over-substrings idiom. It would probably be faster to:
.toASCII()
String#split()
There is probably more but that is what stood out from a quick look.
Patches welcome.
cc @trevnorris
The text was updated successfully, but these errors were encountered:
No branches or pull requests
@bnoordhuis posted some excellent ideas here:
Patches welcome.
cc @trevnorris
The text was updated successfully, but these errors were encountered: