loplugin:passstuffbyref also for {css::uno,rtl}::Reference

Change-Id: Ie05f9427dcd88d3b64cfe02c2ece5c0cf40be547

16 files changed, 40 insertions, 40 deletions

diff --git a/package/inc/ByteChucker.hxx b/package/inc/ByteChucker.hxx
index c18e76e99ad6..a70aedcdfc8b 100644
--- a/package/inc/ByteChucker.hxx
+++ b/package/inc/ByteChucker.hxx
@@ -39,7 +39,7 @@ protected:
     sal_Int8 * const p1Sequence, * const p2Sequence, * const p4Sequence;
 
 public:
-    ByteChucker (css::uno::Reference<css::io::XOutputStream> xOstream);
+    ByteChucker (css::uno::Reference<css::io::XOutputStream> const & xOstream);
     ~ByteChucker();
 
     void WriteBytes( const css::uno::Sequence< sal_Int8 >& aData )
diff --git a/package/inc/ByteGrabber.hxx b/package/inc/ByteGrabber.hxx
index 0d522f1d08b4..bfdb8e35e9a7 100644
--- a/package/inc/ByteGrabber.hxx
+++ b/package/inc/ByteGrabber.hxx
@@ -43,7 +43,7 @@ protected:
     const sal_Int8 *pSequence;
 
 public:
-    ByteGrabber (css::uno::Reference < css::io::XInputStream > xIstream);
+    ByteGrabber (css::uno::Reference < css::io::XInputStream > const & xIstream);
     ~ByteGrabber();
 
     void setInputStream (const css::uno::Reference < css::io::XInputStream >& xNewStream);
diff --git a/package/source/manifest/ManifestExport.cxx b/package/source/manifest/ManifestExport.cxx
index 2c12b5f574d1..10b30fb5910e 100644
--- a/package/source/manifest/ManifestExport.cxx
+++ b/package/source/manifest/ManifestExport.cxx
@@ -42,7 +42,7 @@ using namespace ::com::sun::star;
 #define THROW_WHERE ""
 #endif
 
-ManifestExport::ManifestExport( uno::Reference< xml::sax::XDocumentHandler > xHandler,  const uno::Sequence< uno::Sequence < beans::PropertyValue > >& rManList )
+ManifestExport::ManifestExport( uno::Reference< xml::sax::XDocumentHandler > const & xHandler,  const uno::Sequence< uno::Sequence < beans::PropertyValue > >& rManList )
 {
     const OUString sFileEntryElement     ( ELEMENT_FILE_ENTRY );
     const OUString sManifestElement      ( ELEMENT_MANIFEST );
diff --git a/package/source/manifest/ManifestExport.hxx b/package/source/manifest/ManifestExport.hxx
index cb862f110f14..410a5fead470 100644
--- a/package/source/manifest/ManifestExport.hxx
+++ b/package/source/manifest/ManifestExport.hxx
@@ -31,7 +31,7 @@ namespace com { namespace sun { namespace star {
 class ManifestExport
 {
 public:
-    ManifestExport(css::uno::Reference < css::xml::sax::XDocumentHandler > xHandler, const css::uno::Sequence < css::uno::Sequence < css::beans::PropertyValue > > &rManList );
+    ManifestExport(css::uno::Reference < css::xml::sax::XDocumentHandler > const & xHandler, const css::uno::Sequence < css::uno::Sequence < css::beans::PropertyValue > > &rManList );
 };
 
 #endif
diff --git a/package/source/xstor/ocompinstream.cxx b/package/source/xstor/ocompinstream.cxx
index 1f0c61a35e04..24318b1b0c3e 100644
--- a/package/source/xstor/ocompinstream.cxx
+++ b/package/source/xstor/ocompinstream.cxx
@@ -29,7 +29,7 @@
 using namespace ::com::sun::star;
 
 OInputCompStream::OInputCompStream( OWriteStream_Impl& aImpl,
-                                    uno::Reference < io::XInputStream > xStream,
+                                    uno::Reference < io::XInputStream > const & xStream,
                                     const uno::Sequence< beans::PropertyValue >& aProps,
                                     sal_Int32 nStorageType )
 : m_pImpl( &aImpl )
@@ -47,7 +47,7 @@ OInputCompStream::OInputCompStream( OWriteStream_Impl& aImpl,
     assert(m_xStream.is());
 }
 
-OInputCompStream::OInputCompStream( uno::Reference < io::XInputStream > xStream,
+OInputCompStream::OInputCompStream( uno::Reference < io::XInputStream > const & xStream,
                                     const uno::Sequence< beans::PropertyValue >& aProps,
                                     sal_Int32 nStorageType )
 : m_pImpl( nullptr )
diff --git a/package/source/xstor/ocompinstream.hxx b/package/source/xstor/ocompinstream.hxx
index 7600f29aac55..451f2201fb5a 100644
--- a/package/source/xstor/ocompinstream.hxx
+++ b/package/source/xstor/ocompinstream.hxx
@@ -50,11 +50,11 @@ protected:
 
 public:
     OInputCompStream( OWriteStream_Impl& pImpl,
-                      css::uno::Reference< css::io::XInputStream > xStream,
+                      css::uno::Reference< css::io::XInputStream > const & xStream,
                       const css::uno::Sequence< css::beans::PropertyValue >& aProps,
                       sal_Int32 nStorageType );
 
-    OInputCompStream( css::uno::Reference< css::io::XInputStream > xStream,
+    OInputCompStream( css::uno::Reference< css::io::XInputStream > const & xStream,
                       const css::uno::Sequence< css::beans::PropertyValue >& aProps,
                       sal_Int32 nStorageType );
 
diff --git a/package/source/xstor/oseekinstream.cxx b/package/source/xstor/oseekinstream.cxx
index 074f08f5d4e7..95137f4d836b 100644
--- a/package/source/xstor/oseekinstream.cxx
+++ b/package/source/xstor/oseekinstream.cxx
@@ -28,7 +28,7 @@
 using namespace ::com::sun::star;
 
 OInputSeekStream::OInputSeekStream( OWriteStream_Impl& pImpl,
-                                    uno::Reference < io::XInputStream > xStream,
+                                    uno::Reference < io::XInputStream > const & xStream,
                                     const uno::Sequence< beans::PropertyValue >& aProps,
                                     sal_Int32 nStorageType )
 : OInputCompStream( pImpl, xStream, aProps, nStorageType )
@@ -37,7 +37,7 @@ OInputSeekStream::OInputSeekStream( OWriteStream_Impl& pImpl,
     OSL_ENSURE( m_xSeekable.is(), "No seeking support!\n" );
 }
 
-OInputSeekStream::OInputSeekStream( uno::Reference < io::XInputStream > xStream,
+OInputSeekStream::OInputSeekStream( uno::Reference < io::XInputStream > const & xStream,
                                     const uno::Sequence< beans::PropertyValue >& aProps,
                                     sal_Int32 nStorageType )
 : OInputCompStream( xStream, aProps, nStorageType )
diff --git a/package/source/xstor/oseekinstream.hxx b/package/source/xstor/oseekinstream.hxx
index 02e1baa8f294..dcadc763e98e 100644
--- a/package/source/xstor/oseekinstream.hxx
+++ b/package/source/xstor/oseekinstream.hxx
@@ -32,11 +32,11 @@ protected:
 
 public:
     OInputSeekStream( OWriteStream_Impl& pImpl,
-                      css::uno::Reference < css::io::XInputStream > xStream,
+                      css::uno::Reference < css::io::XInputStream > const & xStream,
                       const css::uno::Sequence< css::beans::PropertyValue >& aProps,
                       sal_Int32 nStorageType );
 
-    OInputSeekStream( css::uno::Reference < css::io::XInputStream > xStream,
+    OInputSeekStream( css::uno::Reference < css::io::XInputStream > const & xStream,
                       const css::uno::Sequence< css::beans::PropertyValue >& aProps,
                       sal_Int32 nStorageType );
 
diff --git a/package/source/xstor/owriteablestream.cxx b/package/source/xstor/owriteablestream.cxx
index 5f2fec2ac146..97342933d60f 100644
--- a/package/source/xstor/owriteablestream.cxx
+++ b/package/source/xstor/owriteablestream.cxx
@@ -1696,7 +1696,7 @@ OWriteStream::OWriteStream( OWriteStream_Impl* pImpl, bool bTransacted )
     m_pData.reset(new WSInternalData_Impl(pImpl->m_rMutexRef, m_pImpl->m_nStorageType));
 }
 
-OWriteStream::OWriteStream( OWriteStream_Impl* pImpl, uno::Reference< io::XStream > xStream, bool bTransacted )
+OWriteStream::OWriteStream( OWriteStream_Impl* pImpl, uno::Reference< io::XStream > const & xStream, bool bTransacted )
 : m_pImpl( pImpl )
 , m_bInStreamDisconnected( false )
 , m_bInitOnDemand( false )
diff --git a/package/source/xstor/owriteablestream.hxx b/package/source/xstor/owriteablestream.hxx
index c84a653fca45..a1dcaf3f5587 100644
--- a/package/source/xstor/owriteablestream.hxx
+++ b/package/source/xstor/owriteablestream.hxx
@@ -260,7 +260,7 @@ protected:
     bool m_bTransacted;
 
     OWriteStream( OWriteStream_Impl* pImpl, bool bTransacted );
-    OWriteStream( OWriteStream_Impl* pImpl, css::uno::Reference< css::io::XStream > xStream, bool bTransacted );
+    OWriteStream( OWriteStream_Impl* pImpl, css::uno::Reference< css::io::XStream > const & xStream, bool bTransacted );
 
     void CloseOutput_Impl();
 
diff --git a/package/source/xstor/xstorage.cxx b/package/source/xstor/xstorage.cxx
index 9e59937114ee..f11843ed8b7a 100644
--- a/package/source/xstor/xstorage.cxx
+++ b/package/source/xstor/xstorage.cxx
@@ -182,10 +182,10 @@ SotElement_Impl::~SotElement_Impl()
 }
 
 // most of properties are holt by the storage but are not used
-OStorage_Impl::OStorage_Impl(   uno::Reference< io::XInputStream > xInputStream,
+OStorage_Impl::OStorage_Impl(   uno::Reference< io::XInputStream > const & xInputStream,
                                 sal_Int32 nMode,
                                 const uno::Sequence< beans::PropertyValue >& xProperties,
-                                uno::Reference< uno::XComponentContext > xContext,
+                                uno::Reference< uno::XComponentContext > const & xContext,
                                 sal_Int32 nStorageType )
 : m_rMutexRef( new SotMutexHolder )
 , m_pAntiImpl( nullptr )
@@ -223,10 +223,10 @@ OStorage_Impl::OStorage_Impl(   uno::Reference< io::XInputStream > xInputStream,
 }
 
 // most of properties are holt by the storage but are not used
-OStorage_Impl::OStorage_Impl(   uno::Reference< io::XStream > xStream,
+OStorage_Impl::OStorage_Impl(   uno::Reference< io::XStream > const & xStream,
                                 sal_Int32 nMode,
                                 const uno::Sequence< beans::PropertyValue >& xProperties,
-                                uno::Reference< uno::XComponentContext > xContext,
+                                uno::Reference< uno::XComponentContext > const & xContext,
                                 sal_Int32 nStorageType )
 : m_rMutexRef( new SotMutexHolder )
 , m_pAntiImpl( nullptr )
@@ -267,9 +267,9 @@ OStorage_Impl::OStorage_Impl(   uno::Reference< io::XStream > xStream,
 
 OStorage_Impl::OStorage_Impl(   OStorage_Impl* pParent,
                                 sal_Int32 nMode,
-                                uno::Reference< container::XNameContainer > xPackageFolder,
-                                uno::Reference< lang::XSingleServiceFactory > xPackage,
-                                uno::Reference< uno::XComponentContext > xContext,
+                                uno::Reference< container::XNameContainer > const & xPackageFolder,
+                                uno::Reference< lang::XSingleServiceFactory > const & xPackage,
+                                uno::Reference< uno::XComponentContext > const & xContext,
                                 sal_Int32 nStorageType )
 : m_rMutexRef( new SotMutexHolder )
 , m_pAntiImpl( nullptr )
@@ -1868,10 +1868,10 @@ void OStorage_Impl::CommitRelInfo( const uno::Reference< container::XNameContain
 
 // OStorage implementation
 
-OStorage::OStorage( uno::Reference< io::XInputStream > xInputStream,
+OStorage::OStorage( uno::Reference< io::XInputStream > const & xInputStream,
                     sal_Int32 nMode,
                     const uno::Sequence< beans::PropertyValue >& xProperties,
-                    uno::Reference< uno::XComponentContext > xContext,
+                    uno::Reference< uno::XComponentContext > const & xContext,
                     sal_Int32 nStorageType )
 : m_pImpl( new OStorage_Impl( xInputStream, nMode, xProperties, xContext, nStorageType ) )
 {
@@ -1879,10 +1879,10 @@ OStorage::OStorage( uno::Reference< io::XInputStream > xInputStream,
     m_pData.reset(new StorInternalData_Impl( m_pImpl->m_rMutexRef, m_pImpl->m_bIsRoot, m_pImpl->m_nStorageType, false));
 }
 
-OStorage::OStorage( uno::Reference< io::XStream > xStream,
+OStorage::OStorage( uno::Reference< io::XStream > const & xStream,
                     sal_Int32 nMode,
                     const uno::Sequence< beans::PropertyValue >& xProperties,
-                    uno::Reference< uno::XComponentContext > xContext,
+                    uno::Reference< uno::XComponentContext > const & xContext,
                     sal_Int32 nStorageType )
 : m_pImpl( new OStorage_Impl( xStream, nMode, xProperties, xContext, nStorageType ) )
 {
diff --git a/package/source/xstor/xstorage.hxx b/package/source/xstor/xstorage.hxx
index e73850466a2e..f7f82c086126 100644
--- a/package/source/xstor/xstorage.hxx
+++ b/package/source/xstor/xstorage.hxx
@@ -177,24 +177,24 @@ struct OStorage_Impl
     sal_Int16 m_nRelInfoStatus;
 
     // Constructors
-    OStorage_Impl(  css::uno::Reference< css::io::XInputStream > xInputStream,
+    OStorage_Impl(  css::uno::Reference< css::io::XInputStream > const & xInputStream,
                     sal_Int32 nMode,
                     const css::uno::Sequence< css::beans::PropertyValue >& xProperties,
-                    css::uno::Reference< css::uno::XComponentContext > xContext,
+                    css::uno::Reference< css::uno::XComponentContext > const & xContext,
                     sal_Int32 nStorageType );
 
-    OStorage_Impl(  css::uno::Reference< css::io::XStream > xStream,
+    OStorage_Impl(  css::uno::Reference< css::io::XStream > const & xStream,
                     sal_Int32 nMode,
                     const css::uno::Sequence< css::beans::PropertyValue >& xProperties,
-                    css::uno::Reference< css::uno::XComponentContext > xContext,
+                    css::uno::Reference< css::uno::XComponentContext > const & xContext,
                     sal_Int32 nStorageType );
 
     // constructor for a substorage
     OStorage_Impl(  OStorage_Impl* pParent,
                     sal_Int32 nMode,
-                    css::uno::Reference< css::container::XNameContainer > xPackageFolder,
-                    css::uno::Reference< css::lang::XSingleServiceFactory > xPackage,
-                    css::uno::Reference< css::uno::XComponentContext > xContext,
+                    css::uno::Reference< css::container::XNameContainer > const & xPackageFolder,
+                    css::uno::Reference< css::lang::XSingleServiceFactory > const & xPackage,
+                    css::uno::Reference< css::uno::XComponentContext > const & xContext,
                     sal_Int32 nStorageType );
 
     ~OStorage_Impl();
@@ -304,16 +304,16 @@ protected:
 
 public:
 
-    OStorage(   css::uno::Reference< css::io::XInputStream > xInputStream,
+    OStorage(   css::uno::Reference< css::io::XInputStream > const & xInputStream,
                 sal_Int32 nMode,
                 const css::uno::Sequence< css::beans::PropertyValue >& xProperties,
-                css::uno::Reference< css::uno::XComponentContext > xContext,
+                css::uno::Reference< css::uno::XComponentContext > const & xContext,
                 sal_Int32 nStorageType );
 
-    OStorage(   css::uno::Reference< css::io::XStream > xStream,
+    OStorage(   css::uno::Reference< css::io::XStream > const & xStream,
                 sal_Int32 nMode,
                 const css::uno::Sequence< css::beans::PropertyValue >& xProperties,
-                css::uno::Reference< css::uno::XComponentContext > xContext,
+                css::uno::Reference< css::uno::XComponentContext > const & xContext,
                 sal_Int32 nStorageType );
 
     OStorage(   OStorage_Impl* pImpl, bool bReadOnlyWrap );
diff --git a/package/source/zipapi/ByteChucker.cxx b/package/source/zipapi/ByteChucker.cxx
index 8354c7cf5e7c..5fc449529b78 100644
--- a/package/source/zipapi/ByteChucker.cxx
+++ b/package/source/zipapi/ByteChucker.cxx
@@ -26,7 +26,7 @@ using namespace ::com::sun::star::io;
 using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::lang;
 
-ByteChucker::ByteChucker(Reference<XOutputStream> xOstream)
+ByteChucker::ByteChucker(Reference<XOutputStream> const & xOstream)
 : xStream(xOstream)
 , xSeek (xOstream, UNO_QUERY )
 , a1Sequence ( 1 )
diff --git a/package/source/zipapi/ByteGrabber.cxx b/package/source/zipapi/ByteGrabber.cxx
index 984becfe32b9..ba0dd42b7728 100644
--- a/package/source/zipapi/ByteGrabber.cxx
+++ b/package/source/zipapi/ByteGrabber.cxx
@@ -33,7 +33,7 @@ using namespace ::com::sun::star;
  *  potentially quite slow and may need to be optimised
  */
 
-ByteGrabber::ByteGrabber(uno::Reference  < io::XInputStream > xIstream)
+ByteGrabber::ByteGrabber(uno::Reference  < io::XInputStream > const & xIstream)
 : xStream(xIstream)
 , xSeek (xIstream, uno::UNO_QUERY )
 , aSequence ( 4 )
diff --git a/package/source/zipapi/XUnbufferedStream.cxx b/package/source/zipapi/XUnbufferedStream.cxx
index f10835c6ede9..e04bfc1b40ba 100644
--- a/package/source/zipapi/XUnbufferedStream.cxx
+++ b/package/source/zipapi/XUnbufferedStream.cxx
@@ -42,7 +42,7 @@ XUnbufferedStream::XUnbufferedStream(
                       const uno::Reference< uno::XComponentContext >& xContext,
                       const rtl::Reference<SotMutexHolder>& aMutexHolder,
                       ZipEntry & rEntry,
-                      Reference < XInputStream > xNewZipStream,
+                      Reference < XInputStream > const & xNewZipStream,
                       const ::rtl::Reference< EncryptionData >& rData,
                       sal_Int8 nStreamMode,
                       bool bIsEncrypted,
diff --git a/package/source/zipapi/XUnbufferedStream.hxx b/package/source/zipapi/XUnbufferedStream.hxx
index 60358e778010..1d34c2c158e6 100644
--- a/package/source/zipapi/XUnbufferedStream.hxx
+++ b/package/source/zipapi/XUnbufferedStream.hxx
@@ -67,7 +67,7 @@ public:
                  const css::uno::Reference< css::uno::XComponentContext >& xContext,
                  const rtl::Reference<SotMutexHolder>& aMutexHolder,
                  ZipEntry & rEntry,
-                 css::uno::Reference < css::io::XInputStream > xNewZipStream,
+                 css::uno::Reference < css::io::XInputStream > const & xNewZipStream,
                  const ::rtl::Reference< EncryptionData >& rData,
                  sal_Int8 nStreamMode,
                  bool bIsEncrypted,