From 0fedc28dac0399d9a376efdaef0d90591df7f851 Mon Sep 17 00:00:00 2001
From: Sage Weil <sage@inktank.com>
Date: Sun, 21 Oct 2012 15:29:23 -0700
Subject: [PATCH] monmap: pass by ref

CID 717039 (#1 of 1): Big parameter passed by value (PASS_BY_VALUE)
Passing parameter my_addr of type entity_addr_t (size 136 bytes) by value.

CID 717040: Big parameter passed by value (PASS_BY_VALUE)
Passing parameter a of type entity_addr_t (size 136 bytes) by value.

CID 717041: Big parameter passed by value (PASS_BY_VALUE)
Passing parameter a of type entity_addr_t (size 136 bytes) by value.

CID 717043: Big parameter passed by value (PASS_BY_VALUE)
Passing parameter a of type entity_addr_t (size 136 bytes) by value.

Signed-off-by: Sage Weil <sage@inktank.com>
---
 src/mon/MonMap.cc |  2 +-
 src/mon/MonMap.h  | 10 +++++-----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/mon/MonMap.cc b/src/mon/MonMap.cc
index 88666cb4cbc8e..8d26d41f7b1b9 100644
--- a/src/mon/MonMap.cc
+++ b/src/mon/MonMap.cc
@@ -195,7 +195,7 @@ int MonMap::build_from_host_list(std::string hostlist, std::string prefix)
 
 void MonMap::set_initial_members(CephContext *cct,
 				 list<std::string>& initial_members,
-				 string my_name, entity_addr_t my_addr,
+				 string my_name, const entity_addr_t& my_addr,
 				 set<entity_addr_t> *removed)
 {
   // remove non-initial members
diff --git a/src/mon/MonMap.h b/src/mon/MonMap.h
index 6ff2cf127d0b4..a8734b8eae9db 100644
--- a/src/mon/MonMap.h
+++ b/src/mon/MonMap.h
@@ -130,7 +130,7 @@ class MonMap {
     assert(n < rank_name.size());
     return rank_name[n];
   }
-  string get_name(entity_addr_t a) const {
+  string get_name(const entity_addr_t& a) const {
     map<entity_addr_t,string>::const_iterator p = addr_name.find(a);
     if (p == addr_name.end())
       return string();
@@ -144,13 +144,13 @@ class MonMap {
 	return i;
     return -1;
   }
-  int get_rank(entity_addr_t a) {
+  int get_rank(const entity_addr_t& a) {
     for (unsigned i=0; i<rank_addr.size(); i++)
       if (rank_addr[i] == a)
 	return i;
     return -1;
   }
-  bool get_addr_name(entity_addr_t a, string& name) {
+  bool get_addr_name(const entity_addr_t& a, string& name) {
     if (addr_name.count(a) == 0)
       return false;
     name = addr_name[a];
@@ -165,7 +165,7 @@ class MonMap {
     assert(m < rank_addr.size());
     return rank_addr[m];
   }
-  void set_addr(const string& n, entity_addr_t a) {
+  void set_addr(const string& n, const entity_addr_t& a) {
     assert(mon_addr.count(n));
     mon_addr[n] = a;
     calc_ranks();
@@ -242,7 +242,7 @@ class MonMap {
    */
   void set_initial_members(CephContext *cct,
 			   list<std::string>& initial_members,
-			   string my_name, entity_addr_t my_addr,
+			   string my_name, const entity_addr_t& my_addr,
 			   set<entity_addr_t> *removed);
 
   void print(ostream& out) const;
-- 
2.39.5