diff options
author | Samuel Benzaquen <sbenza@google.com> | 2015-06-04 16:36:58 +0000 |
---|---|---|
committer | Samuel Benzaquen <sbenza@google.com> | 2015-06-04 16:36:58 +0000 |
commit | cfacf8ae9ec6724a291600b848fbbb94bb5435e5 (patch) | |
tree | d6d2f43b3adf09ba141ea98d088d1a64b889eee1 /clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp | |
parent | 73e06fa262a1c05a09e53e6d5394b73ea875c285 (diff) | |
download | bcm5719-llvm-cfacf8ae9ec6724a291600b848fbbb94bb5435e5.tar.gz bcm5719-llvm-cfacf8ae9ec6724a291600b848fbbb94bb5435e5.zip |
[clang-tidy] Force braces around leaf 'else if' for consistency.
Summary:
Force braces around leaf 'else if' for consistency.
This complements r233697.
Reviewers: alexfh
Subscribers: curdeius, cfe-commits
Differential Revision: http://reviews.llvm.org/D10245
llvm-svn: 239054
Diffstat (limited to 'clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp')
-rw-r--r-- | clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp b/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp index 1da7206c276..9b44ac55917 100644 --- a/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp +++ b/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp @@ -254,6 +254,19 @@ TEST(BracesAroundStatementsCheck, IfElseWithShortStatements) { " else return -3;\n" "}", nullptr, "input.cc", None, Options)); + + // If the last else is an else-if, we also force it. + EXPECT_EQ("int main() {\n" + " if (false) { return -1;\n" + " } else if (1 == 2) { return -2;\n" + "}\n" + "}", + runCheckOnCode<BracesAroundStatementsCheck>( + "int main() {\n" + " if (false) return -1;\n" + " else if (1 == 2) return -2;\n" + "}", + nullptr, "input.cc", None, Options)); } TEST(BracesAroundStatementsCheck, For) { |