r358745 - [LibTooling] Fix -Wsign-compare after r358697

Bjorn Pettersson via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 19 02:10:42 PDT 2019


Author: bjope
Date: Fri Apr 19 02:10:42 2019
New Revision: 358745

URL: http://llvm.org/viewvc/llvm-project?rev=358745&view=rev
Log:
[LibTooling] Fix -Wsign-compare after r358697

Modified:
    cfe/trunk/unittests/Tooling/TransformerTest.cpp

Modified: cfe/trunk/unittests/Tooling/TransformerTest.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/Tooling/TransformerTest.cpp?rev=358745&r1=358744&r2=358745&view=diff
==============================================================================
--- cfe/trunk/unittests/Tooling/TransformerTest.cpp (original)
+++ cfe/trunk/unittests/Tooling/TransformerTest.cpp Fri Apr 19 02:10:42 2019
@@ -369,7 +369,7 @@ TEST_F(TransformerTest, OverlappingEdits
   // The rewrite process fails...
   EXPECT_TRUE(rewrite(Input));
   // ... but one AtomicChange was consumed:
-  ASSERT_EQ(Changes.size(), 1);
+  ASSERT_EQ(Changes.size(), 1u);
   EXPECT_TRUE(Changes[0].hasError());
 }
 
@@ -384,7 +384,7 @@ TEST_F(TransformerTest, OverlappingEdits
   // The rewrite process fails because the changes conflict with each other...
   EXPECT_FALSE(rewrite(Input));
   // ... but all changes are (individually) fine:
-  ASSERT_EQ(Changes.size(), 2);
+  ASSERT_EQ(Changes.size(), 2u);
   EXPECT_FALSE(Changes[0].hasError());
   EXPECT_FALSE(Changes[1].hasError());
 }




More information about the cfe-commits mailing list