crush: add const string& versions of accessors

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

diff --git a/src/crush/CrushWrapper.h b/src/crush/CrushWrapper.h
index 5bd9336270cca213b1ad200ddbc2a1927ff722ee..5531ece071932ff7b522701ee2a92dc50da840a2 100644 (file)
--- a/src/crush/CrushWrapper.h
+++ b/src/crush/CrushWrapper.h
@@ -152,19 +152,25 @@ public:
   // item/bucket names
   bool name_exists(const char *s) {
     string name(s);
+    return name_exists(name);
+  }
+  bool name_exists(const string& name) {
     build_rmaps();
     return name_rmap.count(name);
   }
   bool item_exists(int i) {
     return name_map.count(i);
   }
-  int get_item_id(const char *s) {
-    string name(s);
+  int get_item_id(const string& name) {
     build_rmaps();
     if (name_rmap.count(name))
       return name_rmap[name];
     return 0;  /* hrm */
   }
+  int get_item_id(const char *s) {
+    string name(s);
+    return get_item_id(name);
+  }
   const char *get_item_name(int t) const {
     std::map<int,string>::const_iterator p = name_map.find(t);
     if (p != name_map.end())
@@ -173,6 +179,9 @@ public:
   }
   void set_item_name(int i, const char *n) {
     string name(n);
+    set_item_name(i, name);
+  }
+  void set_item_name(int i, const string& name) {
     name_map[i] = name;
     if (have_rmaps)
       name_rmap[name] = i;