]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
kv_flat_btree_async.cc: fix assigned but unused value issue
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Fri, 31 May 2013 20:26:17 +0000 (22:26 +0200)
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>
Mon, 3 Jun 2013 09:55:50 +0000 (11:55 +0200)
Fix "Variable 'r' is assigned a value that is never used." issue from
cppcheck. Check return value and print warning if in verbose mode.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
src/key_value_store/kv_flat_btree_async.cc

index 8573a43b6c951f6343b131568476fe62553ae971..68e1eab6b8401165e3a19dd6127607e4ffe82cb7 100644 (file)
@@ -1371,9 +1371,13 @@ int KvFlatBtreeAsync::setup(int argc, const char** argv) {
   make_max_obj.setxattr("unwritable", to_bl("0"));
   make_max_obj.setxattr("size", to_bl("0"));
   r = io_ctx.operate(client_name, &make_max_obj);
+  if (r < 0) {
+    if (verbose) cout << client_name << ": Setting xattr failed with code "
+       << r
+       << std::endl;
+  }
 
   return 0;
-
 }
 
 int KvFlatBtreeAsync::set(const string &key, const bufferlist &val,