MOSDOpreply: The functions are returned by const ref and parameters passed by ref

get_oid,get_pg,get_replay_version,get_user_version are returning by reference and
corresponding setter is accepting param by reference.

Signed-off-by: Somnath Roy <somnath.roy@sandisk.com>

diff --git a/src/messages/MOSDOpReply.h b/src/messages/MOSDOpReply.h
index 91c50e76dc262e9204eddf59c4a2b76da98adac6..4a677d68180d4e8fe1940acef515236c6757fdcf 100644 (file)
--- a/src/messages/MOSDOpReply.h
+++ b/src/messages/MOSDOpReply.h
@@ -48,16 +48,16 @@ class MOSDOpReply : public Message {
   request_redirect_t redirect;
 
 public:
-  object_t get_oid() const { return oid; }
-  pg_t     get_pg() const { return pgid; }
+  const object_t& get_oid() const { return oid; }
+  const pg_t&     get_pg() const { return pgid; }
   int      get_flags() const { return flags; }
 
   bool     is_ondisk() const { return get_flags() & CEPH_OSD_FLAG_ONDISK; }
   bool     is_onnvram() const { return get_flags() & CEPH_OSD_FLAG_ONNVRAM; }
   
   int get_result() const { return result; }
-  eversion_t get_replay_version() const { return replay_version; }
-  version_t get_user_version() const { return user_version; }
+  const eversion_t& get_replay_version() const { return replay_version; }
+  const version_t& get_user_version() const { return user_version; }
   
   void set_result(int r) { result = r; }
 
@@ -84,7 +84,7 @@ public:
   }
 
   /* Don't fill in replay_version for non-write ops */
-  void set_enoent_reply_versions(eversion_t v, version_t uv) {
+  void set_enoent_reply_versions(const eversion_t& v, const version_t& uv) {
     user_version = uv;
     bad_replay_version = v;
   }