[Remarks] Fix warning for ambigous `else` behind EXPECT macro
authorFrancis Visoiu Mistrih <francisvm@yahoo.com>
Wed, 16 Oct 2019 16:43:34 +0000 (16:43 +0000)
committerFrancis Visoiu Mistrih <francisvm@yahoo.com>
Wed, 16 Oct 2019 16:43:34 +0000 (16:43 +0000)
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

unittests/Remarks/YAMLRemarksParsingTest.cpp

index 5ab7b21367d6a7ee6ef26dbdc8b4abbc667d748d..f20cdb5e6bbb4a6e39b36e6ee676c0152dfbf049 100644 (file)
@@ -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
   // for different OSs.
-  if (Cmp == CmpType::Equal)
+  if (Cmp == CmpType::Equal) {
     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)));
+  }
 }
 
 TEST(YAMLRemarks, ParsingEmpty) {