Skip to content

Fix #1015 - #1030

Merged
marijnh merged 4 commits into
rust-lang:masterfrom
Yoric:newbie
Oct 12, 2011
Merged

Fix #1015#1030
marijnh merged 4 commits into
rust-lang:masterfrom
Yoric:newbie

Conversation

@Yoric

Copy link
Copy Markdown
Contributor

This should fix issue #1015. I have reimplemented str_to_float to extend the grammar that it can parse. Including, say, strings with a decimal part :)

I take the opportunity to replace the implementation of int::pow to use fast exponentiation, to add a few functions to the new float module, as well as a few tests. I also renamed str_to_float and float_to_str into float::from_str and float::to_str, to harmonize with the int module.

@marijnh
marijnh merged commit 75bda42 into rust-lang:masterOct 12, 2011
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
Kobzol pushed a commit to Kobzol/rust that referenced this pull request Dec 30, 2024
The section was removed in rust-lang#1030 and re-added in a different place in
2d42cf7.
bors pushed a commit to rust-lang-ci/rust that referenced this pull request Jan 2, 2025
The section was removed in rust-lang#1030 and re-added in a different place in
2d42cf7.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
1032: Fix apply_substs assertion failure r=matklad a=flodiebold
Fixesrust-lang#1030.
Co-authored-by: Florian Diebold <flodiebold@gmail.com>
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
1145: Fixrust-lang#1099, plug self type type parameters in infer_path_expr r=flodiebold a=edwin0cheng
As discussed in rust-lang#1099, this PR try to "plug" a `self type` type parameters in `infer_path_expr`.
All the cases in 1099 was fixed and tested.
And luckily, this PR fixed bug rust-lang#1030 again and make the test output correct.
Co-authored-by: Edwin Cheng <edwin0cheng@gmail.com>
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
update comment re: not using higher mir-opt-level
Sign up for freeto 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

@Yoric@marijnh