]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
unittest_bluestore_types: fix warnings
authorSage Weil <sage@redhat.com>
Wed, 23 Dec 2015 16:58:17 +0000 (11:58 -0500)
committerSage Weil <sage@redhat.com>
Fri, 1 Jan 2016 18:08:53 +0000 (13:08 -0500)
Signed-off-by: Sage Weil <sage@redhat.com>
src/test/objectstore/test_bluestore_types.cc

index 4e169d3f7df83ed84783a9ee63c84cff10cbb0b6..bfe46d41e70cbdfa2564f39a48a973b45501604d 100644 (file)
@@ -12,13 +12,13 @@ TEST(bluestore_extent_ref_map_t, add)
 {
   bluestore_extent_ref_map_t m;
   m.add(10, 10);
-  ASSERT_EQ(1, m.ref_map.size());
+  ASSERT_EQ(1u, m.ref_map.size());
   cout << m << std::endl;
   m.add(20, 10);
   cout << m << std::endl;
-  ASSERT_EQ(1, m.ref_map.size());
+  ASSERT_EQ(1u, m.ref_map.size());
   ASSERT_EQ(20u, m.ref_map[10].length);
-  ASSERT_EQ(2, m.ref_map[10].refs);
+  ASSERT_EQ(2u, m.ref_map[10].refs);
   m.add(40, 10);
   cout << m << std::endl;
   ASSERT_EQ(2u, m.ref_map.size());
@@ -84,48 +84,48 @@ TEST(bluestore_extent_ref_map_t, put)
   m.add(10, 30, 1);
   m.put(10, 30, &r);
   cout << m << " " << r << std::endl;
-  ASSERT_EQ(0, m.ref_map.size());
-  ASSERT_EQ(1, r.size());
-  ASSERT_EQ(10, r[0].offset);
-  ASSERT_EQ(30, r[0].length);
+  ASSERT_EQ(0u, m.ref_map.size());
+  ASSERT_EQ(1u, r.size());
+  ASSERT_EQ(10u, r[0].offset);
+  ASSERT_EQ(30u, r[0].length);
   r.clear();
   m.add(10, 30, 1);
   m.get(20, 10);
   m.put(10, 30, &r);
   cout << m << " " << r << std::endl;
-  ASSERT_EQ(1, m.ref_map.size());
-  ASSERT_EQ(10, m.ref_map[20].length);
-  ASSERT_EQ(1, m.ref_map[20].refs);
-  ASSERT_EQ(2, r.size());
-  ASSERT_EQ(10, r[0].offset);
-  ASSERT_EQ(10, r[0].length);
-  ASSERT_EQ(30, r[1].offset);
-  ASSERT_EQ(10, r[1].length);
+  ASSERT_EQ(1u, m.ref_map.size());
+  ASSERT_EQ(10u, m.ref_map[20].length);
+  ASSERT_EQ(1u, m.ref_map[20].refs);
+  ASSERT_EQ(2u, r.size());
+  ASSERT_EQ(10u, r[0].offset);
+  ASSERT_EQ(10u, r[0].length);
+  ASSERT_EQ(30u, r[1].offset);
+  ASSERT_EQ(10u, r[1].length);
   r.clear();
   m.add(30, 10);
   m.put(20, 15, &r);
   cout << m << " " << r << std::endl;
-  ASSERT_EQ(2, m.ref_map.size());
-  ASSERT_EQ(5, m.ref_map[30].length);
-  ASSERT_EQ(1, m.ref_map[30].refs);
-  ASSERT_EQ(5, m.ref_map[35].length);
-  ASSERT_EQ(2, m.ref_map[35].refs);
-  ASSERT_EQ(1, r.size());
-  ASSERT_EQ(20, r[0].offset);
-  ASSERT_EQ(10, r[0].length);
+  ASSERT_EQ(2u, m.ref_map.size());
+  ASSERT_EQ(5u, m.ref_map[30].length);
+  ASSERT_EQ(1u, m.ref_map[30].refs);
+  ASSERT_EQ(5u, m.ref_map[35].length);
+  ASSERT_EQ(2u, m.ref_map[35].refs);
+  ASSERT_EQ(1u, r.size());
+  ASSERT_EQ(20u, r[0].offset);
+  ASSERT_EQ(10u, r[0].length);
   r.clear();
   m.put(33, 5, &r);
   cout << m << " " << r << std::endl;
-  ASSERT_EQ(3, m.ref_map.size());
-  ASSERT_EQ(3, m.ref_map[30].length);
-  ASSERT_EQ(1, m.ref_map[30].refs);
-  ASSERT_EQ(3, m.ref_map[35].length);
-  ASSERT_EQ(1, m.ref_map[35].refs);
-  ASSERT_EQ(2, m.ref_map[38].length);
-  ASSERT_EQ(2, m.ref_map[38].refs);
-  ASSERT_EQ(1, r.size());
-  ASSERT_EQ(33, r[0].offset);
-  ASSERT_EQ(2, r[0].length);
+  ASSERT_EQ(3u, m.ref_map.size());
+  ASSERT_EQ(3u, m.ref_map[30].length);
+  ASSERT_EQ(1u, m.ref_map[30].refs);
+  ASSERT_EQ(3u, m.ref_map[35].length);
+  ASSERT_EQ(1u, m.ref_map[35].refs);
+  ASSERT_EQ(2u, m.ref_map[38].length);
+  ASSERT_EQ(2u, m.ref_map[38].refs);
+  ASSERT_EQ(1u, r.size());
+  ASSERT_EQ(33u, r[0].offset);
+  ASSERT_EQ(2u, r[0].length);
 }
 
 TEST(bluestore_extent_ref_map_t, contains)