]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Rollup merge of #97498 - ijchen:master, r=Mark-Simulacrum
authorYuki Okushi <jtitor@2k36.org>
Wed, 1 Jun 2022 14:36:49 +0000 (23:36 +0900)
committerGitHub <noreply@github.com>
Wed, 1 Jun 2022 14:36:49 +0000 (23:36 +0900)
commite1d2e65463e9bc5b75f92209a9e22a6ba7b7aa24
tree04f4f0d5bb62a2ff05cd1e3af92ffc6ae1bc69f8
parenta1dc541c69bcd657eb42406a367663fed2a3b0e2
parent0484cfb6a91d1236809cfe4c536aa29b498e518d
Rollup merge of #97498 - ijchen:master, r=Mark-Simulacrum

Corrected EBNF grammar for from_str

Hello! This is my first time contributing to an open-source project. I'm excited to have the chance to contribute to the rust community ðŸ¥³

I noticed an issue with the documentation for `from_str` in `f32` and `f64`. It states that "All strings that adhere to the following [EBNF](https://www.w3.org/TR/REC-xml/#sec-notation) grammar when lowercased will result in an `Ok` being returned. I believe this is incorrect for the string `"."`, which is valid for the given EBNF grammar, but does not result in an `Ok` being returned ([playground](https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=09f891aa87963a56d3b0d715d8cbc2b4)). I have simplified the grammar in a way which fixes that, but is otherwise identical.

Previously, the `Number` part of the EBNF grammar had an option for `'.' Digit*`, which would include the string `"."`. This is not valid, and does not return an Ok as stated. The corrected version removes this, and still allows for the `'.' Digit+` case with the already existing `Digit* '.' Digit+` case.