Ver código fonte

[Remarks] Fix warning for ambigous `else` behind EXPECT macro

http://lab.llvm.org:8011/builders/clang-ppc64be-linux-lnt/builds/31902/steps/ninja%20check%201/logs/stdio

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@375015 91177308-0d34-0410-b5e6-96231b3b80d8
Francis Visoiu Mistrih 5 anos atrás
pai
commit
a55be6b48f
1 arquivos alterados com 4 adições e 2 exclusões
  1. 4 2
      unittests/Remarks/YAMLRemarksParsingTest.cpp

+ 4 - 2
unittests/Remarks/YAMLRemarksParsingTest.cpp

@@ -82,11 +82,13 @@ void parseExpectErrorMeta(StringRef Buf, const char *Error, CmpType Cmp,
 
 
   // Use a case insensitive comparision due to case differences in error strings
   // Use a case insensitive comparision due to case differences in error strings
   // for different OSs.
   // for different OSs.
-  if (Cmp == CmpType::Equal)
+  if (Cmp == CmpType::Equal) {
     EXPECT_EQ(StringRef(Stream.str()).lower(), StringRef(Error).lower());
     EXPECT_EQ(StringRef(Stream.str()).lower(), StringRef(Error).lower());
+  }
 
 
-  if (Cmp == CmpType::Contains)
+  if (Cmp == CmpType::Contains) {
     EXPECT_TRUE(StringRef(Stream.str()).contains(StringRef(Error)));
     EXPECT_TRUE(StringRef(Stream.str()).contains(StringRef(Error)));
+  }
 }
 }
 
 
 TEST(YAMLRemarks, ParsingEmpty) {
 TEST(YAMLRemarks, ParsingEmpty) {