detect unnecessary curly braces

[mimoo]

What it does

In the following example, there's a single expression inside curly braces. I think clippy should detect that because it is probably not intended. (We have some of these in our codebase due to refactor.)

Lint Name

unecessary_curly_braces

Category

style

Advantage

• Remove lines of code
• Removes indentation
• (Improves readability)

Drawbacks

No response

Example

let a = {
  if cond {
    None
  } else {
    Some(5)
  }
};

Better:

let a = if cond {
  None
} else {
  Some(5)
};

[kraktus]

Hey, thanks for reporting. There’s already a lint at the rustc level: unused_braces. Unfortunately it only triggers on one-line braces, but there’s a PR to expand it, although it’s been stalled on review: rust-lang/rust#102432