osd/SnapMapper: pass snaps set by const ref

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/osd/SnapMapper.cc b/src/osd/SnapMapper.cc
index 4a6ed07c68e8826957202b603fc0633b58921070..3a5e09884a8971a1837ca80a975282f01db4840a 100644 (file)
--- a/src/osd/SnapMapper.cc
+++ b/src/osd/SnapMapper.cc
@@ -217,7 +217,7 @@ int SnapMapper::update_snaps(
 
 void SnapMapper::add_oid(
   const hobject_t &oid,
-  set<snapid_t> snaps,
+  const set<snapid_t>& snaps,
   MapCacher::Transaction<std::string, bufferlist> *t)
 {
   dout(20) << __func__ << " " << oid << " " << snaps << dendl;

diff --git a/src/osd/SnapMapper.h b/src/osd/SnapMapper.h
index 5565e175bd120ce621e62ee03883f2f0636b3929..d597a6b6cfd15c877a3073d52dcaeec7de193473 100644 (file)
--- a/src/osd/SnapMapper.h
+++ b/src/osd/SnapMapper.h
@@ -206,7 +206,7 @@ public:
   /// Add mapping for oid, must not already be mapped
   void add_oid(
     const hobject_t &oid,       ///< [in] oid to add
-    std::set<snapid_t> new_snaps, ///< [in] snaps
+    const std::set<snapid_t>& new_snaps, ///< [in] snaps
     MapCacher::Transaction<std::string, bufferlist> *t ///< [out] transaction
     );