add a pair of whitespace after remove parentheses

[bravomikekilo]

fix issue-63156.
add a pair of whitespace after remove parentheses.

[rust-highfive]

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @cramertj (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

[cramertj]

I don't think we should add the spaces unconditionally, since that would cause there to be double-spaces in the code that most folks probably tried to write: if (x < 0) { ... } -> if x < 0 { ... }

[Centril]

Would be good to add a regression test also.

[bravomikekilo]

OK. I will check the position of if while and match and add spaces conditionally.

[bravomikekilo]

I add some conditional spaces. But there is a problem, I can't find the Span of in keyword in ForLoop, so code like for i in(0..3) {..} will become for i in0..3 {}. Can anyone give me some suggestion or just keep the old behavior?

[bravomikekilo]

Tidy problem is fixed. ForLoop now is keeping the old behavior. new tests are added.

[bravomikekilo]

Can this pull request be merged or there are other problems to be fixed? @cramertj

[cramertj]

It looks good to me, although it'd be great to add a rustfix test, to make sure that our suggestions are being applied correctly. r=me with that change.

[bravomikekilo]

r? @cramertj
rustfix test added

[bravomikekilo]

r= @cramertj

[cramertj]

@bors r+

[bors]

📌 Commit 3a95c71 has been approved by cramertj