librados: clean up code.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

diff --git a/src/librados/librados.cc b/src/librados/librados.cc
index 6c67f63d2517a11de5cc61b101c3a581d19ea937..538902cc5494cc8df55c24ff877c056bbb8b4b1a 100644 (file)
--- a/src/librados/librados.cc
+++ b/src/librados/librados.cc
@@ -2217,9 +2217,8 @@ extern "C" int rados_create_with_context(rados_t *pcluster, rados_config_t cct_)
   CephContext *cct = (CephContext *)cct_;
   librados::RadosClient *radosp = new librados::RadosClient(cct);
   *pcluster = (void *)radosp;
-  int retval = 0;
-  tracepoint(librados, rados_create_with_context_exit, retval, *pcluster);
-  return retval;
+  tracepoint(librados, rados_create_with_context_exit, 0, *pcluster);
+  return 0;
 }
 
 extern "C" rados_config_t rados_cct(rados_t cluster)
@@ -2287,9 +2286,8 @@ extern "C" int rados_conf_read_file(rados_t cluster, const char *path_list)
 
   conf->apply_changes(NULL);
   complain_about_parse_errors(client->cct, &parse_errors);
-  int retval = 0;
-  tracepoint(librados, rados_conf_read_file_exit, retval);
-  return retval;
+  tracepoint(librados, rados_conf_read_file_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_conf_parse_argv(rados_t cluster, int argc, const char **argv)
@@ -2309,9 +2307,8 @@ extern "C" int rados_conf_parse_argv(rados_t cluster, int argc, const char **arg
     return ret;
   }
   conf->apply_changes(NULL);
-  int retval = 0;
-  tracepoint(librados, rados_conf_parse_argv_exit, retval);
-  return retval;
+  tracepoint(librados, rados_conf_parse_argv_exit, 0);
+  return 0;
 }
 
 // like above, but return the remainder of argv to contain remaining
@@ -2347,9 +2344,8 @@ extern "C" int rados_conf_parse_argv_remainder(rados_t cluster, int argc,
       remargv[i] = (const char *)NULL;
     tracepoint(librados, rados_conf_parse_argv_remainder_remarg, remargv[i]);
   }
-  int retval = 0;
-  tracepoint(librados, rados_conf_parse_argv_remainder_exit, retval);
-  return retval;
+  tracepoint(librados, rados_conf_parse_argv_remainder_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_conf_parse_env(rados_t cluster, const char *env)
@@ -2365,9 +2361,8 @@ extern "C" int rados_conf_parse_env(rados_t cluster, const char *env)
     return ret;
   }
   conf->apply_changes(NULL);
-  int retval = 0;
-  tracepoint(librados, rados_conf_parse_env_exit, retval);
-  return retval;
+  tracepoint(librados, rados_conf_parse_env_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_conf_set(rados_t cluster, const char *option, const char *value)
@@ -2381,9 +2376,8 @@ extern "C" int rados_conf_set(rados_t cluster, const char *option, const char *v
     return ret;
   }
   conf->apply_changes(NULL);
-  int retval = 0;
-  tracepoint(librados, rados_conf_set_exit, retval);
-  return retval;
+  tracepoint(librados, rados_conf_set_exit, 0);
+  return 0;
 }
 
 /* cluster info */
@@ -2721,9 +2715,8 @@ extern "C" int rados_ioctx_create(rados_t cluster, const char *name, rados_ioctx
 
   *io = ctx;
   ctx->get();
-  int retval = 0;
-  tracepoint(librados, rados_ioctx_create_exit, retval, ctx);
-  return retval;
+  tracepoint(librados, rados_ioctx_create_exit, 0, ctx);
+  return 0;
 }
 
 extern "C" int rados_ioctx_create2(rados_t cluster, int64_t pool_id,
@@ -2741,9 +2734,8 @@ extern "C" int rados_ioctx_create2(rados_t cluster, int64_t pool_id,
 
   *io = ctx;
   ctx->get();
-  int retval = 0;
-  tracepoint(librados, rados_ioctx_create2_exit, retval, ctx);
-  return retval;
+  tracepoint(librados, rados_ioctx_create2_exit, 0, ctx);
+  return 0;
 }
 
 extern "C" void rados_ioctx_destroy(rados_ioctx_t io)
@@ -2783,9 +2775,8 @@ extern "C" int rados_ioctx_pool_stat(rados_ioctx_t io, struct rados_pool_stat_t
   stats->num_rd_kb = r.stats.sum.num_rd_kb;
   stats->num_wr = r.stats.sum.num_wr;
   stats->num_wr_kb = r.stats.sum.num_wr_kb;
-  int retval = 0;
-  tracepoint(librados, rados_ioctx_pool_stat_exit, retval, stats);
-  return retval;
+  tracepoint(librados, rados_ioctx_pool_stat_exit, 0, stats);
+  return 0;
 }
 
 extern "C" rados_config_t rados_ioctx_cct(rados_ioctx_t io)
@@ -3198,9 +3189,8 @@ extern "C" int rados_ioctx_snap_get_name(rados_ioctx_t io, rados_snap_t id,
     return retval;
   }
   strncpy(name, sname.c_str(), maxlen);
-  int retval = 0;
-  tracepoint(librados, rados_ioctx_snap_get_name_exit, retval, name);
-  return retval;
+  tracepoint(librados, rados_ioctx_snap_get_name_exit, 0, name);
+  return 0;
 }
 
 extern "C" int rados_ioctx_snap_get_stamp(rados_ioctx_t io, rados_snap_t id, time_t *t)
@@ -3428,9 +3418,8 @@ extern "C" int rados_nobjects_list_open(rados_ioctx_t io, rados_list_ctx_t *list
   h->pool_snap_seq = ctx->snap_seq;
   h->nspace = ctx->oloc.nspace;        // After dropping compatibility need nspace
   *listh = (void *)new librados::ObjListCtx(ctx, h);
-  int retval = 0;
-  tracepoint(librados, rados_nobjects_list_open_exit, retval, *listh);
-  return retval;
+  tracepoint(librados, rados_nobjects_list_open_exit, 0, *listh);
+  return 0;
 }
 
 extern "C" void rados_nobjects_list_close(rados_list_ctx_t h)
@@ -3481,9 +3470,8 @@ extern "C" int rados_objects_list_open(rados_ioctx_t io, rados_list_ctx_t *listh
   h->pool_snap_seq = ctx->snap_seq;
   h->nspace = ctx->oloc.nspace;
   *listh = (void *)new librados::ObjListCtx(ctx, h);
-  int retval = 0;
-  tracepoint(librados, rados_objects_list_open_exit, retval, *listh);
-  return retval;
+  tracepoint(librados, rados_objects_list_open_exit, 0, *listh);
+  return 0;
 }
 
 // Deprecated, but using it for compatibility with older OSDs
@@ -3556,9 +3544,8 @@ extern "C" int rados_nobjects_list_next(rados_list_ctx_t listctx, const char **e
   }
   if (nspace)
     *nspace = h->list.front().nspace.c_str();
-  int retval = 0;
   tracepoint(librados, rados_nobjects_list_next_exit, 0, *entry, key, nspace);
-  return retval;
+  return 0;
 }
 
 // DEPRECATED
@@ -3597,9 +3584,8 @@ extern "C" int rados_objects_list_next(rados_list_ctx_t listctx, const char **en
     else
       *key = NULL;
   }
-  int retval = 0;
   tracepoint(librados, rados_objects_list_next_exit, 0, *entry, key);
-  return retval;
+  return 0;
 }
 
 
@@ -3619,9 +3605,8 @@ extern "C" int rados_aio_create_completion(void *cb_arg,
   if (cb_safe)
     c->set_safe_callback(cb_arg, cb_safe);
   *pc = c;
-  int retval = 0;
-  tracepoint(librados, rados_aio_create_completion_exit, retval, *pc);
-  return retval;
+  tracepoint(librados, rados_aio_create_completion_exit, 0, *pc);
+  return 0;
 }
 
 extern "C" int rados_aio_wait_for_complete(rados_completion_t c)
@@ -3791,9 +3776,8 @@ extern "C" int rados_aio_flush_async(rados_ioctx_t io,
   tracepoint(librados, rados_aio_flush_async_enter, io, completion);
   librados::IoCtxImpl *ctx = (librados::IoCtxImpl *)io;
   ctx->flush_aio_writes_async((librados::AioCompletionImpl*)completion);
-  int retval = 0;
-  tracepoint(librados, rados_aio_flush_async_exit, retval);
-  return retval;
+  tracepoint(librados, rados_aio_flush_async_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_aio_flush(rados_ioctx_t io)
@@ -3801,9 +3785,8 @@ extern "C" int rados_aio_flush(rados_ioctx_t io)
   tracepoint(librados, rados_aio_flush_enter, io);
   librados::IoCtxImpl *ctx = (librados::IoCtxImpl *)io;
   ctx->flush_aio_writes();
-  int retval = 0;
-  tracepoint(librados, rados_aio_flush_exit, retval);
-  return retval;
+  tracepoint(librados, rados_aio_flush_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_aio_stat(rados_ioctx_t io, const char *o, 
@@ -3968,9 +3951,8 @@ extern "C" int rados_notify_ack(rados_ioctx_t io, const char *o,
     bl.push_back(p);
   }
   ctx->notify_ack(oid, notify_id, handle, bl);
-  int retval = 0;
-  tracepoint(librados, rados_notify_ack_exit, retval);
-  return retval;
+  tracepoint(librados, rados_notify_ack_exit, 0);
+  return 0;
 }
 
 extern "C" int rados_watch_flush(rados_t cluster)
@@ -4643,9 +4625,8 @@ extern "C" int rados_omap_get_next(rados_omap_iter_t iter,
     *key = NULL;
     *val = NULL;
     *len = 0;
-    int retval = 0;
-    tracepoint(librados, rados_omap_get_next_exit, retval, key, val, len);
-    return retval;
+    tracepoint(librados, rados_omap_get_next_exit, 0, key, val, len);
+    return 0;
   }
   if (key)
     *key = (char*)it->i->first.c_str();
@@ -4654,9 +4635,8 @@ extern "C" int rados_omap_get_next(rados_omap_iter_t iter,
   if (len)
     *len = it->i->second.length();
   ++it->i;
-  int retval = 0;
-  tracepoint(librados, rados_omap_get_next_exit, retval, key, val, len);
-  return retval;
+  tracepoint(librados, rados_omap_get_next_exit, 0, key, val, len);
+  return 0;
 }
 
 extern "C" void rados_omap_get_end(rados_omap_iter_t iter)