cleanup: use string & to avoid unnecessary copy

Signed-off-by: Yunchuan Wen <yunchuan.wen@kylin-cloud.com>

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 6daf383cbc25adcd9d889f65de825434845ffd96..11a068682ae31242ae5746bf5f4f280d693bcd76 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -1868,9 +1868,9 @@ bool PG::op_has_sufficient_caps(OpRequestRef& op)
   OSDCap& caps = session->caps;
   session->put();
 
-  string key = req->get_object_locator().key;
-  if (key.length() == 0)
-    key = req->get_oid().name;
+  const string &key = req->get_object_locator().key.empty() ?
+                      req->get_oid().name :
+                      req->get_object_locator().key;
 
   bool cap = caps.is_capable(pool.name, req->get_object_locator().nspace,
                              pool.auid, key,