fix false positives for unused_parens around unary and binary operations

[lukas-code]

fix #110251

[rustbot]

r? @oli-obk

(rustbot has picked a reviewer for you, use r? to override)

[lukas-code]

wait why does this parse

fn what() {
    loop { if break {} }
}

but not

fn what() {
    loop { if return {} }
}

is this a bug or a feature?

[DaniPopes]

I believe that can be answered in rust-quiz #20 😄

[lukas-code]

Yeah, looks like this is a feature, and removing the parens can change behavior, so not linting to remove them is correct.

[lukas-code]

I pushed a second commit to deal with break more correctly.

[oli-obk]

@bors r+ rollup

[bors]

📌 Commit 0d0949d has been approved by oli-obk

It is now in the queue for this repository.