[clang-tools-extra] [clang-tidy] Add check readability-return-expression-in-void-function (PR #76249)

[Piotr Zegar via cfe-commits]

Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>,
Danny =?utf-8?q?Mösch?= <danny.moesch at icloud.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/76249 at github.com>


PiotrZSL wrote:

I run check on llvm-project, finds thousands of issues. Most of them comes from google test but other issues are like this one:
```
if (x)
  return call();
```

Just some simplification to not write {}.
I think that both IgnoreMacros and StrictMode(false) could be needed. If StrictMode is false then only returnStmt that got compoundStmt as parent should be check, to reduce impact.

https://github.com/llvm/llvm-project/pull/76249