<div dir="ltr">FWIW, a common way to address this issue is to use the 'unsigned' suffix, as in:<br>EXPECT_EQ(x, 5u);</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 23, 2021 at 4:24 PM Nemanja Ivanovic via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Author: Nemanja Ivanovic<br>
Date: 2021-09-23T18:24:39-05:00<br>
New Revision: 76d845cb169f048cb6f2176c3e7a6534dc5af097<br>
<br>
URL: <a href="https://github.com/llvm/llvm-project/commit/76d845cb169f048cb6f2176c3e7a6534dc5af097" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/76d845cb169f048cb6f2176c3e7a6534dc5af097</a><br>
DIFF: <a href="https://github.com/llvm/llvm-project/commit/76d845cb169f048cb6f2176c3e7a6534dc5af097.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/76d845cb169f048cb6f2176c3e7a6534dc5af097.diff</a><br>
<br>
LOG: [clang-format] Fix unittest failures with -Werror<br>
<br>
Commit a44ab1702539 added a unit test that fails to build with<br>
-Werror which causes build bot breaks on bots that include that<br>
option in their build. This patch just adds the necessary casts to<br>
silence the warnings.<br>
<br>
Added: <br>
<br>
<br>
Modified: <br>
    clang/unittests/Format/QualifierFixerTest.cpp<br>
<br>
Removed: <br>
<br>
<br>
<br>
################################################################################<br>
diff  --git a/clang/unittests/Format/QualifierFixerTest.cpp b/clang/unittests/Format/QualifierFixerTest.cpp<br>
index 1bb1792113049..0592cef1eaae5 100755<br>
--- a/clang/unittests/Format/QualifierFixerTest.cpp<br>
+++ b/clang/unittests/Format/QualifierFixerTest.cpp<br>
@@ -554,7 +554,7 @@ TEST_F(QualifierFixerTest, ConstVolatileQualifiersOrder) {<br>
   Style.QualifierOrder = {"inline", "static", "const", "volatile", "type"};<br>
<br>
   // The Default<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 5);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)5);<br>
<br>
   verifyFormat("const volatile int a;", "const volatile int a;", Style);<br>
   verifyFormat("const volatile int a;", "volatile const int a;", Style);<br>
@@ -603,7 +603,7 @@ TEST_F(QualifierFixerTest, InlineStatics) {<br>
   FormatStyle Style = getLLVMStyle();<br>
   Style.QualifierAlignment = FormatStyle::QAS_Left;<br>
   Style.QualifierOrder = {"inline", "static", "const", "volatile", "type"};<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 5);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)5);<br>
<br>
   verifyFormat("inline static const volatile int a;",<br>
                "const inline static volatile int a;", Style);<br>
@@ -621,7 +621,7 @@ TEST_F(QualifierFixerTest, AmpEqual) {<br>
   FormatStyle Style = getLLVMStyle();<br>
   Style.QualifierAlignment = FormatStyle::QAS_Custom;<br>
   Style.QualifierOrder = {"static", "type", "const"};<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 3);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)3);<br>
<br>
   verifyFormat("foo(std::string const & = std::string()) const",<br>
                "foo(const std::string & = std::string()) const", Style);<br>
@@ -634,7 +634,7 @@ TEST_F(QualifierFixerTest, MoveConstBeyondTypeSmall) {<br>
   FormatStyle Style = getLLVMStyle();<br>
   Style.QualifierAlignment = FormatStyle::QAS_Custom;<br>
   Style.QualifierOrder = {"type", "const"};<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 2);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)2);<br>
<br>
   verifyFormat("int const a;", "const int a;", Style);<br>
   verifyFormat("int const *a;", "const int*a;", Style);<br>
@@ -648,7 +648,7 @@ TEST_F(QualifierFixerTest, MoveConstBeforeTypeSmall) {<br>
   FormatStyle Style = getLLVMStyle();<br>
   Style.QualifierAlignment = FormatStyle::QAS_Custom;<br>
   Style.QualifierOrder = {"const", "type"};<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 2);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)2);<br>
<br>
   verifyFormat("const int a;", "int const a;", Style);<br>
   verifyFormat("const int *a;", "int const *a;", Style);<br>
@@ -670,7 +670,7 @@ TEST_F(QualifierFixerTest, MoveConstBeyondType) {<br>
   FormatStyle Style = getLLVMStyle();<br>
   Style.QualifierAlignment = FormatStyle::QAS_Custom;<br>
   Style.QualifierOrder = {"static", "inline", "type", "const", "volatile"};<br>
-  EXPECT_EQ(Style.QualifierOrder.size(), 5);<br>
+  EXPECT_EQ(Style.QualifierOrder.size(), (size_t)5);<br>
<br>
   verifyFormat("static inline int const volatile a;",<br>
                "const inline static volatile int a;", Style);<br>
@@ -698,8 +698,8 @@ TEST_F(QualifierFixerTest, PrepareLeftRightOrdering) {<br>
   QualifierAlignmentFixer::PrepareLeftRightOrdering(Style.QualifierOrder, Left,<br>
                                                     Right, ConfiguredTokens);<br>
<br>
-  EXPECT_EQ(Left.size(), 2);<br>
-  EXPECT_EQ(Right.size(), 2);<br>
+  EXPECT_EQ(Left.size(), (size_t)2);<br>
+  EXPECT_EQ(Right.size(), (size_t)2);<br>
<br>
   std::vector<std::string> LeftResult = {"inline", "static"};<br>
   std::vector<std::string> RightResult = {"const", "volatile"};<br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>