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

(cherry picked from commit e9cded40c8727cec7d0d29219c14ef0d0eef5195)

Change-Id: I3e8cd7cedb3e7b7ef05760d21b10994ce615324b

5 files changed, 9 insertions, 9 deletions

diff --git a/include/svx/PaletteManager.hxx b/include/svx/PaletteManager.hxx
index 561dcac43d93..2d4c528a1984 100644
--- a/include/svx/PaletteManager.hxx
+++ b/include/svx/PaletteManager.hxx
@@ -72,7 +72,7 @@ public:
     void        SetBtnUpdater(svx::ToolboxButtonColorUpdater* pBtnUpdater);
     void        PopupColorPicker(const OUString& aCommand);
 
-    void        SetColorSelectFunction(std::function<void(const OUString&, const Color&)> aColorSelectFunction);
+    void        SetColorSelectFunction(const std::function<void(const OUString&, const Color&)>& aColorSelectFunction);
 
     static void DispatchColorCommand(const OUString& aCommand, const Color& rColor);
 };
diff --git a/include/svx/SmartTagItem.hxx b/include/svx/SmartTagItem.hxx
index 63292ff50972..0cc025db9aca 100644
--- a/include/svx/SmartTagItem.hxx
+++ b/include/svx/SmartTagItem.hxx
@@ -57,9 +57,9 @@ public:
                      const css::uno::Sequence < css::uno::Sequence< css::uno::Reference< css::smarttags::XSmartTagAction > > >& rActionComponentsSequence,
                      const css::uno::Sequence < css::uno::Sequence< sal_Int32 > >& rActionIndicesSequence,
                      const css::uno::Sequence< css::uno::Reference< css::container::XStringKeyMap > >& rStringKeyMaps,
-                     const css::uno::Reference<css::text::XTextRange> rRange,
-                     const css::uno::Reference<css::frame::XController> rController,
-                     const css::lang::Locale rLocale,
+                     const css::uno::Reference<css::text::XTextRange>& rRange,
+                     const css::uno::Reference<css::frame::XController>& rController,
+                     const css::lang::Locale& rLocale,
                      const OUString& rApplicationName,
                      const OUString& rRangeText );
 
diff --git a/include/svx/svdmodel.hxx b/include/svx/svdmodel.hxx
index edc754af08df..02b9057b06f0 100644
--- a/include/svx/svdmodel.hxx
+++ b/include/svx/svdmodel.hxx
@@ -565,14 +565,14 @@ public:
     void SetDisableTextEditUsesCommonUndoManager(bool bNew) { mbDisableTextEditUsesCommonUndoManager = bNew; }
 
     css::uno::Reference< css::uno::XInterface > getUnoModel();
-    void setUnoModel( css::uno::Reference< css::uno::XInterface > xModel );
+    void setUnoModel( const css::uno::Reference< css::uno::XInterface >& xModel );
 
     // these functions are used by the api to disable repaints during a
     // set of api calls.
     bool isLocked() const { return mbModelLocked; }
     void setLock( bool bLock );
 
-    void            SetForbiddenCharsTable( rtl::Reference<SvxForbiddenCharactersTable> xForbiddenChars );
+    void            SetForbiddenCharsTable( const rtl::Reference<SvxForbiddenCharactersTable>& xForbiddenChars );
     rtl::Reference<SvxForbiddenCharactersTable> GetForbiddenCharsTable() const { return mpForbiddenCharactersTable;}
 
     void SetCharCompressType( sal_uInt16 nType );
diff --git a/include/svx/tbcontrl.hxx b/include/svx/tbcontrl.hxx
index 179e8e7316d3..04cf60808c5b 100644
--- a/include/svx/tbcontrl.hxx
+++ b/include/svx/tbcontrl.hxx
@@ -251,7 +251,7 @@ public:
     virtual sal_Bool SAL_CALL opensSubToolbar() throw (css::uno::RuntimeException, std::exception) override;
     virtual void SAL_CALL updateImage() throw (css::uno::RuntimeException, std::exception) override;
 
-    void setColorSelectFunction(ColorSelectFunction aColorSelectFunction);
+    void setColorSelectFunction(const ColorSelectFunction& aColorSelectFunction);
 };
 
 class SVX_DLLPUBLIC SvxFrameToolBoxControl : public SfxToolBoxControl
diff --git a/include/svx/unoapi.hxx b/include/svx/unoapi.hxx
index 022350894a41..5d4a40d45a58 100644
--- a/include/svx/unoapi.hxx
+++ b/include/svx/unoapi.hxx
@@ -46,13 +46,13 @@ SVX_DLLPUBLIC SvxShape* CreateSvxShapeByTypeAndInventor(sal_uInt16 nType, sal_uI
 SVX_DLLPUBLIC css::uno::Reference< css::drawing::XShape > GetXShapeForSdrObject( SdrObject* pObj ) throw ();
 
 /** Returns the SdrObject from the given StarOffice API wrapper */
-SVX_DLLPUBLIC SdrObject* GetSdrObjectFromXShape( css::uno::Reference< css::drawing::XShape > xShape ) throw() ;
+SVX_DLLPUBLIC SdrObject* GetSdrObjectFromXShape( const css::uno::Reference< css::drawing::XShape >& xShape ) throw() ;
 
 /** Returns a StarOffice API wrapper for the given SdrPage */
 SVX_DLLPUBLIC css::uno::Reference< css::drawing::XDrawPage > GetXDrawPageForSdrPage( SdrPage* pPage ) throw ();
 
 /** Returns the SdrPage from the given StarOffice API wrapper */
-SVX_DLLPUBLIC SdrPage* GetSdrPageFromXDrawPage( css::uno::Reference< css::drawing::XDrawPage > xDrawPage ) throw() ;
+SVX_DLLPUBLIC SdrPage* GetSdrPageFromXDrawPage( const css::uno::Reference< css::drawing::XDrawPage >& xDrawPage ) throw() ;
 
 /**
  * Maps the vcl MapUnit enum to a API constant MeasureUnit.