From: Adam C. Emerson <aemerson@redhat.com>
Date: Fri, 9 Feb 2018 17:51:31 +0000 (-0500)
Subject: {client,test}: Explicitly initialize empty inode_t
X-Git-Tag: v13.0.2~288^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=b27e40d254f63ebe4a37de9aaf784ed5a1fdec9d;p=ceph.git

{client,test}: Explicitly initialize empty inode_t

Using an explicit {} initializer. This unbreaks Clang.

Signed-off-by: Adam C. Emerson <aemerson@redhat.com>
---

diff --git a/src/client/SyntheticClient.cc b/src/client/SyntheticClient.cc
index a610f1aebd87..502d251d19db 100644
--- a/src/client/SyntheticClient.cc
+++ b/src/client/SyntheticClient.cc
@@ -3357,7 +3357,7 @@ int SyntheticClient::chunk_file(string &filename)
   uint64_t size = st.st_size;
   dout(0) << "file " << filename << " size is " << size << dendl;
 
-  inode_t inode;
+  inode_t inode{};
   memset(&inode, 0, sizeof(inode));
   inode.ino = st.st_ino;
   ret = client->fdescribe_layout(fd, &inode.layout);
diff --git a/src/test/fs/mds_types.cc b/src/test/fs/mds_types.cc
index b29b95f4c600..0a698203ea1c 100644
--- a/src/test/fs/mds_types.cc
+++ b/src/test/fs/mds_types.cc
@@ -20,8 +20,8 @@
 
 TEST(inode_t, compare_equal)
 {
-  inode_t foo;
-  inode_t bar;
+  inode_t foo{};
+  inode_t bar{};
   int compare_r;
   bool divergent;
   compare_r = foo.compare(bar, &divergent);
@@ -51,8 +51,8 @@ TEST(inode_t, compare_equal)
 
 TEST(inode_t, compare_aged)
 {
-  inode_t foo;
-  inode_t bar;
+  inode_t foo{};
+  inode_t bar{};
 
   foo.ino = 1234;
   foo.ctime.set_from_double(10.0);
@@ -78,8 +78,8 @@ TEST(inode_t, compare_aged)
 
 TEST(inode_t, compare_divergent)
 {
-  inode_t foo;
-  inode_t bar;
+  inode_t foo{};
+  inode_t bar{};
 
   foo.ino = 1234;
   foo.ctime.set_from_double(10.0);