improve passstuffbyref return analysis

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

7 files changed, 22 insertions, 22 deletions

diff --git a/sw/inc/authfld.hxx b/sw/inc/authfld.hxx
index 219335120d9b..6cf7a2093d3c 100644
--- a/sw/inc/authfld.hxx
+++ b/sw/inc/authfld.hxx
@@ -36,7 +36,7 @@ public:
     SwAuthEntry( const SwAuthEntry& rCopy );
     bool            operator==(const SwAuthEntry& rComp);
 
-    inline OUString         GetAuthorField(ToxAuthorityField ePos) const;
+    inline OUString const & GetAuthorField(ToxAuthorityField ePos) const;
     inline void             SetAuthorField(ToxAuthorityField ePos,
                                             const OUString& rField);
 
@@ -175,7 +175,7 @@ public:
     virtual OUString GetDescription() const override;
 };
 
-inline OUString SwAuthEntry::GetAuthorField(ToxAuthorityField ePos) const
+inline OUString const & SwAuthEntry::GetAuthorField(ToxAuthorityField ePos) const
 {
     SAL_WARN_IF(AUTH_FIELD_END <= ePos, "sw", "wrong index");
     return aAuthFields[ePos];
diff --git a/sw/inc/breakit.hxx b/sw/inc/breakit.hxx
index 47615f45bc0f..2978f4472ef0 100644
--- a/sw/inc/breakit.hxx
+++ b/sw/inc/breakit.hxx
@@ -63,7 +63,7 @@ public:
 public:
     static SwBreakIt * Get();
 
-    css::uno::Reference< css::i18n::XBreakIterator > GetBreakIter()
+    css::uno::Reference< css::i18n::XBreakIterator > const & GetBreakIter()
     {
         createBreakIterator();
         return xBreak;
diff --git a/sw/inc/dbmgr.hxx b/sw/inc/dbmgr.hxx
index 222e55c07df5..c14f20a2ed96 100644
--- a/sw/inc/dbmgr.hxx
+++ b/sw/inc/dbmgr.hxx
@@ -344,7 +344,7 @@ public:
                             OUString& rResult, double* pNumber);
     /** create and store or find an already stored connection to a data source for use
     in SwFieldMgr and SwDBTreeList */
-    css::uno::Reference< css::sdbc::XConnection>
+    css::uno::Reference< css::sdbc::XConnection> const &
                     RegisterConnection(OUString const& rSource);
 
     void            CreateDSData(const SwDBData& rData)
diff --git a/sw/inc/doc.hxx b/sw/inc/doc.hxx
index cfaefece1ed1..dc96eb0af54d 100644
--- a/sw/inc/doc.hxx
+++ b/sw/inc/doc.hxx
@@ -1606,7 +1606,7 @@ public:
     const css::uno::Reference<css::container::XNameContainer>&
         getXForms() const { return mxXForms;}
 
-    css::uno::Reference< css::linguistic2::XProofreadingIterator > GetGCIterator() const;
+    css::uno::Reference< css::linguistic2::XProofreadingIterator > const & GetGCIterator() const;
 
     // #i31958# is this an XForms document?
     bool isXForms() const;
@@ -1621,7 +1621,7 @@ public:
     void SetDefaultPageMode(bool bSquaredPageMode);
     bool IsSquaredPageMode() const;
 
-    css::uno::Reference< css::script::vba::XVBAEventProcessor > GetVbaEventProcessor();
+    css::uno::Reference< css::script::vba::XVBAEventProcessor > const & GetVbaEventProcessor();
     void SetVBATemplateToProjectCache( css::uno::Reference< css::container::XNameContainer >& xCache ) { m_xTemplateToProjectCache = xCache; };
     const css::uno::Reference< css::container::XNameContainer >& GetVBATemplateToProjectCache() { return m_xTemplateToProjectCache; };
     ::sfx2::IXmlIdRegistry& GetXmlIdRegistry();
diff --git a/sw/inc/ndole.hxx b/sw/inc/ndole.hxx
index 372b053e315e..6dc1b481ce9e 100644
--- a/sw/inc/ndole.hxx
+++ b/sw/inc/ndole.hxx
@@ -71,7 +71,7 @@ public:
 
     // try to get OLE visualization in form of a Primitive2DSequence
     // and the corresponding B2DRange. This data may be locally buffered
-    drawinglayer::primitive2d::Primitive2DContainer tryToGetChartContentAsPrimitive2DSequence(
+    drawinglayer::primitive2d::Primitive2DContainer const & tryToGetChartContentAsPrimitive2DSequence(
         basegfx::B2DRange& rRange,
         bool bSynchron);
     void resetBufferedData();
diff --git a/sw/inc/swmodule.hxx b/sw/inc/swmodule.hxx
index 2d088022fe1e..c749f9e3d9cc 100644
--- a/sw/inc/swmodule.hxx
+++ b/sw/inc/swmodule.hxx
@@ -232,10 +232,10 @@ public:
             GetLngSvcEvtListener();
     void    CreateLngSvcEvtListener();
 
-    css::uno::Reference< css::scanner::XScannerManager2 >
+    css::uno::Reference< css::scanner::XScannerManager2 > const &
             GetScannerManager();
 
-    css::uno::Reference< css::linguistic2::XLanguageGuessing >
+    css::uno::Reference< css::linguistic2::XLanguageGuessing > const &
             GetLanguageGuesser();
 };
 
diff --git a/sw/inc/tox.hxx b/sw/inc/tox.hxx
index 91d19beb7b58..63cf28621892 100644
--- a/sw/inc/tox.hxx
+++ b/sw/inc/tox.hxx
@@ -120,11 +120,11 @@ public:
     inline void             SetSecondaryKeyReading(const OUString& rStr);
     inline void             SetCitationKeyReading(const OUString& rStr);
 
-    inline OUString         GetPrimaryKey() const;
-    inline OUString         GetSecondaryKey() const;
-    inline OUString         GetTextReading() const;
-    inline OUString         GetPrimaryKeyReading() const;
-    inline OUString         GetSecondaryKeyReading() const;
+    inline OUString const & GetPrimaryKey() const;
+    inline OUString const & GetSecondaryKey() const;
+    inline OUString const & GetTextReading() const;
+    inline OUString const & GetPrimaryKeyReading() const;
+    inline OUString const & GetSecondaryKeyReading() const;
 
     bool                    IsAutoGenerated() const {return m_bAutoGenerated;}
     void                    SetAutoGenerated(bool bSet) {m_bAutoGenerated = bSet;}
@@ -292,7 +292,7 @@ public:
     SwForm& operator=( const SwForm& rForm );
 
     inline void SetTemplate(sal_uInt16 nLevel, const OUString& rName);
-    inline OUString GetTemplate(sal_uInt16 nLevel) const;
+    inline OUString const & GetTemplate(sal_uInt16 nLevel) const;
 
     // #i21237#
     void    SetPattern(sal_uInt16 nLevel, const SwFormTokens& rName);
@@ -472,7 +472,7 @@ public:
 
     // index of objects
 
-    OUString                GetStyleNames(sal_uInt16 nLevel) const
+    OUString const &        GetStyleNames(sal_uInt16 nLevel) const
                                 {
                                 SAL_WARN_IF( nLevel >= MAXLEVEL, "sw", "Which level?");
                                 return m_aStyleNames[nLevel];
@@ -599,31 +599,31 @@ inline sal_uInt16 SwTOXMark::GetLevel() const
     return m_nLevel;
 }
 
-inline OUString SwTOXMark::GetPrimaryKey() const
+inline OUString const & SwTOXMark::GetPrimaryKey() const
 {
     SAL_WARN_IF(GetTOXType()->GetType() != TOX_INDEX, "sw", "Wrong type");
     return m_aPrimaryKey;
 }
 
-inline OUString SwTOXMark::GetSecondaryKey() const
+inline OUString const & SwTOXMark::GetSecondaryKey() const
 {
     SAL_WARN_IF(GetTOXType()->GetType() != TOX_INDEX, "sw", "Wrong type");
     return m_aSecondaryKey;
 }
 
-inline OUString SwTOXMark::GetTextReading() const
+inline OUString const & SwTOXMark::GetTextReading() const
 {
     SAL_WARN_IF(GetTOXType()->GetType() != TOX_INDEX, "sw", "Wrong type");
     return m_aTextReading;
 }
 
-inline OUString SwTOXMark::GetPrimaryKeyReading() const
+inline OUString const & SwTOXMark::GetPrimaryKeyReading() const
 {
     SAL_WARN_IF(GetTOXType()->GetType() != TOX_INDEX, "sw", "Wrong type");
     return m_aPrimaryKeyReading;
 }
 
-inline OUString SwTOXMark::GetSecondaryKeyReading() const
+inline OUString const & SwTOXMark::GetSecondaryKeyReading() const
 {
     SAL_WARN_IF(GetTOXType()->GetType() != TOX_INDEX, "sw", "Wrong type");
     return m_aSecondaryKeyReading;
@@ -637,7 +637,7 @@ inline void SwForm::SetTemplate(sal_uInt16 nLevel, const OUString& rTemplate)
     m_aTemplate[nLevel] = rTemplate;
 }
 
-inline OUString SwForm::GetTemplate(sal_uInt16 nLevel) const
+inline OUString const & SwForm::GetTemplate(sal_uInt16 nLevel) const
 {
     SAL_WARN_IF(nLevel >= GetFormMax(), "sw", "Index >= GetFormMax()");
     return m_aTemplate[nLevel];