From: Sage Weil Date: Wed, 23 Dec 2015 16:58:17 +0000 (-0500) Subject: unittest_bluestore_types: fix warnings X-Git-Tag: v10.0.3~154^2~33 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=602b8de8348d04f60e6a76853150282eeb351390;p=ceph.git unittest_bluestore_types: fix warnings Signed-off-by: Sage Weil --- diff --git a/src/test/objectstore/test_bluestore_types.cc b/src/test/objectstore/test_bluestore_types.cc index 4e169d3f7df..bfe46d41e70 100644 --- a/src/test/objectstore/test_bluestore_types.cc +++ b/src/test/objectstore/test_bluestore_types.cc @@ -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)