OSDMap: rename _raw_to_temp_osds() -> _get_temp_osds()

This function does not (and never has!) used the raw vector, so remove it
and don't use a name which implies it is doing any sort of conversion.

Signed-off-by: Greg Farnum <greg@inktank.com>

diff --git a/src/osd/OSDMap.cc b/src/osd/OSDMap.cc
index 55742d9fc7c1ce6ec17cf64792e8c2fbbc935e5b..7c0f5dbb240a5b7b4f1691272a6759c99f162300 100644 (file)
--- a/src/osd/OSDMap.cc
+++ b/src/osd/OSDMap.cc
@@ -1153,7 +1153,7 @@ void OSDMap::_raw_to_up_osds(pg_t pg, vector<int>& raw, vector<int>& up) const
   }
 }
   
-bool OSDMap::_raw_to_temp_osds(const pg_pool_t& pool, pg_t pg, vector<int>& raw, vector<int>& temp) const
+bool OSDMap::_get_temp_osds(const pg_pool_t& pool, pg_t pg, vector<int>& temp) const
 {
   pg = pool.raw_pg_to_pg(pg);
   map<pg_t,vector<int> >::const_iterator p = pg_temp->find(pg);
@@ -1196,7 +1196,7 @@ void OSDMap::_pg_to_up_acting_osds(pg_t pg, vector<int> *up, vector<int>& acting
   vector<int> *_up = (up ? up : new vector<int>);
   _pg_to_osds(*pool, pg, raw);
   _raw_to_up_osds(pg, raw, *up);
-  if (!_raw_to_temp_osds(*pool, pg, raw, acting))
+  if (!_get_temp_osds(*pool, pg, acting))
     acting = *_up;
   if (_up != up)
     delete _up;

diff --git a/src/osd/OSDMap.h b/src/osd/OSDMap.h
index cfc9e0df4ca6bebcafeac94ed2e8f1668880be83..e44faf0c98264d8fd9289efb6a8d34501a547b35 100644 (file)
--- a/src/osd/OSDMap.h
+++ b/src/osd/OSDMap.h
@@ -504,7 +504,7 @@ private:
   /// pg -> (up osd list)
   void _raw_to_up_osds(pg_t pg, vector<int>& raw, vector<int>& up) const;
 
-  bool _raw_to_temp_osds(const pg_pool_t& pool, pg_t pg, vector<int>& raw, vector<int>& temp) const;
+  bool _get_temp_osds(const pg_pool_t& pool, pg_t pg, vector<int>& temp) const;
 
   /// map to up and acting. Only provides up if pointer is non-NULL
   void _pg_to_up_acting_osds(pg_t pg, vector<int> *up,