forked from OSchip/llvm-project
eb6d821cda
Summary: A followup for D32942. Malcolm Parsons has provided a valid testcase that the initial version of the check complained about nested `if`'s. As it turns out, the culprit is the **partially** un-intentional `switch` fallthrough. So rewrite the NestingThreshold logic without ab-using+mis-using that switch with fallthrough, and add testcases with nested `if`' where there should be a warning and shouldn't be a warning. This results in a cleaner, simpler code, too. I guess, now it would be actually possible to pick some reasonable default for `NestingThreshold` setting. Fixes PR33454. Reviewers: malcolm.parsons, alexfh Reviewed By: malcolm.parsons Subscribers: sbenza, xazax.hun, cfe-commits, aaron.ballman Tags: #clang-tools-extra Differential Revision: https://reviews.llvm.org/D34202 llvm-svn: 305554 |
||
|---|---|---|
| .. | ||
| Unit | ||
| change-namespace | ||
| clang-apply-replacements | ||
| clang-move | ||
| clang-query | ||
| clang-rename | ||
| clang-reorder-fields | ||
| clang-tidy | ||
| clangd | ||
| include-fixer | ||
| modularize | ||
| pp-trace | ||
| .clang-format | ||
| CMakeLists.txt | ||
| lit.cfg | ||
| lit.site.cfg.in | ||