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>

5 files changed, 7 insertions, 7 deletions

diff --git a/ucb/source/core/providermap.hxx b/ucb/source/core/providermap.hxx
index b7a242aca0f2..ba793aad9e73 100644
--- a/ucb/source/core/providermap.hxx
+++ b/ucb/source/core/providermap.hxx
@@ -37,7 +37,7 @@ class ProviderListEntry_Impl
         css::ucb::XContentProvider > m_xResolvedProvider;
 
 private:
-    css::uno::Reference< css::ucb::XContentProvider > resolveProvider() const;
+    css::uno::Reference< css::ucb::XContentProvider > const & resolveProvider() const;
 
 public:
     explicit ProviderListEntry_Impl(
diff --git a/ucb/source/core/ucb.cxx b/ucb/source/core/ucb.cxx
index 09437c20576d..5f9f16b346be 100644
--- a/ucb/source/core/ucb.cxx
+++ b/ucb/source/core/ucb.cxx
@@ -941,7 +941,7 @@ bool UniversalContentBroker::getContentProviderData(
 // ProviderListEntry_Impl implementation.
 
 
-Reference< XContentProvider > ProviderListEntry_Impl::resolveProvider() const
+Reference< XContentProvider > const & ProviderListEntry_Impl::resolveProvider() const
 {
     if ( !m_xResolvedProvider.is() )
     {
diff --git a/ucb/source/ucp/cmis/cmis_content.cxx b/ucb/source/ucp/cmis/cmis_content.cxx
index 6180c6d7d30e..5dd9951402f1 100644
--- a/ucb/source/ucp/cmis/cmis_content.cxx
+++ b/ucb/source/ucp/cmis/cmis_content.cxx
@@ -446,7 +446,7 @@ namespace cmis
         return m_pSession;
     }
 
-    libcmis::ObjectTypePtr Content::getObjectType( const uno::Reference< ucb::XCommandEnvironment >& xEnv )
+    libcmis::ObjectTypePtr const & Content::getObjectType( const uno::Reference< ucb::XCommandEnvironment >& xEnv )
     {
         if ( nullptr == m_pObjectType.get( ) && m_bTransient )
         {
@@ -496,7 +496,7 @@ namespace cmis
     }
 
 
-    libcmis::ObjectPtr Content::getObject( const uno::Reference< ucb::XCommandEnvironment >& xEnv ) throw (css::uno::RuntimeException, css::ucb::CommandFailedException, libcmis::Exception)
+    libcmis::ObjectPtr const & Content::getObject( const uno::Reference< ucb::XCommandEnvironment >& xEnv ) throw (css::uno::RuntimeException, css::ucb::CommandFailedException, libcmis::Exception)
     {
         // can't get the session for some reason
         // the recent file opening at start up is an example.
diff --git a/ucb/source/ucp/cmis/cmis_content.hxx b/ucb/source/ucp/cmis/cmis_content.hxx
index 0252ed59de29..9e6a0516995b 100644
--- a/ucb/source/ucp/cmis/cmis_content.hxx
+++ b/ucb/source/ucp/cmis/cmis_content.hxx
@@ -80,7 +80,7 @@ private:
             const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv );
 
     libcmis::Session* getSession( const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv );
-    libcmis::ObjectTypePtr getObjectType( const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv );
+    libcmis::ObjectTypePtr const & getObjectType( const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv );
 
 private:
     typedef rtl::Reference< Content > ContentRef;
@@ -199,7 +199,7 @@ public:
 
     virtual std::list< css::uno::Reference< css::ucb::XContent > > getChildren( ) override;
 
-    libcmis::ObjectPtr getObject( const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv ) throw (css::uno::RuntimeException, css::ucb::CommandFailedException, libcmis::Exception);
+    libcmis::ObjectPtr const & getObject( const css::uno::Reference< css::ucb::XCommandEnvironment >& xEnv ) throw (css::uno::RuntimeException, css::ucb::CommandFailedException, libcmis::Exception);
 };
 
 }
diff --git a/ucb/source/ucp/file/filtask.hxx b/ucb/source/ucp/file/filtask.hxx
index f2b0d89eab0c..24f78bb7cb93 100644
--- a/ucb/source/ucp/file/filtask.hxx
+++ b/ucb/source/ucp/file/filtask.hxx
@@ -110,7 +110,7 @@ namespace fileaccess
                 return m_nMinorCode;
             }
 
-            css::uno::Reference< css::task::XInteractionHandler > SAL_CALL
+            css::uno::Reference< css::task::XInteractionHandler > const & SAL_CALL
             getInteractionHandler()
             {
                 if( ! m_xInteractionHandler.is() && m_xCommandEnvironment.is() )