summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp
diff options
context:
space:
mode:
authorSamuel Benzaquen <sbenza@google.com>2015-06-04 16:36:58 +0000
committerSamuel Benzaquen <sbenza@google.com>2015-06-04 16:36:58 +0000
commitcfacf8ae9ec6724a291600b848fbbb94bb5435e5 (patch)
treed6d2f43b3adf09ba141ea98d088d1a64b889eee1 /clang-tools-extra/unittests/clang-tidy/ReadabilityModuleTest.cpp
parent73e06fa262a1c05a09e53e6d5394b73ea875c285 (diff)
downloadbcm5719-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.cpp13
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) {
OpenPOWER on IntegriCloud