clang-tidy performance-unnecessary-value-param in various

Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907
Reviewed-on: https://gerrit.libreoffice.org/24019
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>

1 files changed, 3 insertions, 3 deletions

diff --git a/basegfx/source/range/b2drangeclipper.cxx b/basegfx/source/range/b2drangeclipper.cxx
index 882b4a88cfc2..46f9e490e37c 100644
--- a/basegfx/source/range/b2drangeclipper.cxx
+++ b/basegfx/source/range/b2drangeclipper.cxx
@@ -706,14 +706,14 @@ namespace basegfx
 
         // wow what a hack. necessary because stl's list::erase does
         // not eat reverse_iterator
-        template<typename Cont, typename Iter> Iter eraseFromList(Cont&, Iter);
+        template<typename Cont, typename Iter> Iter eraseFromList(Cont&, const Iter&);
         template<> inline ListOfEdges::iterator eraseFromList(
-            ListOfEdges& rList, ListOfEdges::iterator aIter)
+            ListOfEdges& rList, const ListOfEdges::iterator& aIter)
         {
             return rList.erase(aIter);
         }
         template<> inline ListOfEdges::reverse_iterator eraseFromList(
-            ListOfEdges& rList, ListOfEdges::reverse_iterator aIter)
+            ListOfEdges& rList, const ListOfEdges::reverse_iterator& aIter)
         {
             return ListOfEdges::reverse_iterator(
                     rList.erase(boost::prior(aIter.base())));