]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
test: silence warning from -Wsign-compare 17790/head
authorJos Collin <jcollin@redhat.com>
Tue, 19 Sep 2017 06:30:26 +0000 (12:00 +0530)
committerJos Collin <jcollin@redhat.com>
Tue, 19 Sep 2017 06:57:37 +0000 (12:27 +0530)
Fixed warnings:
ceph/src/test/test_xlist.cc:46:3:   required from here
ceph/src/googletest/googletest/include/gtest/gtest.h:1392:11: warning: comparison between
signed and unsigned integer expressions [-Wsign-compare]

ceph/src/test/librados/tier.cc:1360:5:   required from here
ceph/src/googletest/googletest/include/gtest/gtest.h:1526:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

ceph/src/test/librados/tier.cc:1365:2:   required from here
ceph/src/googletest/googletest/include/gtest/gtest.h:1526:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

Signed-off-by: Jos Collin <jcollin@redhat.com>
src/test/librados/tier.cc
src/test/test_xlist.cc

index f84fcccf5bd70ce597f1ad0a280b9065d999faf2..70f9c9445aede4b544ba189a05944efba91a1804 100644 (file)
@@ -1357,12 +1357,12 @@ TEST_F(LibRadosTwoPoolsPP, ListSnap){
       0, NULL));
     completion->wait_for_safe();
     ASSERT_EQ(0, snap_ret);
-    ASSERT_LT(0, snap_set.clones.size());
+    ASSERT_LT(0u, snap_set.clones.size());
     for (vector<librados::clone_info_t>::const_iterator r = snap_set.clones.begin();
        r != snap_set.clones.end();
        ++r) {
       if (r->cloneid != librados::SNAP_HEAD) {
-       ASSERT_LT(0, r->snaps.size());
+       ASSERT_LT(0u, r->snaps.size());
       }
     }
   }
index f13bf41dd68809369476e4cccd388d259882de84..35b62edcf7903454f06c1b59b57b7946bbf2165d 100644 (file)
@@ -43,14 +43,14 @@ protected:
 TEST_F(XlistTest, capability) {
   ItemList list;
   ASSERT_TRUE(list.empty());
-  ASSERT_EQ(list.size(), 0);
+  ASSERT_EQ(0u, list.size());
 
   std::copy(refs.begin(), refs.end(), std::back_inserter(list));
   ASSERT_EQ((size_t)list.size(), refs.size());
 
   list.clear();
   ASSERT_TRUE(list.empty());
-  ASSERT_EQ(list.size(), 0);
+  ASSERT_EQ(0u, list.size());
 }
 
 TEST_F(XlistTest, traverse) {