From 4adbafaa4316c67addbde624a3dbbc7fa2bf8ee5 Mon Sep 17 00:00:00 2001
From: Dodji Seketeli <dodji@redhat.com>
Date: Sat, 7 Nov 2015 23:04:01 +0100
Subject: [PATCH] Pass a bunch of parameters by reference as they ought to be

	* include/abg-ir.h (operator==): In the overload for
	elf_symbol_sptr, pass the parameters by reference.
	* src/abg-ir.cc (operator==): Do the same at definition site.
	* src/abg-comparison.cc (maybe_report_diff_for_member): Pass
	parameters by reference.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
---
 include/abg-ir.h      |  2 +-
 src/abg-comparison.cc | 20 ++++++++++----------
 src/abg-ir.cc         |  2 +-
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/include/abg-ir.h b/include/abg-ir.h
index 6d42c2ce..51e2ca5f 100644
--- a/include/abg-ir.h
+++ b/include/abg-ir.h
@@ -646,7 +646,7 @@ bool
 string_to_elf_symbol_binding(const string&, elf_symbol::binding&);
 
 bool
-operator==(const elf_symbol_sptr lhs, const elf_symbol_sptr rhs);
+operator==(const elf_symbol_sptr& lhs, const elf_symbol_sptr& rhs);
 
 bool
 elf_symbols_alias(const elf_symbol& s1, const elf_symbol& s2);
diff --git a/src/abg-comparison.cc b/src/abg-comparison.cc
index 372a1c4a..86f27c83 100644
--- a/src/abg-comparison.cc
+++ b/src/abg-comparison.cc
@@ -6245,11 +6245,11 @@ get_pretty_representation(diff* d)
 }
 
 static bool
-maybe_report_diff_for_member(decl_base_sptr	decl1,
-			     decl_base_sptr	decl2,
-			     diff_context_sptr	ctxt,
-			     ostream&		out,
-			     const string&	indent);
+maybe_report_diff_for_member(const decl_base_sptr&	decl1,
+			     const decl_base_sptr&	decl2,
+			     const diff_context_sptr&	ctxt,
+			     ostream&			out,
+			     const string&		indent);
 
 /// Stream a string representation for a member function.
 ///
@@ -6899,11 +6899,11 @@ compute_diff(const var_decl_sptr	first,
 ///
 /// @return true if something was reported, false otherwise.
 static bool
-maybe_report_diff_for_member(decl_base_sptr	decl1,
-			     decl_base_sptr	decl2,
-			     diff_context_sptr	ctxt,
-			     ostream&		out,
-			     const string&	indent)
+maybe_report_diff_for_member(const decl_base_sptr&	decl1,
+			     const decl_base_sptr&	decl2,
+			     const diff_context_sptr&	ctxt,
+			     ostream&			out,
+			     const string&		indent)
 
 {
   bool reported = false;
diff --git a/src/abg-ir.cc b/src/abg-ir.cc
index 9799cd57..faebf536 100644
--- a/src/abg-ir.cc
+++ b/src/abg-ir.cc
@@ -1315,7 +1315,7 @@ elf_symbol::does_alias(const elf_symbol& o) const
 }
 
 bool
-operator==(const elf_symbol_sptr lhs, const elf_symbol_sptr rhs)
+operator==(const elf_symbol_sptr& lhs, const elf_symbol_sptr& rhs)
 {
   if (!!lhs != !!rhs)
     return false;