diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-09-09 17:06:09 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-09-09 17:06:09 +0000 |
| commit | ffc277989be5531ebf3f2a6b023f54d7929067b9 (patch) | |
| tree | 1f3f18678530c7d1f431328368eb2d62befab6ed /clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp | |
| parent | ef67d768691f7dd2c69f455f5d4479608eba792f (diff) | |
| download | bcm5719-llvm-ffc277989be5531ebf3f2a6b023f54d7929067b9.tar.gz bcm5719-llvm-ffc277989be5531ebf3f2a6b023f54d7929067b9.zip | |
[clang-tidy] Fix PR22785.
Fix http://llvm.org/PR22785. Bug 22785 - readability-braces-around-statements
doesn't work well with macros.
http://reviews.llvm.org/D12729
Patch by Marek Kurdej!
llvm-svn: 247163
Diffstat (limited to 'clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp')
| -rw-r--r-- | clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp b/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp index 9b44ac55917..446775bdce6 100644 --- a/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp +++ b/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp @@ -457,6 +457,27 @@ TEST(BracesAroundStatementsCheck, Macros) { "int main() {\n" " FOR(;;)\n" "}"); + EXPECT_EQ("#define DO_IT ++i\n" + "int i = 0;\n" + "int main() {\n" + " if (false) {\n" + " DO_IT;\n" + " } else if (1 == 2) {\n" + " DO_IT;\n" + " } else {\n" + " DO_IT;\n" + "}\n" + "}", + runCheckOnCode<BracesAroundStatementsCheck>("#define DO_IT ++i\n" + "int i = 0;\n" + "int main() {\n" + " if (false)\n" + " DO_IT;\n" + " else if (1 == 2)\n" + " DO_IT;\n" + " else\n" + " DO_IT;\n" + "}")); } } // namespace test |
