mds/Mantle: pass parameters by const ref

Signed-off-by: Michal Jarzabek <stiopa@gmail.com>

diff --git a/src/mds/Mantle.cc b/src/mds/Mantle.cc
index eb4fff5520a74aab8eeeee9f8e3f773e8df5517d..e9a65ef1a0cba4a71a8b2a1e87ec1eb16ed6f9d4 100644 (file)
--- a/src/mds/Mantle.cc
+++ b/src/mds/Mantle.cc
@@ -67,7 +67,7 @@ int Mantle::start()
   return 0;
 }
 
-int Mantle::execute(string script)
+int Mantle::execute(const string &script)
 {
   if (L == NULL) {
     dout(0) << "ERROR: mantle was not started" << dendl;
@@ -93,9 +93,9 @@ int Mantle::execute(string script)
   return 0;
 }
 
-int Mantle::balance(string script,
+int Mantle::balance(const string &script,
                     mds_rank_t whoami,
-                    vector < map<string, double> > metrics,
+                    const vector < map<string, double> > &metrics,
                     map<mds_rank_t,double> &my_targets)
 {
   if (start() != 0)
@@ -114,7 +114,7 @@ int Mantle::balance(string script,
     lua_newtable(L);
 
     /* push values into this mds's table; setfield assigns key/pops val */
-    for (map<string, double>::iterator it = metrics[i].begin();
+    for (map<string, double>::const_iterator it = metrics[i].begin();
          it != metrics[i].end();
          it++) {
       lua_pushnumber(L, it->second);

diff --git a/src/mds/Mantle.h b/src/mds/Mantle.h
index 7aee990b6a12e98cc9b4e77695b6bdc9863ff5cf..e3348a64f96a22d8878830c822082f5ded127b80 100644 (file)
--- a/src/mds/Mantle.h
+++ b/src/mds/Mantle.h
@@ -31,10 +31,10 @@ class Mantle {
   public:
     Mantle() : L(NULL) {};
     int start();
-    int execute(string script);
-    int balance(string script,
+    int execute(const string &script);
+    int balance(const string &script,
                 mds_rank_t whoami,
-                vector < map<string, double> > metrics,
+                const vector < map<string, double> > &metrics,
                 map<mds_rank_t,double> &my_targets);
 };