loplugin:passstuffbyref

Change-Id: I6e6c47020073ec183c6c8ff9cd026a0bec23b177
Reviewed-on: https://gerrit.libreoffice.org/57088
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

3 files changed, 9 insertions, 9 deletions

diff --git a/sc/source/ui/dataprovider/datatransformation.cxx b/sc/source/ui/dataprovider/datatransformation.cxx
index fd779bf6587a..27bf47d75ea0 100644
--- a/sc/source/ui/dataprovider/datatransformation.cxx
+++ b/sc/source/ui/dataprovider/datatransformation.cxx
@@ -49,7 +49,7 @@ TransformationType ColumnRemoveTransformation::getTransformationType() const
     return TransformationType::DELETE_TRANSFORMATION;
 }
 
-std::set<SCCOL> ColumnRemoveTransformation::getColumns() const
+const std::set<SCCOL> & ColumnRemoveTransformation::getColumns() const
 {
     return maColumns;
 }
@@ -144,12 +144,12 @@ TransformationType MergeColumnTransformation::getTransformationType() const
     return TransformationType::MERGE_TRANSFORMATION;
 }
 
-OUString MergeColumnTransformation::getMergeString() const
+const OUString & MergeColumnTransformation::getMergeString() const
 {
     return maMergeString;
 }
 
-std::set<SCCOL> MergeColumnTransformation::getColumns() const
+const std::set<SCCOL> & MergeColumnTransformation::getColumns() const
 {
     return maColumns;
 }
@@ -169,7 +169,7 @@ TransformationType SortTransformation::getTransformationType() const
     return TransformationType::SORT_TRANSFORMATION;
 }
 
-ScSortParam SortTransformation::getSortParam() const
+const ScSortParam & SortTransformation::getSortParam() const
 {
     return maSortParam;
 }
diff --git a/sc/source/ui/inc/datatransformation.hxx b/sc/source/ui/inc/datatransformation.hxx
index 305e35a3cd54..6e49a3263bc9 100644
--- a/sc/source/ui/inc/datatransformation.hxx
+++ b/sc/source/ui/inc/datatransformation.hxx
@@ -60,7 +60,7 @@ public:
     virtual ~ColumnRemoveTransformation() override;
     virtual void Transform(ScDocument& rDoc) const override;
     virtual TransformationType getTransformationType() const override;
-    std::set<SCCOL> getColumns() const;
+    const std::set<SCCOL> & getColumns() const;
 };
 
 class SC_DLLPUBLIC SplitColumnTransformation : public DataTransformation
@@ -87,8 +87,8 @@ public:
     MergeColumnTransformation(const std::set<SCCOL>& rColumns, const OUString& rMergeString);
     virtual void Transform(ScDocument& rDoc) const  override;
     virtual TransformationType getTransformationType() const override;
-    OUString getMergeString() const;
-    std::set<SCCOL> getColumns() const;
+    const OUString & getMergeString() const;
+    const std::set<SCCOL> & getColumns() const;
 };
 
 class SC_DLLPUBLIC SortTransformation : public DataTransformation
@@ -99,7 +99,7 @@ public:
     SortTransformation(const ScSortParam& rParam);
     virtual void Transform(ScDocument& rDoc) const override;
     virtual TransformationType getTransformationType() const override;
-    ScSortParam getSortParam() const;
+    const ScSortParam & getSortParam() const;
 };
 
 class SC_DLLPUBLIC TextTransformation : public DataTransformation
diff --git a/sc/source/ui/inc/docsh.hxx b/sc/source/ui/inc/docsh.hxx
index 41c7e9b5845e..1005799e192f 100644
--- a/sc/source/ui/inc/docsh.hxx
+++ b/sc/source/ui/inc/docsh.hxx
@@ -227,7 +227,7 @@ public:
     ScDocument&     GetDocument()   { return m_aDocument; }
     ScDocFunc&      GetDocFunc()    { return *m_pDocFunc; }
 
-    css::uno::Reference<css::datatransfer::XTransferable2> GetClipData() { return m_xClipData; }
+    css::uno::Reference<css::datatransfer::XTransferable2> const & GetClipData() { return m_xClipData; }
     void SetClipData(const css::uno::Reference<css::datatransfer::XTransferable2>& xTransferable) { m_xClipData = xTransferable; }
 
     SfxPrinter*     GetPrinter( bool bCreateIfNotExist = true );