loplugin:passstuffbyref in svtools

Change-Id: Ie166eaef65e56fafe4e57a5559b587d7558d7aa4

16 files changed, 25 insertions, 25 deletions

diff --git a/include/svtools/breadcrumb.hxx b/include/svtools/breadcrumb.hxx
index ac109ecbbfcd..6eff94445ff8 100644
--- a/include/svtools/breadcrumb.hxx
+++ b/include/svtools/breadcrumb.hxx
@@ -56,7 +56,7 @@ class SVT_DLLPUBLIC Breadcrumb : public VclHBox
         void EnableFields( bool bEnable );
 
         void SetClickHdl( const Link<Breadcrumb*,void>& rLink );
-        OUString GetHdlURL();
+        const OUString& GetHdlURL();
 
         void SetRootName( const OUString& rURL );
         void SetURL( const OUString& rURL );
diff --git a/include/svtools/calendar.hxx b/include/svtools/calendar.hxx
index 029afb2752cf..12e60cfedc82 100644
--- a/include/svtools/calendar.hxx
+++ b/include/svtools/calendar.hxx
@@ -276,7 +276,7 @@ public:
 
     void            SetCurDate( const Date& rNewDate );
     void            SetFirstDate( const Date& rNewFirstDate );
-    Date            GetFirstDate() const { return maFirstDate; }
+    const Date&     GetFirstDate() const { return maFirstDate; }
     Date            GetLastDate() const { return GetFirstDate() + mnDayCount; }
     Date            GetFirstMonth() const;
     Date            GetLastMonth() const;
diff --git a/include/svtools/ctrlbox.hxx b/include/svtools/ctrlbox.hxx
index acb68242f449..1e7cfadf7c84 100644
--- a/include/svtools/ctrlbox.hxx
+++ b/include/svtools/ctrlbox.hxx
@@ -335,7 +335,7 @@ public:
     inline void     SetSourceUnit( FieldUnit eNewUnit ) { eSourceUnit = eNewUnit; }
 
     void            SetColor( const Color& rColor );
-    Color           GetColor() const { return aColor; }
+    const Color&    GetColor() const { return aColor; }
 
 protected:
 
diff --git a/include/svtools/editbrowsebox.hxx b/include/svtools/editbrowsebox.hxx
index d28511e49250..497321e05989 100644
--- a/include/svtools/editbrowsebox.hxx
+++ b/include/svtools/editbrowsebox.hxx
@@ -600,7 +600,7 @@ namespace svt
         virtual void RemoveRows();
         virtual void Dispatch(sal_uInt16 nId);
 
-        CellControllerRef Controller() const { return aController; }
+        const CellControllerRef& Controller() const { return aController; }
         EditBrowseBoxFlags  GetBrowserFlags() const { return m_nBrowserFlags; }
         void                SetBrowserFlags(EditBrowseBoxFlags nFlags);
 
diff --git a/include/svtools/extcolorcfg.hxx b/include/svtools/extcolorcfg.hxx
index 8db65e563a9a..be0982870171 100644
--- a/include/svtools/extcolorcfg.hxx
+++ b/include/svtools/extcolorcfg.hxx
@@ -49,8 +49,8 @@ public:
     ,m_nDefaultColor(_nDefaultColor)
     {}
 
-    inline OUString        getName()         const { return m_sName; }
-    inline OUString        getDisplayName()  const { return m_sDisplayName; }
+    inline const OUString& getName()         const { return m_sName; }
+    inline const OUString& getDisplayName()  const { return m_sDisplayName; }
     inline sal_Int32       getColor()        const { return m_nColor; }
     inline sal_Int32       getDefaultColor() const { return m_nDefaultColor; }
 
diff --git a/include/svtools/fmtfield.hxx b/include/svtools/fmtfield.hxx
index ef04c004c44e..5cc8bd3f4230 100644
--- a/include/svtools/fmtfield.hxx
+++ b/include/svtools/fmtfield.hxx
@@ -184,7 +184,7 @@ public:
     OUString  GetTextValue() const;
 
     void      SetDefaultText(const OUString& rDefault) { m_sDefaultText = rDefault; }
-    OUString  GetDefaultText() const { return m_sDefaultText; }
+    const OUString& GetDefaultText() const { return m_sDefaultText; }
 
     // The last colour from the Formatter's last output operation. Output operations get triggered by:
     // SetValue, SetTextValue, SetTextFormatted, also indirectly via SetMin - / -MaxValue
@@ -285,7 +285,7 @@ class DoubleCurrencyField : public FormattedField
 public:
     DoubleCurrencyField(vcl::Window* pParent, WinBits nStyle = 0);
 
-    OUString    getCurrencySymbol() const { return m_sCurrencySymbol; }
+    const OUString& getCurrencySymbol() const { return m_sCurrencySymbol; }
     void        setCurrencySymbol(const OUString& rSymbol);
 
     bool        getPrependCurrSym() const { return m_bPrependCurrSym; }
diff --git a/include/svtools/grfmgr.hxx b/include/svtools/grfmgr.hxx
index f5e94b278be0..517aca59ef18 100644
--- a/include/svtools/grfmgr.hxx
+++ b/include/svtools/grfmgr.hxx
@@ -394,12 +394,12 @@ public:
     bool                    HasLink() const { return !maLink.isEmpty(); }
     void                    SetLink();
     void                    SetLink( const OUString& rLink );
-    OUString                GetLink() const { return maLink; }
+    const OUString&         GetLink() const { return maLink; }
 
     bool                    HasUserData() const { return !maUserData.isEmpty(); }
     void                    SetUserData();
     void                    SetUserData( const OUString& rUserData );
-    OUString                GetUserData() const { return maUserData; }
+    const OUString&         GetUserData() const { return maUserData; }
 
     OString                 GetUniqueID() const;
 
diff --git a/include/svtools/inettbc.hxx b/include/svtools/inettbc.hxx
index e6435f1a2ce6..1b39f8fb8720 100644
--- a/include/svtools/inettbc.hxx
+++ b/include/svtools/inettbc.hxx
@@ -87,7 +87,7 @@ public:
                                         { bIsAutoCompleteEnabled = _bEnable; }
     void                            SetPlaceHolder( const OUString& sPlaceHolder )
                                         { aPlaceHolder = sPlaceHolder; }
-    OUString                        GetPlaceHolder() { return aPlaceHolder; }
+    const OUString&                 GetPlaceHolder() { return aPlaceHolder; }
     bool                            MatchesPlaceHolder( const OUString& sToMatch ) const
                                         { return ( !aPlaceHolder.isEmpty() ) && ( aPlaceHolder == sToMatch ); }
 };
diff --git a/include/svtools/ivctrl.hxx b/include/svtools/ivctrl.hxx
index ec6e0e93750b..907197ca3043 100644
--- a/include/svtools/ivctrl.hxx
+++ b/include/svtools/ivctrl.hxx
@@ -121,12 +121,12 @@ public:
                             SvxIconChoiceCtrlEntry( const OUString& rText, const Image& rImage );
                             ~SvxIconChoiceCtrlEntry () {}
 
-    Image                   GetImage () const { return aImage; }
+    const Image&            GetImage () const { return aImage; }
     void                    SetText ( const OUString& rText ) { aText = rText; }
-    OUString                GetText () const { return aText; }
+    const OUString&         GetText () const { return aText; }
     OUString SVT_DLLPUBLIC  GetDisplayText() const;
     void                    SetQuickHelpText( const OUString& rText ) { aQuickHelpText = rText; }
-    OUString                GetQuickHelpText() const { return aQuickHelpText; }
+    const OUString&         GetQuickHelpText() const { return aQuickHelpText; }
     void                    SetUserData ( void* _pUserData ) { pUserData = _pUserData; }
     void*                   GetUserData () { return pUserData; }
 
diff --git a/include/svtools/ruler.hxx b/include/svtools/ruler.hxx
index 775f26dbc288..620f3f3f9dcb 100644
--- a/include/svtools/ruler.hxx
+++ b/include/svtools/ruler.hxx
@@ -694,8 +694,8 @@ private:
 
 protected:
     long            GetRulerVirHeight() const { return mnVirHeight;}
-    MapMode         GetCurrentMapMode() const { return maMapMode; }
-    RulerUnitData   GetCurrentRulerUnit() const;
+    const MapMode&  GetCurrentMapMode() const { return maMapMode; }
+    const RulerUnitData& GetCurrentRulerUnit() const;
 
 public:
             Ruler( vcl::Window* pParent, WinBits nWinStyle = WB_STDRULER );
@@ -726,7 +726,7 @@ public:
     long            GetPageOffset() const;
     void            SetBorderPos( long nOff = 0 );
     long            GetBorderOffset() const { return mnBorderOff; }
-    Rectangle       GetExtraRect() const { return maExtraRect; }
+    const Rectangle& GetExtraRect() const { return maExtraRect; }
 
     void            SetUnit( FieldUnit eNewUnit );
     FieldUnit       GetUnit() const { return meUnit; }
@@ -750,7 +750,7 @@ public:
     long            GetClickPos() const { return mnDragPos; }
     RulerType       GetClickType() const { return meDragType; }
 
-    RulerSelection  GetHoverSelection() const { return maHoverSelection; }
+    const RulerSelection& GetHoverSelection() const { return maHoverSelection; }
 
     using Window::GetType;
     RulerType       GetType( const Point& rPos, sal_uInt16* pAryPos = nullptr );
diff --git a/include/svtools/simptabl.hxx b/include/svtools/simptabl.hxx
index fa3fd950135a..55d21fd64741 100644
--- a/include/svtools/simptabl.hxx
+++ b/include/svtools/simptabl.hxx
@@ -114,7 +114,7 @@ public:
     sal_uInt16      GetSortedCol(){ return nSortCol;}
     SvLBoxItem*     GetEntryAtPos( SvTreeListEntry* pEntry, sal_uInt16 nPos ) const;
 
-    CommandEvent    GetCommandEvent() const { return aCEvt; }
+    const CommandEvent& GetCommandEvent() const { return aCEvt; }
     inline bool     IsFocusOnCellEnabled() const { return IsCellFocusEnabled(); }
     void            SetCommandHdl( const Link<SvSimpleTable*,void>& rLink ) { aCommandLink = rLink; }
 
diff --git a/include/svtools/svlbitm.hxx b/include/svtools/svlbitm.hxx
index 26bdb2123d9b..516e2311fdd5 100644
--- a/include/svtools/svlbitm.hxx
+++ b/include/svtools/svlbitm.hxx
@@ -115,7 +115,7 @@ public:
                               SvTreeListEntry* pEntry,
                               SvViewDataItem* pViewData) override;
 
-    OUString GetText() const
+    const OUString& GetText() const
     {
         return maText;
     }
diff --git a/include/svtools/toolboxcontroller.hxx b/include/svtools/toolboxcontroller.hxx
index 1f0f08100855..249ed1643951 100644
--- a/include/svtools/toolboxcontroller.hxx
+++ b/include/svtools/toolboxcontroller.hxx
@@ -162,9 +162,9 @@ class SVT_DLLPUBLIC ToolboxController :
         void unbindListener();
         bool isBound() const;
         // TODO remove
-        css::uno::Reference< css::util::XURLTransformer > getURLTransformer() const { return m_xUrlTransformer;}
+        const css::uno::Reference< css::util::XURLTransformer >& getURLTransformer() const { return m_xUrlTransformer;}
         // TODO remove
-        css::uno::Reference< css::awt::XWindow > getParent() const { return m_xParentWindow;}
+        const css::uno::Reference< css::awt::XWindow >& getParent() const { return m_xParentWindow;}
 
         bool                                                      m_bInitialized : 1,
                                                                   m_bDisposed : 1;
diff --git a/include/svtools/treelistbox.hxx b/include/svtools/treelistbox.hxx
index 77dbc23e9050..4ce6da90f46b 100644
--- a/include/svtools/treelistbox.hxx
+++ b/include/svtools/treelistbox.hxx
@@ -677,7 +677,7 @@ public:
     static const Image&    GetCollapsedEntryBmp(const SvTreeListEntry* _pEntry );
 
     void            SetCheckButtonHdl( const Link<SvTreeListBox*,void>& rLink )  { aCheckButtonHdl=rLink; }
-    Link<SvTreeListBox*,void>          GetCheckButtonHdl() const { return aCheckButtonHdl; }
+    const Link<SvTreeListBox*,void>& GetCheckButtonHdl() const { return aCheckButtonHdl; }
     virtual void    CheckButtonHdl();
 
     void            SetSublistOpenWithReturn();      // open/close sublist with return/enter
diff --git a/include/svtools/treelistentry.hxx b/include/svtools/treelistentry.hxx
index c5c523ce85b6..7467dbaf8914 100644
--- a/include/svtools/treelistentry.hxx
+++ b/include/svtools/treelistentry.hxx
@@ -113,7 +113,7 @@ public:
     void SetMarked( bool IsMarked ) { bIsMarked = IsMarked; }
 
     void SetBackColor( const Color& aColor ) { maBackColor = aColor; }
-    Color GetBackColor() const { return maBackColor; }
+    const Color& GetBackColor() const { return maBackColor; }
 };
 
 #endif
diff --git a/include/svtools/viewdataentry.hxx b/include/svtools/viewdataentry.hxx
index 18c4fe16d1b4..15acf855a882 100644
--- a/include/svtools/viewdataentry.hxx
+++ b/include/svtools/viewdataentry.hxx
@@ -77,7 +77,7 @@ public:
     SvViewDataItem& GetItem(size_t nPos);
 
     void SetPaintRectangle(Rectangle aRectangle);
-    Rectangle GetPaintRectangle() const;
+    const Rectangle& GetPaintRectangle() const;
 
 };