From 0947af7ac539368b1bee8c21052b843753f2f190 Mon Sep 17 00:00:00 2001 From: Francis Visoiu Mistrih Date: Wed, 16 Oct 2019 16:43:34 +0000 Subject: [PATCH] [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 llvm-svn: 375015 --- llvm/unittests/Remarks/YAMLRemarksParsingTest.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/llvm/unittests/Remarks/YAMLRemarksParsingTest.cpp b/llvm/unittests/Remarks/YAMLRemarksParsingTest.cpp index 5ab7b21..f20cdb5 100644 --- a/llvm/unittests/Remarks/YAMLRemarksParsingTest.cpp +++ b/llvm/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 // 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) { -- 2.7.4