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

Change-Id: Ide1102f4dcc5f31de0f896b570fe453b8074454d

19 files changed, 42 insertions, 42 deletions

diff --git a/include/xmloff/shapeexport.hxx b/include/xmloff/shapeexport.hxx
index 70cc92378100..000f9a731aa9 100644
--- a/include/xmloff/shapeexport.hxx
+++ b/include/xmloff/shapeexport.hxx
@@ -288,7 +288,7 @@ public:
         If this is a non NULL reference, the animation information from all shapes given to exportShape()
         from now on are collected.
     */
-    void setAnimationsExporter( rtl::Reference< XMLAnimationsExporter > xAnimExport ) { mxAnimationsExporter = xAnimExport; }
+    void setAnimationsExporter( rtl::Reference< XMLAnimationsExporter > const & xAnimExport ) { mxAnimationsExporter = xAnimExport; }
 
     /** returns the last set XMLAnimationExport */
     const rtl::Reference< XMLAnimationsExporter >& getAnimationsExporter() const { return mxAnimationsExporter; }
diff --git a/include/xmloff/xformsimport.hxx b/include/xmloff/xformsimport.hxx
index 4aa509d3d502..6624a94acd77 100644
--- a/include/xmloff/xformsimport.hxx
+++ b/include/xmloff/xformsimport.hxx
@@ -45,7 +45,7 @@ XMLOFF_DLLPUBLIC SvXMLImportContext* createXFormsModelContext(
  *  @param pair<XForms binding ID, reference to control>
  */
 void bindXFormsValueBinding(
-    css::uno::Reference<css::frame::XModel>,
+    css::uno::Reference<css::frame::XModel> const &,
     std::pair<css::uno::Reference<css::beans::XPropertySet>,OUString> );
 
 
@@ -54,7 +54,7 @@ void bindXFormsValueBinding(
  *  @param pair<XForms binding ID, reference to control>
  */
 void bindXFormsListBinding(
-    css::uno::Reference<css::frame::XModel>,
+    css::uno::Reference<css::frame::XModel> const &,
     std::pair<css::uno::Reference<css::beans::XPropertySet>,OUString> );
 
 /** perform the actual binding of an XForms submission with the suitable control
@@ -62,7 +62,7 @@ void bindXFormsListBinding(
  *  @param pair<XForms submission ID, reference to control>
  */
 void bindXFormsSubmission(
-    css::uno::Reference<css::frame::XModel>,
+    css::uno::Reference<css::frame::XModel> const &,
     std::pair<css::uno::Reference<css::beans::XPropertySet>,OUString> );
 
 /** applies the given settings to the given XForms container
diff --git a/xmloff/source/chart/MultiPropertySetHandler.hxx b/xmloff/source/chart/MultiPropertySetHandler.hxx
index f425c947f3f9..e263790d5158 100644
--- a/xmloff/source/chart/MultiPropertySetHandler.hxx
+++ b/xmloff/source/chart/MultiPropertySetHandler.hxx
@@ -122,7 +122,7 @@ public:
             is casted later to one of the two of them.
     */
     explicit MultiPropertySetHandler (css::uno::Reference<
-        css::uno::XInterface> xObject);
+        css::uno::XInterface> const & xObject);
     ~MultiPropertySetHandler();
     /** @descr  Add a property to handle.  The type given implicitly by the
             reference to a variable is used to create an instance of
@@ -175,7 +175,7 @@ private:
 };
 
 MultiPropertySetHandler::MultiPropertySetHandler (css::uno::Reference<
-    css::uno::XInterface> xObject)
+    css::uno::XInterface> const & xObject)
         :   mxObject (xObject)
 {
 }
diff --git a/xmloff/source/chart/SchXMLAxisContext.cxx b/xmloff/source/chart/SchXMLAxisContext.cxx
index 41fad4d761f0..db192fc9133a 100644
--- a/xmloff/source/chart/SchXMLAxisContext.cxx
+++ b/xmloff/source/chart/SchXMLAxisContext.cxx
@@ -97,7 +97,7 @@ private:
 
 SchXMLAxisContext::SchXMLAxisContext( SchXMLImportHelper& rImpHelper,
                                       SvXMLImport& rImport, const OUString& rLocalName,
-                                      Reference< chart::XDiagram > xDiagram,
+                                      Reference< chart::XDiagram > const & xDiagram,
                                       std::vector< SchXMLAxis >& rAxes,
                                       OUString & rCategoriesAddress,
                                       bool bAddMissingXAxisForNetCharts,
diff --git a/xmloff/source/chart/SchXMLAxisContext.hxx b/xmloff/source/chart/SchXMLAxisContext.hxx
index bfc5c3e76dbe..1b4da16aeaf8 100644
--- a/xmloff/source/chart/SchXMLAxisContext.hxx
+++ b/xmloff/source/chart/SchXMLAxisContext.hxx
@@ -27,7 +27,7 @@ class SchXMLAxisContext : public SvXMLImportContext
 public:
     SchXMLAxisContext( SchXMLImportHelper& rImpHelper,
                        SvXMLImport& rImport, const OUString& rLocalName,
-                       css::uno::Reference< css::chart::XDiagram > xDiagram,
+                       css::uno::Reference< css::chart::XDiagram > const & xDiagram,
                        std::vector< SchXMLAxis >& aAxes,
                        OUString& rCategoriesAddress,
                        bool bAddMissingXAxisForNetCharts,
diff --git a/xmloff/source/chart/SchXMLExport.cxx b/xmloff/source/chart/SchXMLExport.cxx
index cde978867028..652e59ce46dc 100644
--- a/xmloff/source/chart/SchXMLExport.cxx
+++ b/xmloff/source/chart/SchXMLExport.cxx
@@ -136,7 +136,7 @@ public:
 
     // auto-styles
     /// parse chart and collect all auto-styles used in current pool
-    void collectAutoStyles( css::uno::Reference< css::chart::XChartDocument > rChartDoc );
+    void collectAutoStyles( css::uno::Reference< css::chart::XChartDocument > const & rChartDoc );
 
     /// write the styles collected into the current pool as <style:style> elements
     void exportAutoStyles();
@@ -152,7 +152,7 @@ public:
         which is the outer element of a chart. So these attributes can easily
         be parsed again by the container
      */
-    void exportChart( css::uno::Reference< css::chart::XChartDocument > rChartDoc,
+    void exportChart( css::uno::Reference< css::chart::XChartDocument > const & rChartDoc,
                       bool bIncludeTable );
 
     const rtl::Reference<XMLPropertySetMapper>& GetPropertySetMapper() const;
@@ -181,7 +181,7 @@ public:
     { return mrAutoStylePool; }
 
     /// if bExportContent is false the auto-styles are collected
-    void parseDocument( css::uno::Reference< css::chart::XChartDocument >& rChartDoc,
+    void parseDocument( css::uno::Reference< css::chart::XChartDocument > const & rChartDoc,
                         bool bExportContent,
                         bool bIncludeTable = false );
     void exportTable();
@@ -1070,12 +1070,12 @@ SchXMLExportHelper_Impl::~SchXMLExportHelper_Impl()
 {
 }
 
-void SchXMLExportHelper_Impl::collectAutoStyles( Reference< chart::XChartDocument > rChartDoc )
+void SchXMLExportHelper_Impl::collectAutoStyles( Reference< chart::XChartDocument > const & rChartDoc )
 {
     parseDocument( rChartDoc, false );
 }
 
-void SchXMLExportHelper_Impl::exportChart( Reference< chart::XChartDocument > rChartDoc,
+void SchXMLExportHelper_Impl::exportChart( Reference< chart::XChartDocument > const & rChartDoc,
                                       bool bIncludeTable )
 {
     parseDocument( rChartDoc, true, bIncludeTable );
@@ -1106,7 +1106,7 @@ static OUString lcl_GetStringFromNumberSequence( const css::uno::Sequence< sal_I
 }
 
 /// if bExportContent is false the auto-styles are collected
-void SchXMLExportHelper_Impl::parseDocument( Reference< chart::XChartDocument >& rChartDoc,
+void SchXMLExportHelper_Impl::parseDocument( Reference< chart::XChartDocument > const & rChartDoc,
                                         bool bExportContent,
                                         bool bIncludeTable )
 {
diff --git a/xmloff/source/chart/SchXMLPropertyMappingContext.cxx b/xmloff/source/chart/SchXMLPropertyMappingContext.cxx
index 7c92cbda67e8..28074eedd8e1 100644
--- a/xmloff/source/chart/SchXMLPropertyMappingContext.cxx
+++ b/xmloff/source/chart/SchXMLPropertyMappingContext.cxx
@@ -65,7 +65,7 @@ SchXMLPropertyMappingContext::SchXMLPropertyMappingContext( SchXMLImportHelper&
         SvXMLImport& rImport, const OUString& rLocalName,
         tSchXMLLSequencesPerIndex & rLSequencesPerIndex,
         uno::Reference<
-        chart2::XDataSeries > xSeries ):
+        chart2::XDataSeries > const & xSeries ):
     SvXMLImportContext( rImport, XML_NAMESPACE_LO_EXT, rLocalName ),
     mrImportHelper( rImpHelper ),
     mxDataSeries(xSeries),
diff --git a/xmloff/source/chart/SchXMLPropertyMappingContext.hxx b/xmloff/source/chart/SchXMLPropertyMappingContext.hxx
index 2a0ba1c61413..44942bfe4469 100644
--- a/xmloff/source/chart/SchXMLPropertyMappingContext.hxx
+++ b/xmloff/source/chart/SchXMLPropertyMappingContext.hxx
@@ -30,7 +30,7 @@ public:
                           SvXMLImport& rImport, const OUString& rLocalName,
                           tSchXMLLSequencesPerIndex& rLSequencesPerIndex,
                               css::uno::Reference<
-                              css::chart2::XDataSeries > xSeries );
+                              css::chart2::XDataSeries > const & xSeries );
 
     virtual ~SchXMLPropertyMappingContext();
 
diff --git a/xmloff/source/core/PropertySetMerger.cxx b/xmloff/source/core/PropertySetMerger.cxx
index dd32897722c9..e9e14144a037 100644
--- a/xmloff/source/core/PropertySetMerger.cxx
+++ b/xmloff/source/core/PropertySetMerger.cxx
@@ -39,7 +39,7 @@ private:
     Reference< XPropertySetInfo > mxPropSet2Info;
 
 public:
-    PropertySetMergerImpl( const Reference< XPropertySet > rPropSet1, const Reference< XPropertySet > rPropSet2 );
+    PropertySetMergerImpl( const Reference< XPropertySet > & rPropSet1, const Reference< XPropertySet > & rPropSet2 );
     virtual ~PropertySetMergerImpl();
 
     // XPropertySet
@@ -65,7 +65,7 @@ public:
 
 // Interface implementation
 
-PropertySetMergerImpl::PropertySetMergerImpl( Reference< XPropertySet > rPropSet1, Reference< XPropertySet > rPropSet2 )
+PropertySetMergerImpl::PropertySetMergerImpl( Reference< XPropertySet > const & rPropSet1, Reference< XPropertySet > const & rPropSet2 )
 : mxPropSet1( rPropSet1 )
 , mxPropSet1State( rPropSet1, UNO_QUERY )
 , mxPropSet1Info( rPropSet1->getPropertySetInfo() )
diff --git a/xmloff/source/draw/XMLImageMapContext.cxx b/xmloff/source/draw/XMLImageMapContext.cxx
index a502abadaf3d..ef37224a45ee 100644
--- a/xmloff/source/draw/XMLImageMapContext.cxx
+++ b/xmloff/source/draw/XMLImageMapContext.cxx
@@ -126,7 +126,7 @@ public:
         SvXMLImport& rImport,
         sal_uInt16 nPrefix,
         const OUString& rLocalName,
-        css::uno::Reference<css::container::XIndexContainer> xMap,
+        css::uno::Reference<css::container::XIndexContainer> const & xMap,
         const sal_Char* pServiceName);
 
     void StartElement(
@@ -154,7 +154,7 @@ XMLImageMapObjectContext::XMLImageMapObjectContext(
     SvXMLImport& rImport,
     sal_uInt16 nPrefix,
     const OUString& rLocalName,
-    Reference<XIndexContainer> xMap,
+    Reference<XIndexContainer> const & xMap,
     const sal_Char* pServiceName) :
         SvXMLImportContext(rImport, nPrefix, rLocalName),
         sBoundary("Boundary"),
@@ -309,7 +309,7 @@ public:
         SvXMLImport& rImport,
         sal_uInt16 nPrefix,
         const OUString& rLocalName,
-        css::uno::Reference<css::container::XIndexContainer> xMap);
+        css::uno::Reference<css::container::XIndexContainer> const & xMap);
 
     virtual ~XMLImageMapRectangleContext();
 
@@ -327,7 +327,7 @@ XMLImageMapRectangleContext::XMLImageMapRectangleContext(
     SvXMLImport& rImport,
     sal_uInt16 nPrefix,
     const OUString& rLocalName,
-    Reference<XIndexContainer> xMap) :
+    Reference<XIndexContainer> const & xMap) :
         XMLImageMapObjectContext(rImport, nPrefix, rLocalName, xMap,
                                  "com.sun.star.image.ImageMapRectangleObject"),
         bXOK(false),
@@ -411,7 +411,7 @@ public:
         SvXMLImport& rImport,
         sal_uInt16 nPrefix,
         const OUString& rLocalName,
-        css::uno::Reference<css::container::XIndexContainer> xMap);
+        css::uno::Reference<css::container::XIndexContainer> const & xMap);
 
     virtual ~XMLImageMapPolygonContext();
 
@@ -429,7 +429,7 @@ XMLImageMapPolygonContext::XMLImageMapPolygonContext(
     SvXMLImport& rImport,
     sal_uInt16 nPrefix,
     const OUString& rLocalName,
-    Reference<XIndexContainer> xMap) :
+    Reference<XIndexContainer> const & xMap) :
         XMLImageMapObjectContext(rImport, nPrefix, rLocalName, xMap,
                                  "com.sun.star.image.ImageMapPolygonObject"),
         bViewBoxOK(false),
@@ -500,7 +500,7 @@ public:
         SvXMLImport& rImport,
         sal_uInt16 nPrefix,
         const OUString& rLocalName,
-        css::uno::Reference<css::container::XIndexContainer> xMap);
+        css::uno::Reference<css::container::XIndexContainer> const & xMap);
 
     virtual ~XMLImageMapCircleContext();
 
@@ -518,7 +518,7 @@ XMLImageMapCircleContext::XMLImageMapCircleContext(
     SvXMLImport& rImport,
     sal_uInt16 nPrefix,
     const OUString& rLocalName,
-    Reference<XIndexContainer> xMap)
+    Reference<XIndexContainer> const & xMap)
     : XMLImageMapObjectContext(rImport, nPrefix, rLocalName, xMap,
           "com.sun.star.image.ImageMapCircleObject")
     , nRadius(0)
diff --git a/xmloff/source/draw/numithdl.cxx b/xmloff/source/draw/numithdl.cxx
index aac3002398bd..e8395198be46 100644
--- a/xmloff/source/draw/numithdl.cxx
+++ b/xmloff/source/draw/numithdl.cxx
@@ -25,7 +25,7 @@ using namespace ::com::sun::star;
 // class XMLNumRulePropHdl
 
 
-XMLNumRulePropHdl::XMLNumRulePropHdl( css::uno::Reference< css::ucb::XAnyCompare > xNumRuleCompare )
+XMLNumRulePropHdl::XMLNumRulePropHdl( css::uno::Reference< css::ucb::XAnyCompare > const & xNumRuleCompare )
 : mxNumRuleCompare( xNumRuleCompare )
 {
 }
diff --git a/xmloff/source/draw/numithdl.hxx b/xmloff/source/draw/numithdl.hxx
index 99857ef04505..0ea5cfa0b51d 100644
--- a/xmloff/source/draw/numithdl.hxx
+++ b/xmloff/source/draw/numithdl.hxx
@@ -31,7 +31,7 @@ class XMLNumRulePropHdl : public XMLPropertyHandler
 private:
     css::uno::Reference< css::ucb::XAnyCompare > mxNumRuleCompare;
 public:
-    explicit XMLNumRulePropHdl( css::uno::Reference< css::ucb::XAnyCompare > xNumRuleCompare );
+    explicit XMLNumRulePropHdl( css::uno::Reference< css::ucb::XAnyCompare > const & xNumRuleCompare );
     virtual ~XMLNumRulePropHdl();
 
     virtual bool equals( const css::uno::Any& r1, const css::uno::Any& r2 ) const override;
diff --git a/xmloff/source/draw/sdpropls.cxx b/xmloff/source/draw/sdpropls.cxx
index b64adc3b589d..00e2968ccf74 100644
--- a/xmloff/source/draw/sdpropls.cxx
+++ b/xmloff/source/draw/sdpropls.cxx
@@ -853,12 +853,12 @@ bool XMLSdHeaderFooterVisibilityTypeHdl::exportXML(
     return bRet;
 }
 
-XMLSdPropHdlFactory::XMLSdPropHdlFactory( uno::Reference< frame::XModel > xModel, SvXMLImport& rImport )
+XMLSdPropHdlFactory::XMLSdPropHdlFactory( uno::Reference< frame::XModel > const & xModel, SvXMLImport& rImport )
 : mxModel( xModel ), mpExport(nullptr), mpImport( &rImport )
 {
 }
 
-XMLSdPropHdlFactory::XMLSdPropHdlFactory( uno::Reference< frame::XModel > xModel, SvXMLExport& rExport )
+XMLSdPropHdlFactory::XMLSdPropHdlFactory( uno::Reference< frame::XModel > const & xModel, SvXMLExport& rExport )
 : mxModel( xModel ), mpExport( &rExport ), mpImport(nullptr)
 {
 }
diff --git a/xmloff/source/draw/sdpropls.hxx b/xmloff/source/draw/sdpropls.hxx
index 22683e7aef32..9a71457557f1 100644
--- a/xmloff/source/draw/sdpropls.hxx
+++ b/xmloff/source/draw/sdpropls.hxx
@@ -60,8 +60,8 @@ private:
     SvXMLImport* mpImport;
 
 public:
-    XMLSdPropHdlFactory( css::uno::Reference< css::frame::XModel >, SvXMLExport& rExport );
-    XMLSdPropHdlFactory( css::uno::Reference< css::frame::XModel >, SvXMLImport& rImport );
+    XMLSdPropHdlFactory( css::uno::Reference< css::frame::XModel > const & xModel, SvXMLExport& rExport );
+    XMLSdPropHdlFactory( css::uno::Reference< css::frame::XModel > const & xModel, SvXMLImport& rImport );
     virtual ~XMLSdPropHdlFactory();
     virtual const XMLPropertyHandler* GetPropertyHandler( sal_Int32 nType ) const override;
 };
diff --git a/xmloff/source/text/txtimp.cxx b/xmloff/source/text/txtimp.cxx
index 468c31ea7f88..dd96833d0be6 100644
--- a/xmloff/source/text/txtimp.cxx
+++ b/xmloff/source/text/txtimp.cxx
@@ -824,7 +824,7 @@ namespace
         public:
             typedef pair<OUString,OUString> field_param_t;
             typedef vector<field_param_t> field_params_t;
-            FieldParamImporter(const field_params_t* const pInParams, Reference<XNameContainer> xOutParams)
+            FieldParamImporter(const field_params_t* const pInParams, Reference<XNameContainer> const & xOutParams)
                 : m_pInParams(pInParams)
                 , m_xOutParams(xOutParams)
             { };
diff --git a/xmloff/source/text/txtparae.cxx b/xmloff/source/text/txtparae.cxx
index 147d56531269..79b167839d6d 100644
--- a/xmloff/source/text/txtparae.cxx
+++ b/xmloff/source/text/txtparae.cxx
@@ -216,7 +216,7 @@ namespace
     class FieldParamExporter
     {
         public:
-            FieldParamExporter(SvXMLExport* const pExport, Reference<XNameContainer> xFieldParams)
+            FieldParamExporter(SvXMLExport* const pExport, Reference<XNameContainer> const & xFieldParams)
                 : m_pExport(pExport)
                 , m_xFieldParams(xFieldParams)
                 { };
diff --git a/xmloff/source/xforms/xformsapi.cxx b/xmloff/source/xforms/xformsapi.cxx
index 13e22047c35a..e15c135f3cc0 100644
--- a/xmloff/source/xforms/xformsapi.cxx
+++ b/xmloff/source/xforms/xformsapi.cxx
@@ -95,7 +95,7 @@ void xforms_addXFormsModel(
 }
 
 static Reference<XPropertySet> lcl_findXFormsBindingOrSubmission(
-    Reference<frame::XModel>& xDocument,
+    Reference<frame::XModel> const & xDocument,
     const OUString& rBindingID,
     bool bBinding )
 {
@@ -150,14 +150,14 @@ static Reference<XPropertySet> lcl_findXFormsBindingOrSubmission(
 }
 
 Reference<XPropertySet> xforms_findXFormsBinding(
-    Reference<frame::XModel>& xDocument,
+    Reference<frame::XModel> const & xDocument,
     const OUString& rBindingID )
 {
     return lcl_findXFormsBindingOrSubmission( xDocument, rBindingID, true );
 }
 
 Reference<XPropertySet> xforms_findXFormsSubmission(
-    Reference<frame::XModel>& xDocument,
+    Reference<frame::XModel> const & xDocument,
     const OUString& rBindingID )
 {
     return lcl_findXFormsBindingOrSubmission( xDocument, rBindingID, false );
diff --git a/xmloff/source/xforms/xformsapi.hxx b/xmloff/source/xforms/xformsapi.hxx
index b24caf77e349..0af4c48223ad 100644
--- a/xmloff/source/xforms/xformsapi.hxx
+++ b/xmloff/source/xforms/xformsapi.hxx
@@ -40,9 +40,9 @@ void xforms_addXFormsModel(
     const css::uno::Reference<css::frame::XModel>& xDocument,
     const css::uno::Reference<css::xforms::XModel2>& xModel );
 
-css::uno::Reference<css::beans::XPropertySet> xforms_findXFormsBinding( css::uno::Reference<css::frame::XModel>&, const OUString& );
+css::uno::Reference<css::beans::XPropertySet> xforms_findXFormsBinding( css::uno::Reference<css::frame::XModel> const &, const OUString& );
 
-css::uno::Reference<css::beans::XPropertySet> xforms_findXFormsSubmission( css::uno::Reference<css::frame::XModel>&, const OUString& );
+css::uno::Reference<css::beans::XPropertySet> xforms_findXFormsSubmission( css::uno::Reference<css::frame::XModel> const &, const OUString& );
 
 void xforms_setValue(
     css::uno::Reference<css::beans::XPropertySet>& xPropSet,
diff --git a/xmloff/source/xforms/xformsimport.cxx b/xmloff/source/xforms/xformsimport.cxx
index 7789936a1514..12741083741f 100644
--- a/xmloff/source/xforms/xformsimport.cxx
+++ b/xmloff/source/xforms/xformsimport.cxx
@@ -64,7 +64,7 @@ SvXMLImportContext* createXFormsModelContext(
 }
 
 void bindXFormsValueBinding(
-    Reference<XModel> xModel,
+    Reference<XModel> const & xModel,
     pair<Reference<XPropertySet>,OUString> aPair )
 {
     Reference<XBindableValue> xBindable(
@@ -89,7 +89,7 @@ void bindXFormsValueBinding(
 }
 
 void bindXFormsListBinding(
-    Reference<XModel> xModel,
+    Reference<XModel> const & xModel,
     ::pair<Reference<XPropertySet>,OUString> aPair )
 {
     Reference<XListEntrySink> xListEntrySink(
@@ -114,7 +114,7 @@ void bindXFormsListBinding(
 }
 
 void bindXFormsSubmission(
-    Reference<XModel> xModel,
+    Reference<XModel> const & xModel,
     pair<Reference<XPropertySet>,OUString> aPair )
 {
     Reference<XSubmissionSupplier> xSubmissionSupp( aPair.first, UNO_QUERY );