-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Specify the range of legal radix string num.toRadixString #461
Labels
area-core-library
SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
Comments
EcmaScript goes up to 36. Any reason we shouldn't do the same? |
This comment was originally written by domi...@google.com Whatever you do here, please reflect it in the String.toInt method. |
Fixed to go up to 36 on both implementation. Handling of out of bounds radices has been unified as well. Added Fixed label. |
ghost
added
Type-Defect
area-core-library
SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
labels
Sep 19, 2012
This was referenced Oct 29, 2020
This issue was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area-core-library
SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries.
Currently implemented toRadixString for radix 1..16. Specify the range.
LibTest/core/int/toRadixString/int/toRadixString/A01/t01
The text was updated successfully, but these errors were encountered: