-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Fixing #18659 #18713
Fixing #18659 #18713
Conversation
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon. |
Awesome, thanks @juxiliary! |
Could you rebase this on master as well to get rid of the merge commit? |
This change is the only thing blocking servo's rust upgrade right now. Please p=1 when you r+. |
Rabble rabble beat me to it 😜. |
@alexcrichton rebased it |
* `from_str_radix_float` gives incorrect results for negative float strings. Changes the accumulator used to start at -0.0 instead of -1.0. * Adds missing tests
Thanks! Really sorry for the trouble I caused :( |
internal: Cleanup label structure of `CompletionItem`
from_str_radix_float
gives incorrect results for negative float strings. Changes the accumulator used to start at -0.0 instead of -1.0.