loplugin:passstuffbyref in cui

Change-Id: I13c2956ad5012373a41c8e6c2af70583fa18c339

14 files changed, 35 insertions, 35 deletions

diff --git a/cui/source/customize/macropg_impl.hxx b/cui/source/customize/macropg_impl.hxx
index efb53e6a26bc..514ba9cbd432 100644
--- a/cui/source/customize/macropg_impl.hxx
+++ b/cui/source/customize/macropg_impl.hxx
@@ -52,7 +52,7 @@ public:
     virtual ~AssignComponentDialog();
     virtual void dispose() override;
 
-    OUString getURL() const
+    const OUString& getURL() const
         { return maURL; }
 };
 
diff --git a/cui/source/dialogs/SpellDialog.cxx b/cui/source/dialogs/SpellDialog.cxx
index 75a14c890488..21d83096c10e 100644
--- a/cui/source/dialogs/SpellDialog.cxx
+++ b/cui/source/dialogs/SpellDialog.cxx
@@ -145,7 +145,7 @@ public:
     bool                    IsErrorLanguageSelected() const {return m_bIsErrorLanguageSelected;}
 
     void                    SetDictionary(const Reference<XDictionary>& xDict) { m_xDictionary = xDict; }
-    Reference<XDictionary>  GetDictionary() const {return m_xDictionary;}
+    const Reference<XDictionary>& GetDictionary() const { return m_xDictionary; }
     void                    SetAddedWord(const OUString& rWord) {m_sAddedWord = rWord;}
     const OUString&         GetAddedWord() const { return m_sAddedWord;}
 
diff --git a/cui/source/inc/cfg.hxx b/cui/source/inc/cfg.hxx
index acbdc908a90d..9ca82b462158 100644
--- a/cui/source/inc/cfg.hxx
+++ b/cui/source/inc/cfg.hxx
@@ -125,20 +125,20 @@ public:
     bool IsReadOnly( ) { return bReadOnly; }
     bool IsDocConfig( ) { return bDocConfig; }
 
-    css::uno::Reference
-        < css::ui::XUIConfigurationManager >
+    const css::uno::Reference
+        < css::ui::XUIConfigurationManager >&
             GetConfigManager() { return m_xCfgMgr; };
 
-    css::uno::Reference
-        < css::ui::XUIConfigurationManager >
+    const css::uno::Reference
+        < css::ui::XUIConfigurationManager >&
             GetParentConfigManager() { return m_xParentCfgMgr; };
 
-    css::uno::Reference
-        < css::ui::XImageManager >
+    const css::uno::Reference
+        < css::ui::XImageManager >&
             GetImageManager() { return m_xImgMgr; };
 
-    css::uno::Reference
-        < css::ui::XImageManager >
+    const css::uno::Reference
+        < css::ui::XImageManager >&
             GetParentImageManager() { return m_xParentImgMgr; };
 
     css::uno::Reference
@@ -307,7 +307,7 @@ public:
     void    SetBackupGraphic( css::uno::Reference< css::graphic::XGraphic > graphic )
                 { xBackupGraphic = graphic; }
 
-    css::uno::Reference< css::graphic::XGraphic > GetBackupGraphic()
+    const css::uno::Reference< css::graphic::XGraphic >& GetBackupGraphic()
                 { return xBackupGraphic; }
 
     bool    IsIconModified() { return xBackupGraphic.is(); }
diff --git a/cui/source/inc/cuitabarea.hxx b/cui/source/inc/cuitabarea.hxx
index 575d290095f6..dfffa39f13c6 100644
--- a/cui/source/inc/cuitabarea.hxx
+++ b/cui/source/inc/cuitabarea.hxx
@@ -79,22 +79,22 @@ public:
 
     void                SetNewColorList( XColorListRef pColorList )
                             { mpNewColorList = pColorList; }
-    XColorListRef         GetNewColorList() const { return mpNewColorList; }
-    const XColorListRef   GetColorList() const { return mpColorList; }
+    const XColorListRef&  GetNewColorList() const { return mpNewColorList; }
+    const XColorListRef&  GetColorList() const { return mpColorList; }
 
-    void                SetNewGradientList( XGradientListRef pGrdLst)
+    void                  SetNewGradientList( XGradientListRef pGrdLst)
                             { mpNewGradientList = pGrdLst; }
-    XGradientListRef       GetNewGradientList() const
+    const XGradientListRef&  GetNewGradientList() const
                             { return mpNewGradientList; }
 
-    void                 SetNewHatchingList( XHatchListRef pHtchLst)
+    void                  SetNewHatchingList( XHatchListRef pHtchLst)
                             { mpNewHatchingList = pHtchLst; }
-    XHatchListRef          GetNewHatchingList() const
+    const XHatchListRef&  GetNewHatchingList() const
                             { return mpNewHatchingList; }
 
-    void                 SetNewBitmapList( XBitmapListRef pBmpLst)
+    void                  SetNewBitmapList( XBitmapListRef pBmpLst)
                             { mpNewBitmapList = pBmpLst; }
-    XBitmapListRef         GetNewBitmapList() const { return mpNewBitmapList; }
+    const XBitmapListRef& GetNewBitmapList() const { return mpNewBitmapList; }
 };
 
 /************************************************************************/
@@ -755,7 +755,7 @@ public:
     void             SetPropertyList( XPropertyListType t, const XPropertyListRef &xRef );
 
     void    SetColorList( const XColorListRef& pColList );
-    XColorListRef GetColorList() { return pColorList; }
+    const XColorListRef& GetColorList() { return pColorList; }
     void    SaveToViewFrame( SfxViewFrame *pViewFrame );
     void    SetupForViewFrame( SfxViewFrame *pViewFrame );
 
diff --git a/cui/source/inc/cuitabline.hxx b/cui/source/inc/cuitabline.hxx
index 00b4ce502324..d51780e98fe1 100644
--- a/cui/source/inc/cuitabline.hxx
+++ b/cui/source/inc/cuitabline.hxx
@@ -68,15 +68,15 @@ public:
 
     void                SetNewDashList( XDashListRef pInLst)
                         { pNewDashList = pInLst; }
-    XDashListRef        GetNewDashList() const { return pNewDashList; }
+    const XDashListRef& GetNewDashList() const { return pNewDashList; }
 
     void                SetNewLineEndList( XLineEndListRef pInLst)
                         { pNewLineEndList = pInLst; }
-    XLineEndListRef     GetNewLineEndList() const { return pNewLineEndList; }
+    const XLineEndListRef& GetNewLineEndList() const { return pNewLineEndList; }
 
     void                SetNewColorList( XColorListRef pColTab ) { mpNewColorList = pColTab; }
-    XColorListRef       GetNewColorList() const { return mpNewColorList; }
-    XColorListRef       GetColorList() const { return pColorList; }
+    const XColorListRef& GetNewColorList() const { return mpNewColorList; }
+    const XColorListRef& GetColorList() const { return pColorList; }
 };
 
 /*************************************************************************/
diff --git a/cui/source/inc/iconcdlg.hxx b/cui/source/inc/iconcdlg.hxx
index d97b03bc7e5b..131e4d001353 100644
--- a/cui/source/inc/iconcdlg.hxx
+++ b/cui/source/inc/iconcdlg.hxx
@@ -102,7 +102,7 @@ public:
 
     virtual void        ActivatePage( const SfxItemSet& );
     virtual int         DeactivatePage( SfxItemSet* pSet );
-    OUString            GetUserData() { return aUserString; }
+    const OUString&     GetUserData() { return aUserString; }
     virtual bool        QueryClose();
 
     void                StateChanged( StateChangedType nType ) override;
diff --git a/cui/source/inc/insdlg.hxx b/cui/source/inc/insdlg.hxx
index 67b51a02c9b3..c4624a6f70be 100644
--- a/cui/source/inc/insdlg.hxx
+++ b/cui/source/inc/insdlg.hxx
@@ -48,7 +48,7 @@ protected:
         const OUString& rUIXMLDescription,
         const css::uno::Reference < css::embed::XStorage >& xStorage);
 public:
-    css::uno::Reference < css::embed::XEmbeddedObject > GetObject()
+    const css::uno::Reference < css::embed::XEmbeddedObject >& GetObject()
                         { return m_xObj; }
     virtual css::uno::Reference< css::io::XInputStream > GetIconIfIconified( OUString* pGraphicMediaType );
     virtual bool IsCreateNew() const;
diff --git a/cui/source/inc/optdict.hxx b/cui/source/inc/optdict.hxx
index 5f97eb9e8fe9..ad46c19a9b32 100644
--- a/cui/source/inc/optdict.hxx
+++ b/cui/source/inc/optdict.hxx
@@ -65,8 +65,8 @@ public:
     virtual ~SvxNewDictionaryDialog();
     virtual void dispose() override;
 
-    css::uno::Reference<
-        css::linguistic2::XDictionary >
+    const css::uno::Reference<
+        css::linguistic2::XDictionary >&
                 GetNewDictionary() { return xNewDic; }
 };
 
diff --git a/cui/source/inc/scriptdlg.hxx b/cui/source/inc/scriptdlg.hxx
index 53c92b05ef3a..54b659e9b81b 100644
--- a/cui/source/inc/scriptdlg.hxx
+++ b/cui/source/inc/scriptdlg.hxx
@@ -125,8 +125,8 @@ public:
                             const css::uno::Reference< css::frame::XModel >& entryModel) { nType = nT; nodes = entryNodes; loaded=false; model = entryModel; }
                     SFEntry( const SFEntry& r ) { nType = r.nType; nodes = r.nodes; loaded = r.loaded; }
     virtual         ~SFEntry() {}
-    css::uno::Reference< css::script::browse::XBrowseNode > GetNode() { return nodes ;}
-    css::uno::Reference< css::frame::XModel > GetModel() { return model ;};
+    const css::uno::Reference< css::script::browse::XBrowseNode >& GetNode() { return nodes ;}
+    const css::uno::Reference< css::frame::XModel >& GetModel() { return model ;};
     bool            isLoaded() const                    { return loaded; }
     void            setLoaded()                         { loaded=true; }
 };
diff --git a/cui/source/inc/thesdlg.hxx b/cui/source/inc/thesdlg.hxx
index 82f2332ec8ef..925ccf9c8675 100644
--- a/cui/source/inc/thesdlg.hxx
+++ b/cui/source/inc/thesdlg.hxx
@@ -139,7 +139,7 @@ public:
     virtual void dispose() override;
 
     bool                    WordFound() const { return m_bWordFound; }
-    OUString                getErrStr() const { return m_aErrStr; }
+    const OUString&         getErrStr() const { return m_aErrStr; }
 
     // Handler
     DECL_LINK_TYPED( ReplaceBtnHdl_Impl, Button *, void );
diff --git a/cui/source/options/cfgchart.hxx b/cui/source/options/cfgchart.hxx
index 401485f57c6c..3b8b0be87553 100644
--- a/cui/source/options/cfgchart.hxx
+++ b/cui/source/options/cfgchart.hxx
@@ -67,7 +67,7 @@ private:
     css::uno::Sequence< OUString >
                                 maPropertyNames;
 
-    inline css::uno::Sequence< OUString > GetPropertyNames() const
+    const css::uno::Sequence< OUString >& GetPropertyNames() const
         { return maPropertyNames; }
     bool RetrieveOptions();
 
diff --git a/cui/source/options/personalization.cxx b/cui/source/options/personalization.cxx
index bd97e912b2ff..2e990cf77436 100644
--- a/cui/source/options/personalization.cxx
+++ b/cui/source/options/personalization.cxx
@@ -202,7 +202,7 @@ void SelectPersonaDialog::SetAppliedPersonaSetting( OUString& rPersonaSetting )
     m_aAppliedPersona = rPersonaSetting;
 }
 
-OUString SelectPersonaDialog::GetAppliedPersonaSetting() const
+const OUString& SelectPersonaDialog::GetAppliedPersonaSetting() const
 {
     return m_aAppliedPersona;
 }
diff --git a/cui/source/options/personalization.hxx b/cui/source/options/personalization.hxx
index 362fa6cdc5d1..7bef5a6c6ac1 100644
--- a/cui/source/options/personalization.hxx
+++ b/cui/source/options/personalization.hxx
@@ -102,7 +102,7 @@ public:
     void AddPersonaSetting( OUString& );
     void ClearSearchResults();
     void SetAppliedPersonaSetting( OUString& );
-    OUString GetAppliedPersonaSetting() const;
+    const OUString& GetAppliedPersonaSetting() const;
 
 private:
     /// Handle the Search button
diff --git a/cui/source/options/personasdochandler.hxx b/cui/source/options/personasdochandler.hxx
index c54616b271bd..7875be68d4f3 100644
--- a/cui/source/options/personasdochandler.hxx
+++ b/cui/source/options/personasdochandler.hxx
@@ -22,7 +22,7 @@ private:
     bool m_isLearnmoreTag, m_hasResults;
 public:
     PersonasDocHandler(){ m_isLearnmoreTag = false; m_hasResults = false; }
-    std::vector<OUString> getLearnmoreURLs() { return m_vLearnmoreURLs; }
+    const std::vector<OUString>& getLearnmoreURLs() { return m_vLearnmoreURLs; }
     bool hasResults() { return m_hasResults; }
 
     // XDocumentHandler