]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
test: test_denc.cc: silence warning from -Wsign-compare 15355/head
authorJos Collin <jcollin@redhat.com>
Tue, 30 May 2017 03:05:40 +0000 (08:35 +0530)
committerJos Collin <jcollin@redhat.com>
Tue, 30 May 2017 03:06:01 +0000 (08:36 +0530)
The following warning appears during make:
ceph/src/test/test_denc.cc:669:5:   required from here
ceph/src/googletest/googletest/include/gtest/gtest.h:1530:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
ceph/src/googletest/googletest/include/gtest/gtest.h:1510:7:
   if (val1 op val2) {\

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

index 8f702735bc809d39ede1b61f040ba149dfaa72ce..fea7e987eebc2322460c25439c80dec974852ff2 100644 (file)
@@ -666,7 +666,7 @@ TEST(denc, no_copy_if_segmented_and_lengthy)
     bufferlist segmented;
     segmented.append(large_bl);
     segmented.append(small_bl);
-    ASSERT_GT(segmented.get_num_buffers(), 1);
+    ASSERT_GT(segmented.get_num_buffers(), 1u);
     ASSERT_GT(segmented.length(), CEPH_PAGE_SIZE);
     auto p = segmented.begin();
     p.advance(large_bl.length());
@@ -684,7 +684,7 @@ TEST(denc, no_copy_if_segmented_and_lengthy)
     bufferlist segmented;
     segmented.append(small_bl);
     segmented.append(large_bl);
-    ASSERT_GT(segmented.get_num_buffers(), 1);
+    ASSERT_GT(segmented.get_num_buffers(), 1u);
     ASSERT_GT(segmented.length(), CEPH_PAGE_SIZE);
     auto p = segmented.begin();
     p.advance(small_bl.length());