rgw: Do not std::move to return locally constructed objects

It prevents return value optimization.

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

diff --git a/src/rgw/rgw_b64.h b/src/rgw/rgw_b64.h
index 36c984e815a8e8a4a5c52ee1966d216ae95f2948..52374f49363518bc2a3d352b80009f3ca6c01a22 100644 (file)
--- a/src/rgw/rgw_b64.h
+++ b/src/rgw/rgw_b64.h
@@ -55,7 +55,7 @@ namespace rgw {
     for (size_t ix = 0; ix < (psize-sref.size()); ++ix)
       outstr.push_back('=');
 
-    return std::move(outstr);
+    return outstr;
   }
 
   inline std::string from_base64(boost::string_ref sref)
@@ -78,7 +78,7 @@ namespace rgw {
     std::string outstr(b64_iter(sref.data()),
                      b64_iter(sref.data() + sref.size()));
 
-    return std::move(outstr);
+    return outstr;
   }
 
 } /* namespace */

diff --git a/src/rgw/rgw_ldap.cc b/src/rgw/rgw_ldap.cc
index e8915e30daff84cf94d88dc660bdff11b6e5853f..d07b295153e68e49db650c43373c6a6df1e3b856 100644 (file)
--- a/src/rgw/rgw_ldap.cc
+++ b/src/rgw/rgw_ldap.cc
@@ -35,7 +35,7 @@ std::string parse_rgw_ldap_bindpw(CephContext* ctx)
     }
   }
 
-  return std::move(ldap_bindpw);
+  return ldap_bindpw;
 }
 
 #if defined(HAVE_OPENLDAP)

diff --git a/src/rgw/rgw_token.h b/src/rgw/rgw_token.h
index c357733e3f5272a1e9f127efcefd8e7275c5bb00..b13fe468634bcab15818a4032ac2b57dd7eb1725 100644 (file)
--- a/src/rgw/rgw_token.h
+++ b/src/rgw/rgw_token.h
@@ -144,7 +144,7 @@ namespace rgw {
       encode_json(f);
       std::ostringstream os;
       f->flush(os);
-      return std::move(to_base64(std::move(os.str())));
+      return to_base64(std::move(os.str()));
     }
 
     friend inline ostream& operator<<(ostream& os, const RGWToken& token);