]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
test: Fix warnings in chain_xattr tests 8914/head
authorAdam C. Emerson <aemerson@redhat.com>
Tue, 3 May 2016 16:53:33 +0000 (12:53 -0400)
committerAdam C. Emerson <aemerson@redhat.com>
Tue, 3 May 2016 17:26:54 +0000 (13:26 -0400)
Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
src/test/objectstore/chain_xattr.cc

index d3b4b1e9eeafae4d393e366fbf6d8229a982329b..475eb1665f6b8526981fc9836623ad5b8f9e4310 100644 (file)
@@ -298,30 +298,30 @@ TEST(chain_xattr, fskip_chain_cleanup_and_ensure_single_attr)
   memset(buf, sizeof(buf), 0x1F);
   // set chunked without either
   {
-    int r = chain_fsetxattr(fd, name, buf, sizeof(buf));
+    std::size_t r = chain_fsetxattr(fd, name, buf, sizeof(buf));
     ASSERT_EQ(r, sizeof(buf));
-    ASSERT_GT(get_xattrs(fd).size(), 1);
+    ASSERT_GT(get_xattrs(fd).size(), 1UL);
   }
 
   // verify
   {
     char buf2[sizeof(buf)*2];
-    int r = chain_fgetxattr(fd, name, buf2, sizeof(buf2));
+    std::size_t r = chain_fgetxattr(fd, name, buf2, sizeof(buf2));
     ASSERT_EQ(r, sizeof(buf));
     ASSERT_EQ(memcmp(buf, buf2, sizeof(buf)), 0);
   }
 
   // overwrite
   {
-    int r = chain_fsetxattr<false, true>(fd, name, buf, sizeof(buf));
-    ASSERT_EQ(r, sizeof (buf));
-    ASSERT_EQ(get_xattrs(fd).size(), 1);
+    std::size_t r = chain_fsetxattr<false, true>(fd, name, buf, sizeof(buf));
+    ASSERT_EQ(r, sizeof(buf));
+    ASSERT_EQ(get_xattrs(fd).size(), 1UL);
   }
 
   // verify
   {
     char buf2[sizeof(buf)*2];
-    int r = chain_fgetxattr(fd, name, buf2, sizeof(buf2));
+    std::size_t r = chain_fgetxattr(fd, name, buf2, sizeof(buf2));
     ASSERT_EQ(r, sizeof(buf));
     ASSERT_EQ(memcmp(buf, buf2, sizeof(buf)), 0);
   }
@@ -342,30 +342,30 @@ TEST(chain_xattr, skip_chain_cleanup_and_ensure_single_attr)
   memset(buf, sizeof(buf), 0x1F);
   // set chunked without either
   {
-    int r = chain_setxattr(file, name, buf, sizeof(buf));
+    std::size_t r = chain_setxattr(file, name, buf, sizeof(buf));
     ASSERT_EQ(r, sizeof(buf));
-    ASSERT_GT(get_xattrs(file).size(), 1);
+    ASSERT_GT(get_xattrs(file).size(), 1UL);
   }
 
   // verify
   {
     char buf2[sizeof(buf)*2];
-    int r = chain_getxattr(file, name, buf2, sizeof(buf2));
+    std::size_t r = chain_getxattr(file, name, buf2, sizeof(buf2));
     ASSERT_EQ(r, sizeof(buf));
     ASSERT_EQ(memcmp(buf, buf2, sizeof(buf)), 0);
   }
 
   // overwrite
   {
-    int r = chain_setxattr<false, true>(file, name, buf, sizeof(buf));
-    ASSERT_EQ(r, sizeof (buf));
-    ASSERT_EQ(get_xattrs(file).size(), 1);
+    std::size_t r = chain_setxattr<false, true>(file, name, buf, sizeof(buf));
+    ASSERT_EQ(r, sizeof(buf));
+    ASSERT_EQ(get_xattrs(file).size(), 1UL);
   }
 
   // verify
   {
     char buf2[sizeof(buf)*2];
-    int r = chain_getxattr(file, name, buf2, sizeof(buf2));
+    std::size_t r = chain_getxattr(file, name, buf2, sizeof(buf2));
     ASSERT_EQ(r, sizeof(buf));
     ASSERT_EQ(memcmp(buf, buf2, sizeof(buf)), 0);
   }