clang-tidy modernize-pass-by-value in dbaccess

Change-Id: I8f89ec8a73f82fd93320a710165df17180939f96
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135731
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

42 files changed, 121 insertions, 95 deletions

diff --git a/dbaccess/source/ui/app/subcomponentmanager.cxx b/dbaccess/source/ui/app/subcomponentmanager.cxx
index bf92f2400a31..065ed6128edd 100644
--- a/dbaccess/source/ui/app/subcomponentmanager.cxx
+++ b/dbaccess/source/ui/app/subcomponentmanager.cxx
@@ -33,6 +33,7 @@
 
 #include <tools/diagnose_ex.h>
 #include <dbaccess/dataview.hxx>
+#include <utility>
 #include <vcl/svapp.hxx>
 #include <osl/mutex.hxx>
 
@@ -93,9 +94,9 @@ namespace dbaui
             {
             }
 
-            SubComponentDescriptor( const OUString& i_rName, const sal_Int32 i_nComponentType,
+            SubComponentDescriptor( OUString i_sName, const sal_Int32 i_nComponentType,
                     const ElementOpenMode i_eOpenMode, const Reference< XComponent >& i_rComponent )
-                :sName( i_rName )
+                :sName(std::move( i_sName ))
                 ,nComponentType( i_nComponentType )
                 ,eOpenMode( i_eOpenMode )
             {
@@ -168,9 +169,9 @@ namespace dbaui
         struct SubComponentMatch
         {
         public:
-            SubComponentMatch( const OUString& i_rName, const sal_Int32 i_nComponentType,
+            SubComponentMatch( OUString i_sName, const sal_Int32 i_nComponentType,
                     const ElementOpenMode i_eOpenMode )
-                :m_sName( i_rName )
+                :m_sName(std::move( i_sName ))
                 ,m_nComponentType( i_nComponentType )
                 ,m_eOpenMode( i_eOpenMode )
             {
@@ -192,9 +193,9 @@ namespace dbaui
     // SubComponentManager_Data
     struct SubComponentManager_Data
     {
-        SubComponentManager_Data( OApplicationController& _rController, const ::comphelper::SharedMutex& _rMutex )
+        SubComponentManager_Data( OApplicationController& _rController, ::comphelper::SharedMutex  _aMutex )
             :m_rController( _rController )
-            ,m_aMutex( _rMutex )
+            ,m_aMutex(std::move( _aMutex ))
         {
         }
 
diff --git a/dbaccess/source/ui/browser/unodatbr.cxx b/dbaccess/source/ui/browser/unodatbr.cxx
index b993fac9754f..1356aedb1a2c 100644
--- a/dbaccess/source/ui/browser/unodatbr.cxx
+++ b/dbaccess/source/ui/browser/unodatbr.cxx
@@ -88,6 +88,7 @@
 #include <tools/multisel.hxx>
 #include <tools/urlobj.hxx>
 #include <unotools/confignode.hxx>
+#include <utility>
 #include <vcl/split.hxx>
 #include <vcl/svapp.hxx>
 #include <vcl/toolbox.hxx>
@@ -1037,7 +1038,7 @@ namespace
     struct FilterByEntryDataId : public IEntryFilter
     {
         OUString sId;
-        explicit FilterByEntryDataId( const OUString& _rId ) : sId( _rId ) { }
+        explicit FilterByEntryDataId( OUString _aId ) : sId(std::move( _aId )) { }
 
         virtual ~FilterByEntryDataId() {}
 
diff --git a/dbaccess/source/ui/control/dbtreelistbox.cxx b/dbaccess/source/ui/control/dbtreelistbox.cxx
index 472171142368..1dd94c35ee4f 100644
--- a/dbaccess/source/ui/control/dbtreelistbox.cxx
+++ b/dbaccess/source/ui/control/dbtreelistbox.cxx
@@ -36,6 +36,7 @@
 #include <toolkit/awt/vclxmenu.hxx>
 #include <toolkit/helper/vclunohelper.hxx>
 #include <svx/dbaobjectex.hxx>
+#include <utility>
 #include <vcl/commandevent.hxx>
 #include <vcl/event.hxx>
 #include <vcl/svapp.hxx>
@@ -300,8 +301,8 @@ namespace
     class SelectionSupplier : public SelectionSupplier_Base
     {
     public:
-        explicit SelectionSupplier( const Any& _rSelection )
-            :m_aSelection( _rSelection )
+        explicit SelectionSupplier( Any _aSelection )
+            :m_aSelection(std::move( _aSelection ))
         {
         }
 
diff --git a/dbaccess/source/ui/dlg/CollectionView.cxx b/dbaccess/source/ui/dlg/CollectionView.cxx
index 2e02fda505e0..56fd5f6f5b93 100644
--- a/dbaccess/source/ui/dlg/CollectionView.cxx
+++ b/dbaccess/source/ui/dlg/CollectionView.cxx
@@ -42,6 +42,7 @@
 #include <ucbhelper/commandenvironment.hxx>
 #include <ucbhelper/content.hxx>
 #include <connectivity/dbexception.hxx>
+#include <utility>
 
 namespace dbaui
 {
@@ -58,10 +59,10 @@ using namespace comphelper;
 OCollectionView::OCollectionView(weld::Window* pParent,
                                  const Reference< XContent>& _xContent,
                                  const OUString& _sDefaultName,
-                                 const css::uno::Reference< css::uno::XComponentContext >& _rxContext)
+                                 css::uno::Reference< css::uno::XComponentContext > _xContext)
     : GenericDialogController(pParent, "dbaccess/ui/collectionviewdialog.ui", "CollectionView")
     , m_xContent(_xContent)
-    , m_xContext(_rxContext)
+    , m_xContext(std::move(_xContext))
     , m_bCreateForm(true)
     , m_xFTCurrentPath(m_xBuilder->weld_label("currentPathLabel"))
     , m_xNewFolder(m_xBuilder->weld_button("newFolderButton"))
diff --git a/dbaccess/source/ui/dlg/dbfindex.cxx b/dbaccess/source/ui/dlg/dbfindex.cxx
index 89f74ab72b97..061486e91201 100644
--- a/dbaccess/source/ui/dlg/dbfindex.cxx
+++ b/dbaccess/source/ui/dlg/dbfindex.cxx
@@ -29,6 +29,7 @@
 #include <ucbhelper/content.hxx>
 #include <svl/filenotation.hxx>
 #include <rtl/strbuf.hxx>
+#include <utility>
 
 namespace dbaui
 {
@@ -39,9 +40,9 @@ using namespace ::svt;
 constexpr OStringLiteral aGroupIdent("dBase III");
 
 
-ODbaseIndexDialog::ODbaseIndexDialog(weld::Window * pParent, const OUString& aDataSrcName)
+ODbaseIndexDialog::ODbaseIndexDialog(weld::Window * pParent, OUString aDataSrcName)
     : GenericDialogController(pParent, "dbaccess/ui/dbaseindexdialog.ui", "DBaseIndexDialog")
-    , m_aDSN(aDataSrcName)
+    , m_aDSN(std::move(aDataSrcName))
     , m_xPB_OK(m_xBuilder->weld_button("ok"))
     , m_xCB_Tables(m_xBuilder->weld_combo_box("table"))
     , m_xIndexes(m_xBuilder->weld_widget("frame"))
diff --git a/dbaccess/source/ui/dlg/dbfindex.hxx b/dbaccess/source/ui/dlg/dbfindex.hxx
index 53b75640e6f4..0db7ea7b04a6 100644
--- a/dbaccess/source/ui/dlg/dbfindex.hxx
+++ b/dbaccess/source/ui/dlg/dbfindex.hxx
@@ -19,6 +19,7 @@
 
 #pragma once
 
+#include <utility>
 #include <vcl/weld.hxx>
 #include <deque>
 
@@ -34,7 +35,7 @@ private:
 
 public:
     OTableIndex() { }
-    explicit OTableIndex( const OUString& rFileName ) : aIndexFileName( rFileName ) { }
+    explicit OTableIndex( OUString aFileName ) : aIndexFileName(std::move( aFileName )) { }
 
     const OUString& GetIndexFileName() const { return aIndexFileName; }
 };
@@ -53,7 +54,7 @@ private:
     TableIndexList aIndexList;
 
 public:
-    explicit OTableInfo( const OUString& rName ) : aTableName(rName) { }
+    explicit OTableInfo( OUString aName ) : aTableName(std::move(aName)) { }
 
     void WriteInfFile( const OUString& rDSN ) const;
 };
@@ -101,7 +102,7 @@ protected:
     void checkButtons();
 
 public:
-    ODbaseIndexDialog(weld::Window * pParent, const OUString& rDataSrcName);
+    ODbaseIndexDialog(weld::Window * pParent, OUString aDataSrcName);
     virtual ~ODbaseIndexDialog() override;
 };
 
diff --git a/dbaccess/source/ui/dlg/dbwizsetup.cxx b/dbaccess/source/ui/dlg/dbwizsetup.cxx
index f687740dd696..bf794ac3892c 100644
--- a/dbaccess/source/ui/dlg/dbwizsetup.cxx
+++ b/dbaccess/source/ui/dlg/dbwizsetup.cxx
@@ -63,6 +63,7 @@
 #include <tools/diagnose_ex.h>
 #include <osl/diagnose.h>
 #include <connectivity/DriversConfig.hxx>
+#include <utility>
 
 namespace dbaui
 {
@@ -858,7 +859,7 @@ bool ODbTypeWizDialogSetup::SaveDatabaseDocument()
             OAsynchronousLink                    m_aAsyncCaller;
 
         public:
-            AsyncLoader( const Reference< XComponentContext >& _rxORB, const OUString& _rURL );
+            AsyncLoader( const Reference< XComponentContext >& _xORB, OUString _aURL );
 
             void doLoadAsync();
 
@@ -872,8 +873,8 @@ bool ODbTypeWizDialogSetup::SaveDatabaseDocument()
             DECL_LINK( OnOpenDocument, void*, void );
         };
 
-        AsyncLoader::AsyncLoader( const Reference< XComponentContext >& _rxORB, const OUString& _rURL )
-            :m_sURL( _rURL )
+        AsyncLoader::AsyncLoader( const Reference< XComponentContext >& _rxORB, OUString _aURL )
+            :m_sURL(std::move( _aURL ))
             ,m_aAsyncCaller( LINK( this, AsyncLoader, OnOpenDocument ) )
         {
             try
diff --git a/dbaccess/source/ui/dlg/dlgsave.cxx b/dbaccess/source/ui/dlg/dlgsave.cxx
index ce5d16881f1d..18c8a1512cb6 100644
--- a/dbaccess/source/ui/dlg/dlgsave.cxx
+++ b/dbaccess/source/ui/dlg/dlgsave.cxx
@@ -27,6 +27,7 @@
 #include <SqlNameEdit.hxx>
 #include <com/sun/star/sdbc/XDatabaseMetaData.hpp>
 #include <objectnamecheck.hxx>
+#include <utility>
 #include <tools/diagnose_ex.h>
 
 using namespace dbaui;
@@ -64,11 +65,11 @@ public:
 
     OSaveAsDlgImpl( weld::Builder* pParent, sal_Int32 _rType,
                     const css::uno::Reference< css::sdbc::XConnection>& _xConnection,
-                    const OUString& rDefault,
+                    OUString sDefault,
                     const IObjectNameCheck& _rObjectNameCheck,
                     SADFlags _nFlags);
     OSaveAsDlgImpl( weld::Builder* pParent,
-                    const OUString& rDefault,
+                    OUString sDefault,
                     const IObjectNameCheck& _rObjectNameCheck,
                     SADFlags _nFlags);
 };
@@ -86,12 +87,12 @@ IMPL_LINK(OSaveAsDlgImpl, TextFilterHdl, OUString&, rTest, bool)
 OSaveAsDlgImpl::OSaveAsDlgImpl(weld::Builder* pBuilder,
                                sal_Int32 _rType,
                                const Reference< XConnection>& _xConnection,
-                               const OUString& rDefault,
+                               OUString sDefault,
                                const IObjectNameCheck& _rObjectNameCheck,
                                SADFlags _nFlags)
     : m_aQryLabel(DBA_RES(STR_QRY_LABEL))
     , m_sTblLabel(DBA_RES(STR_TBL_LABEL))
-    , m_aName(rDefault)
+    , m_aName(std::move(sDefault))
     , m_rObjectNameCheck( _rObjectNameCheck )
     , m_nType(_rType)
     , m_nFlags(_nFlags)
@@ -120,12 +121,12 @@ OSaveAsDlgImpl::OSaveAsDlgImpl(weld::Builder* pBuilder,
 }
 
 OSaveAsDlgImpl::OSaveAsDlgImpl(weld::Builder* pBuilder,
-                               const OUString& rDefault,
+                               OUString sDefault,
                                const IObjectNameCheck& _rObjectNameCheck,
                                SADFlags _nFlags)
     : m_aQryLabel(DBA_RES(STR_QRY_LABEL))
     , m_sTblLabel(DBA_RES(STR_TBL_LABEL))
-    , m_aName(rDefault)
+    , m_aName(std::move(sDefault))
     , m_rObjectNameCheck( _rObjectNameCheck )
     , m_nType(CommandType::COMMAND)
     , m_nFlags(_nFlags)
diff --git a/dbaccess/source/ui/dlg/generalpage.cxx b/dbaccess/source/ui/dlg/generalpage.cxx
index 5809b7bd7088..f2a11bfc9b24 100644
--- a/dbaccess/source/ui/dlg/generalpage.cxx
+++ b/dbaccess/source/ui/dlg/generalpage.cxx
@@ -25,6 +25,7 @@
 #include <dsitems.hxx>
 #include <sfx2/filedlghelper.hxx>
 #include <sfx2/docfilt.hxx>
+#include <utility>
 #include <vcl/svapp.hxx>
 #include <vcl/weld.hxx>
 #include <svl/stritem.hxx>
@@ -76,7 +77,7 @@ namespace dbaui
             OUString eType;
             OUString sDisplayName;
 
-            DisplayedType( const OUString& _eType, const OUString& _rDisplayName ) : eType( _eType ), sDisplayName( _rDisplayName ) { }
+            DisplayedType( OUString _eType, OUString _sDisplayName ) : eType(std::move( _eType )), sDisplayName(std::move( _sDisplayName )) { }
         };
         typedef std::vector< DisplayedType > DisplayedTypes;
 
diff --git a/dbaccess/source/ui/dlg/sqlmessage.cxx b/dbaccess/source/ui/dlg/sqlmessage.cxx
index eacc5908328d..81fa5100d0ef 100644
--- a/dbaccess/source/ui/dlg/sqlmessage.cxx
+++ b/dbaccess/source/ui/dlg/sqlmessage.cxx
@@ -22,6 +22,7 @@
 #include <strings.hrc>
 #include <com/sun/star/sdbc/SQLException.hpp>
 #include <com/sun/star/sdb/SQLContext.hpp>
+#include <utility>
 #include <vcl/stdtext.hxx>
 #include <vcl/svapp.hxx>
 #include <vcl/weld.hxx>
@@ -50,8 +51,8 @@ namespace
         OUString m_defaultImageID;
 
     public:
-        explicit ImageProvider(const OUString& defaultImageID)
-            : m_defaultImageID(defaultImageID)
+        explicit ImageProvider(OUString defaultImageID)
+            : m_defaultImageID(std::move(defaultImageID))
         {
         }
 
@@ -555,9 +556,9 @@ void OSQLMessageBox::Construct(weld::Window* pParent, MessBoxStyle _nStyle, Mess
     impl_addDetailsButton();
 }
 
-OSQLMessageBox::OSQLMessageBox(weld::Window* pParent, const SQLExceptionInfo& rException, MessBoxStyle nStyle, const OUString& rHelpURL)
+OSQLMessageBox::OSQLMessageBox(weld::Window* pParent, const SQLExceptionInfo& rException, MessBoxStyle nStyle, OUString sHelpURL)
     : m_pImpl(new SQLMessageBox_Impl(rException))
-    , m_sHelpURL(rHelpURL)
+    , m_sHelpURL(std::move(sHelpURL))
 {
     Construct(pParent, nStyle, AUTO);
 }
diff --git a/dbaccess/source/ui/inc/CollectionView.hxx b/dbaccess/source/ui/inc/CollectionView.hxx
index c80e98f5e858..c49a5fd4fc14 100644
--- a/dbaccess/source/ui/inc/CollectionView.hxx
+++ b/dbaccess/source/ui/inc/CollectionView.hxx
@@ -55,7 +55,7 @@ namespace dbaui
         OCollectionView(weld::Window * pParent,
                         const css::uno::Reference< css::ucb::XContent>& _xContent,
                         const OUString& _sDefaultName,
-                        const css::uno::Reference< css::uno::XComponentContext >& _rxContext);
+                        css::uno::Reference< css::uno::XComponentContext > _xContext);
         virtual ~OCollectionView() override;
         const css::uno::Reference< css::ucb::XContent>& getSelectedFolder() const { return m_xContent;}
         OUString getName() const;
diff --git a/dbaccess/source/ui/inc/ConnectionLine.hxx b/dbaccess/source/ui/inc/ConnectionLine.hxx
index 954c7e02d3a3..360395f94b6e 100644
--- a/dbaccess/source/ui/inc/ConnectionLine.hxx
+++ b/dbaccess/source/ui/inc/ConnectionLine.hxx
@@ -45,7 +45,7 @@ namespace dbaui
         Point                   m_aSourceDescrLinePos,
                                 m_aDestDescrLinePos;
     public:
-        OConnectionLine( OTableConnection* pConn, OConnectionLineDataRef const & pLineData );
+        OConnectionLine( OTableConnection* pConn, OConnectionLineDataRef pLineData );
         OConnectionLine( const OConnectionLine& rLine );
         ~OConnectionLine();
 
diff --git a/dbaccess/source/ui/inc/ConnectionLineData.hxx b/dbaccess/source/ui/inc/ConnectionLineData.hxx
index 10ad0bfddfd9..2a41c93d9aab 100644
--- a/dbaccess/source/ui/inc/ConnectionLineData.hxx
+++ b/dbaccess/source/ui/inc/ConnectionLineData.hxx
@@ -48,7 +48,7 @@ namespace dbaui
         virtual ~OConnectionLineData() override;
     public:
         OConnectionLineData();
-        OConnectionLineData( const OUString& rSourceFieldName, const OUString& rDestFieldName );
+        OConnectionLineData( OUString sSourceFieldName, OUString sDestFieldName );
         OConnectionLineData( const OConnectionLineData& rConnLineData );
         // provide a copy of own instance (this is somehow more acceptable for me compared to a virtual assignment operator
         void CopyFrom(const OConnectionLineData& rSource);
diff --git a/dbaccess/source/ui/inc/SqlNameEdit.hxx b/dbaccess/source/ui/inc/SqlNameEdit.hxx
index 257b18ebbfdb..5b613608830a 100644
--- a/dbaccess/source/ui/inc/SqlNameEdit.hxx
+++ b/dbaccess/source/ui/inc/SqlNameEdit.hxx
@@ -19,6 +19,7 @@
 #pragma once
 
 #include <svtools/editbrowsebox.hxx>
+#include <utility>
 #include <vcl/weld.hxx>
 
 namespace dbaui
@@ -28,8 +29,8 @@ namespace dbaui
         OUString m_sAllowedChars;
         bool        m_bCheck;           // true when we should check for invalid chars
     public:
-        OSQLNameChecker(const OUString& _rAllowedChars)
-            :m_sAllowedChars(_rAllowedChars)
+        OSQLNameChecker(OUString _sAllowedChars)
+            :m_sAllowedChars(std::move(_sAllowedChars))
             ,m_bCheck(true)
         {
         }
diff --git a/dbaccess/source/ui/inc/TableConnection.hxx b/dbaccess/source/ui/inc/TableConnection.hxx
index 3c6ce0fdf76c..a38aa1fe030b 100644
--- a/dbaccess/source/ui/inc/TableConnection.hxx
+++ b/dbaccess/source/ui/inc/TableConnection.hxx
@@ -47,7 +47,7 @@ namespace dbaui
         void clearLineData();
 
     public:
-        OTableConnection( OJoinTableView* pContainer, const TTableConnectionData::value_type& pTabConnData );
+        OTableConnection( OJoinTableView* pContainer, TTableConnectionData::value_type aTabConnData );
         OTableConnection( const OTableConnection& rConn );
         /** destructor
 
diff --git a/dbaccess/source/ui/inc/TableConnectionData.hxx b/dbaccess/source/ui/inc/TableConnectionData.hxx
index 38de2289458f..096a6de70ad3 100644
--- a/dbaccess/source/ui/inc/TableConnectionData.hxx
+++ b/dbaccess/source/ui/inc/TableConnectionData.hxx
@@ -47,8 +47,8 @@ namespace dbaui
         OTableConnectionData& operator=( const OTableConnectionData& rConnData );
     public:
         OTableConnectionData();
-        OTableConnectionData( const TTableWindowData::value_type& _pReferencingTable,
-                              const TTableWindowData::value_type& _pReferencedTable );
+        OTableConnectionData( TTableWindowData::value_type _aReferencingTable,
+                              TTableWindowData::value_type _aReferencedTable );
         OTableConnectionData( const OTableConnectionData& rConnData );
         virtual ~OTableConnectionData();
 
diff --git a/dbaccess/source/ui/inc/TableWindow.hxx b/dbaccess/source/ui/inc/TableWindow.hxx
index c3bed06c75b8..2ccf236d0c98 100644
--- a/dbaccess/source/ui/inc/TableWindow.hxx
+++ b/dbaccess/source/ui/inc/TableWindow.hxx
@@ -112,7 +112,7 @@ namespace dbaui
         */
         void    impl_updateImage();
 
-        OTableWindow( vcl::Window* pParent, const TTableWindowData::value_type& pTabWinData );
+        OTableWindow( vcl::Window* pParent, TTableWindowData::value_type  aTabWinData );
 
     public:
         virtual ~OTableWindow() override;
diff --git a/dbaccess/source/ui/inc/TableWindowData.hxx b/dbaccess/source/ui/inc/TableWindowData.hxx
index f7aea00b9713..33fb4a958100 100644
--- a/dbaccess/source/ui/inc/TableWindowData.hxx
+++ b/dbaccess/source/ui/inc/TableWindowData.hxx
@@ -51,9 +51,9 @@ namespace dbaui
 
     public:
         explicit OTableWindowData(  const css::uno::Reference< css::beans::XPropertySet>& _xTable
-                                   ,const OUString& _rComposedName
-                                   ,const OUString& strTableName
-                                   ,const OUString& rWinName );
+                                   ,OUString _sComposedName
+                                   ,OUString strTableName
+                                   ,OUString sWinName );
         virtual ~OTableWindowData() override;
 
         /** late constructor
diff --git a/dbaccess/source/ui/inc/TokenWriter.hxx b/dbaccess/source/ui/inc/TokenWriter.hxx
index 82e36240fc29..b82bd71a5feb 100644
--- a/dbaccess/source/ui/inc/TokenWriter.hxx
+++ b/dbaccess/source/ui/inc/TokenWriter.hxx
@@ -83,7 +83,7 @@ namespace dbaui
                                 const css::uno::Reference< css::util::XNumberFormatter >& _rxNumberF);
 
         // import data
-        ODatabaseImportExport(  const SharedConnection& _rxConnection,
+        ODatabaseImportExport(  SharedConnection _xConnection,
                                 const css::uno::Reference< css::util::XNumberFormatter >& _rxNumberF,
                                 const css::uno::Reference< css::uno::XComponentContext >& _rM);
 
diff --git a/dbaccess/source/ui/inc/WCopyTable.hxx b/dbaccess/source/ui/inc/WCopyTable.hxx
index 6133112835af..d3c0672f8e0b 100644
--- a/dbaccess/source/ui/inc/WCopyTable.hxx
+++ b/dbaccess/source/ui/inc/WCopyTable.hxx
@@ -183,7 +183,7 @@ namespace dbaui
     public:
         NamedTableCopySource(
             const css::uno::Reference< css::sdbc::XConnection >& _rxConnection,
-            const OUString& _rTableName
+            OUString _sTableName
         );
 
         // ICopyTableSourceObject overridables
@@ -309,7 +309,7 @@ namespace dbaui
         // used for importing rtf/html sources
         OCopyTableWizard(
             weld::Window* pParent,
-            const OUString& _rDefaultName,
+            OUString _sDefaultName,
             sal_Int16 _nOperation,
             ODatabaseExport::TColumns&& _rDestColumns,
             const ODatabaseExport::TColumnVector& _rSourceColVec,
diff --git a/dbaccess/source/ui/inc/charsets.hxx b/dbaccess/source/ui/inc/charsets.hxx
index ed1f88d7beab..4d4454e010d1 100644
--- a/dbaccess/source/ui/inc/charsets.hxx
+++ b/dbaccess/source/ui/inc/charsets.hxx
@@ -72,7 +72,7 @@ namespace dbaui
         const OUString& getDisplayName() const      { return m_sDisplayName; }
 
     private:
-        CharsetDisplayDerefHelper(const ::dbtools::CharsetIteratorDerefHelper& _rBase, const OUString& _rDisplayName);
+        CharsetDisplayDerefHelper(const ::dbtools::CharsetIteratorDerefHelper& _rBase, OUString _sDisplayName);
     };
 
     //- OCharsetDisplay::ExtendedCharsetIterator
@@ -97,7 +97,7 @@ namespace dbaui
         const ExtendedCharsetIterator&  operator++();
 
     protected:
-        ExtendedCharsetIterator( const OCharsetDisplay* _pContainer, const base_iterator& _rPosition );
+        ExtendedCharsetIterator( const OCharsetDisplay* _pContainer, base_iterator _aPosition );
     };
 
 }   // namespace dbaui
diff --git a/dbaccess/source/ui/inc/databaseobjectview.hxx b/dbaccess/source/ui/inc/databaseobjectview.hxx
index d6b23878a8f0..46df99181627 100644
--- a/dbaccess/source/ui/inc/databaseobjectview.hxx
+++ b/dbaccess/source/ui/inc/databaseobjectview.hxx
@@ -91,7 +91,7 @@ namespace dbaui
             const css::uno::Reference< css::uno::XComponentContext >& _rxORB,
             const css::uno::Reference< css::sdb::application::XDatabaseDocumentUI >& _rxApplication,
             const css::uno::Reference< css::frame::XFrame >& _rxParentFrame,
-            const OUString& _rComponentURL
+            OUString _sComponentURL
         );
         virtual ~DatabaseObjectView(){}
 
diff --git a/dbaccess/source/ui/inc/datasourceconnector.hxx b/dbaccess/source/ui/inc/datasourceconnector.hxx
index 541c49607c90..56f7bf2f067b 100644
--- a/dbaccess/source/ui/inc/datasourceconnector.hxx
+++ b/dbaccess/source/ui/inc/datasourceconnector.hxx
@@ -48,7 +48,7 @@ namespace dbaui
         ODatasourceConnector(
             const css::uno::Reference< css::uno::XComponentContext >& _rxContext,
             weld::Window* _pMessageParent,
-            const OUString& _rContextInformation
+            OUString _sContextInformation
         );
 
         /// returns <TRUE/> if the object is able to create data source connections
diff --git a/dbaccess/source/ui/inc/linkeddocuments.hxx b/dbaccess/source/ui/inc/linkeddocuments.hxx
index 961b2ea4cddf..c20f3a87ae51 100644
--- a/dbaccess/source/ui/inc/linkeddocuments.hxx
+++ b/dbaccess/source/ui/inc/linkeddocuments.hxx
@@ -53,7 +53,7 @@ namespace dbaui
             const css::uno::Reference< css::uno::XComponentContext >& _rxContext,
             const css::uno::Reference< css::container::XNameAccess >& _rxContainer,
             const css::uno::Reference< css::sdbc::XConnection>& _xConnection,
-            const OUString& _sDataSourceName
+            OUString _sDataSourceName
         );
         ~OLinkedDocumentsAccess();
 
diff --git a/dbaccess/source/ui/inc/sqlmessage.hxx b/dbaccess/source/ui/inc/sqlmessage.hxx
index a2541dc0fe38..ad6cb25b4d63 100644
--- a/dbaccess/source/ui/inc/sqlmessage.hxx
+++ b/dbaccess/source/ui/inc/sqlmessage.hxx
@@ -89,7 +89,7 @@ public:
         weld::Window* pParent,
         const dbtools::SQLExceptionInfo& _rException,
         MessBoxStyle _nStyle = MessBoxStyle::Ok | MessBoxStyle::DefaultOk,
-        const OUString& _rHelpURL = OUString()
+        OUString _sHelpURL = OUString()
     );
 
     /** display a database related error message
diff --git a/dbaccess/source/ui/inc/unodatbr.hxx b/dbaccess/source/ui/inc/unodatbr.hxx
index 4bb5c26c74a5..093cb8bb6a77 100644
--- a/dbaccess/source/ui/inc/unodatbr.hxx
+++ b/dbaccess/source/ui/inc/unodatbr.hxx
@@ -34,6 +34,7 @@
 #include <comphelper/interfacecontainer2.hxx>
 #include <cppuhelper/implbase5.hxx>
 #include "callbacks.hxx"
+#include <utility>
 #include <vcl/transfer.hxx>
 #include <svx/dataaccessdescriptor.hxx>
 #include "TableCopyHelper.hxx"
@@ -74,7 +75,7 @@ namespace dbaui
             bool                        bEnabled;
 
             ExternalFeature() : bEnabled( false ) { }
-            ExternalFeature( const css::util::URL& _rURL ) : aURL( _rURL ), bEnabled( false ) { }
+            ExternalFeature( css::util::URL _aURL ) : aURL(std::move( _aURL )), bEnabled( false ) { }
         };
 
         typedef std::map< sal_uInt16, ExternalFeature >  ExternalFeaturesMap;
diff --git a/dbaccess/source/ui/misc/TokenWriter.cxx b/dbaccess/source/ui/misc/TokenWriter.cxx
index 974337e8a896..9cbf7db7f5d9 100644
--- a/dbaccess/source/ui/misc/TokenWriter.cxx
+++ b/dbaccess/source/ui/misc/TokenWriter.cxx
@@ -48,6 +48,7 @@
 #include <svtools/htmlout.hxx>
 #include <sfx2/frmhtmlw.hxx>
 #include <svl/numuno.hxx>
+#include <utility>
 #include <vcl/svapp.hxx>
 #include <UITools.hxx>
 #include <toolkit/helper/vclunohelper.hxx>
@@ -95,11 +96,11 @@ ODatabaseImportExport::ODatabaseImportExport(const svx::ODataAccessDescriptor& _
 }
 
 // import data
-ODatabaseImportExport::ODatabaseImportExport( const ::dbtools::SharedConnection& _rxConnection,
+ODatabaseImportExport::ODatabaseImportExport( ::dbtools::SharedConnection _xConnection,
         const Reference< XNumberFormatter >& _rxNumberF, const Reference< XComponentContext >& _rM )
     :m_bBookmarkSelection( false )
     ,m_pStream(nullptr)
-    ,m_xConnection(_rxConnection)
+    ,m_xConnection(std::move(_xConnection))
     ,m_xFormatter(_rxNumberF)
     ,m_xContext(_rM)
     ,m_nCommandType(css::sdb::CommandType::TABLE)
diff --git a/dbaccess/source/ui/misc/WCopyTable.cxx b/dbaccess/source/ui/misc/WCopyTable.cxx
index 64301bccd9c3..cbcf650581f9 100644
--- a/dbaccess/source/ui/misc/WCopyTable.cxx
+++ b/dbaccess/source/ui/misc/WCopyTable.cxx
@@ -56,6 +56,7 @@
 #include <tools/diagnose_ex.h>
 
 #include <algorithm>
+#include <utility>
 
 using namespace ::dbaui;
 using namespace ::com::sun::star::uno;
@@ -250,10 +251,10 @@ OUString ObjectCopySource::getSelectStatement() const
 }
 
 // NamedTableCopySource
-NamedTableCopySource::NamedTableCopySource( const Reference< XConnection >& _rxConnection, const OUString& _rTableName )
+NamedTableCopySource::NamedTableCopySource( const Reference< XConnection >& _rxConnection, OUString _sTableName )
     :m_xConnection( _rxConnection, UNO_SET_THROW )
     ,m_xMetaData( _rxConnection->getMetaData(), UNO_SET_THROW )
-    ,m_sTableName( _rTableName )
+    ,m_sTableName(std::move( _sTableName ))
 {
     ::dbtools::qualifiedNameComponents( m_xMetaData, m_sTableName, m_sTableCatalog, m_sTableSchema, m_sTableBareName, ::dbtools::EComposeRule::Complete );
     impl_ensureColumnInfo_throw();
@@ -583,7 +584,7 @@ weld::Container* OCopyTableWizard::CreatePageContainer()
     return pPageContainer;
 }
 
-OCopyTableWizard::OCopyTableWizard( weld::Window* pParent, const OUString& _rDefaultName, sal_Int16 _nOperation,
+OCopyTableWizard::OCopyTableWizard( weld::Window* pParent, OUString _sDefaultName, sal_Int16 _nOperation,
         ODatabaseExport::TColumns&& _rSourceColumns, const ODatabaseExport::TColumnVector& _rSourceColVec,
         const Reference< XConnection >& _xConnection, const Reference< XNumberFormatter >&  _xFormatter,
         TypeSelectionPageFactory _pTypeSelectionPageFactory, SvStream& _rTypeSelectionPageArg, const Reference< XComponentContext >& _rxContext )
@@ -598,7 +599,7 @@ OCopyTableWizard::OCopyTableWizard( weld::Window* pParent, const OUString& _rDef
     , m_nPageCount(0)
     , m_bDeleteSourceColumns(false)
     , m_bInterConnectionCopy( false )
-    , m_sName(_rDefaultName)
+    , m_sName(std::move(_sDefaultName))
     , m_nOperation( _nOperation )
     , m_ePressed( WIZARD_NONE )
     , m_bCreatePrimaryKeyColumn(false)
diff --git a/dbaccess/source/ui/misc/charsets.cxx b/dbaccess/source/ui/misc/charsets.cxx
index 546d21cb0859..3cec6284511e 100644
--- a/dbaccess/source/ui/misc/charsets.cxx
+++ b/dbaccess/source/ui/misc/charsets.cxx
@@ -23,6 +23,7 @@
 #include <strings.hrc>
 #include <rtl/tencinfo.h>
 #include <svx/txenctab.hxx>
+#include <utility>
 
 namespace dbaui
 {
@@ -86,17 +87,17 @@ namespace dbaui
     {
     }
 
-    CharsetDisplayDerefHelper::CharsetDisplayDerefHelper(const ::dbtools::CharsetIteratorDerefHelper& _rBase, const OUString& _rDisplayName)
+    CharsetDisplayDerefHelper::CharsetDisplayDerefHelper(const ::dbtools::CharsetIteratorDerefHelper& _rBase, OUString  _sDisplayName)
         :CharsetDisplayDerefHelper_Base(_rBase)
-        ,m_sDisplayName(_rDisplayName)
+        ,m_sDisplayName(std::move(_sDisplayName))
     {
         OSL_ENSURE( !m_sDisplayName.isEmpty(), "CharsetDisplayDerefHelper::CharsetDisplayDerefHelper: invalid display name!" );
     }
 
     // OCharsetDisplay::ExtendedCharsetIterator
-    OCharsetDisplay::ExtendedCharsetIterator::ExtendedCharsetIterator( const OCharsetDisplay* _pContainer, const base_iterator& _rPosition )
+    OCharsetDisplay::ExtendedCharsetIterator::ExtendedCharsetIterator( const OCharsetDisplay* _pContainer, base_iterator _aPosition )
         :m_pContainer(_pContainer)
-        ,m_aPosition(_rPosition)
+        ,m_aPosition(std::move(_aPosition))
     {
         OSL_ENSURE(m_pContainer, "OCharsetDisplay::ExtendedCharsetIterator::ExtendedCharsetIterator : invalid container!");
     }
diff --git a/dbaccess/source/ui/misc/databaseobjectview.cxx b/dbaccess/source/ui/misc/databaseobjectview.cxx
index cb4478295d04..32fe0440cb25 100644
--- a/dbaccess/source/ui/misc/databaseobjectview.cxx
+++ b/dbaccess/source/ui/misc/databaseobjectview.cxx
@@ -32,6 +32,7 @@
 #include <osl/diagnose.h>
 #include <toolkit/helper/vclunohelper.hxx>
 #include <tools/diagnose_ex.h>
+#include <utility>
 #include <vcl/window.hxx>
 
 namespace dbaui
@@ -51,11 +52,11 @@ namespace dbaui
     DatabaseObjectView::DatabaseObjectView( const Reference< XComponentContext >& _rxORB,
             const Reference< XDatabaseDocumentUI >& _rxApplication,
             const Reference< XFrame >& _rxParentFrame,
-            const OUString& _rComponentURL )
+            OUString _sComponentURL )
         :m_xORB             ( _rxORB            )
         ,m_xParentFrame     ( _rxParentFrame    )
         ,m_xApplication     ( _rxApplication    )
-        ,m_sComponentURL    ( _rComponentURL    )
+        ,m_sComponentURL    (std::move( _sComponentURL    ))
     {
         OSL_ENSURE( m_xORB.is(), "DatabaseObjectView::DatabaseObjectView: invalid service factory!" );
         OSL_ENSURE( m_xApplication.is(), "DatabaseObjectView::DatabaseObjectView: invalid connection!" );
diff --git a/dbaccess/source/ui/misc/datasourceconnector.cxx b/dbaccess/source/ui/misc/datasourceconnector.cxx
index 49053e569000..b615d516384b 100644
--- a/dbaccess/source/ui/misc/datasourceconnector.cxx
+++ b/dbaccess/source/ui/misc/datasourceconnector.cxx
@@ -30,6 +30,7 @@
 #include <connectivity/dbexception.hxx>
 #include <com/sun/star/sdbc/XDataSource.hpp>
 #include <UITools.hxx>
+#include <utility>
 #include <vcl/outdev.hxx>
 #include <vcl/stdtext.hxx>
 #include <vcl/weld.hxx>
@@ -59,10 +60,10 @@ namespace dbaui
     }
 
     ODatasourceConnector::ODatasourceConnector( const Reference< XComponentContext >& _rxContext, weld::Window* _pMessageParent,
-        const OUString& _rContextInformation )
+        OUString _sContextInformation )
         :m_pErrorMessageParent(_pMessageParent)
         ,m_xContext(_rxContext)
-        ,m_sContextInformation( _rContextInformation )
+        ,m_sContextInformation(std::move( _sContextInformation ))
     {
     }
 
diff --git a/dbaccess/source/ui/misc/dsmeta.cxx b/dbaccess/source/ui/misc/dsmeta.cxx
index 286ce63aace2..402bdd7bfa34 100644
--- a/dbaccess/source/ui/misc/dsmeta.cxx
+++ b/dbaccess/source/ui/misc/dsmeta.cxx
@@ -144,7 +144,7 @@ namespace dbaui
     class DataSourceMetaData_Impl
     {
     public:
-        explicit DataSourceMetaData_Impl(const OUString& rURL);
+        explicit DataSourceMetaData_Impl(OUString sURL);
 
         const OUString& getType() const { return m_sURL; }
 
@@ -152,8 +152,8 @@ namespace dbaui
         const OUString m_sURL;
     };
 
-    DataSourceMetaData_Impl::DataSourceMetaData_Impl( const OUString& _sURL )
-        :m_sURL( _sURL )
+    DataSourceMetaData_Impl::DataSourceMetaData_Impl( OUString _sURL )
+        :m_sURL(std::move( _sURL ))
     {
     }
 
diff --git a/dbaccess/source/ui/misc/linkeddocuments.cxx b/dbaccess/source/ui/misc/linkeddocuments.cxx
index 192331ced226..ce0acbd7cb2a 100644
--- a/dbaccess/source/ui/misc/linkeddocuments.cxx
+++ b/dbaccess/source/ui/misc/linkeddocuments.cxx
@@ -38,6 +38,7 @@
 #include <browserids.hxx>
 #include <com/sun/star/container/XHierarchicalNameContainer.hpp>
 #include <comphelper/mimeconfighelper.hxx>
+#include <utility>
 #include <vcl/weld.hxx>
 
 #include <cppuhelper/exc_hlp.hxx>
@@ -88,13 +89,13 @@ namespace dbaui
     // OLinkedDocumentsAccess
     OLinkedDocumentsAccess::OLinkedDocumentsAccess( weld::Window* pDialogParent, const Reference< XDatabaseDocumentUI >& i_rDocumentUI,
         const Reference< XComponentContext >& _rxContext, const Reference< XNameAccess >& _rxContainer,
-        const Reference< XConnection>& _xConnection, const OUString& _sDataSourceName )
+        const Reference< XConnection>& _xConnection, OUString _sDataSourceName )
         :m_xContext(_rxContext)
         ,m_xDocumentContainer(_rxContainer)
         ,m_xConnection(_xConnection)
         ,m_xDocumentUI( i_rDocumentUI )
         ,m_pDialogParent(pDialogParent)
-        ,m_sDataSourceName(_sDataSourceName)
+        ,m_sDataSourceName(std::move(_sDataSourceName))
     {
         OSL_ENSURE(m_xContext.is(), "OLinkedDocumentsAccess::OLinkedDocumentsAccess: invalid service factory!");
         assert(m_pDialogParent && "OLinkedDocumentsAccess::OLinkedDocumentsAccess: really need a dialog parent!");
diff --git a/dbaccess/source/ui/querydesign/ConnectionLine.cxx b/dbaccess/source/ui/querydesign/ConnectionLine.cxx
index 77a67004e995..b22a05424df3 100644
--- a/dbaccess/source/ui/querydesign/ConnectionLine.cxx
+++ b/dbaccess/source/ui/querydesign/ConnectionLine.cxx
@@ -22,6 +22,7 @@
 #include <TableWindow.hxx>
 #include <TableWindowListBox.hxx>
 #include <TableConnection.hxx>
+#include <utility>
 #include <vcl/svapp.hxx>
 #include <math.h>
 #include <osl/diagnose.h>
@@ -121,9 +122,9 @@ namespace
     }
 }
 
-OConnectionLine::OConnectionLine( OTableConnection* _pConn, OConnectionLineDataRef const & _pLineData )
+OConnectionLine::OConnectionLine( OTableConnection* _pConn, OConnectionLineDataRef _pLineData )
     : m_pTabConn( _pConn )
-    , m_pData( _pLineData )
+    , m_pData(std::move( _pLineData ))
 {
 }
 
diff --git a/dbaccess/source/ui/querydesign/ConnectionLineData.cxx b/dbaccess/source/ui/querydesign/ConnectionLineData.cxx
index db7177294e64..8267cc3ed8c2 100644
--- a/dbaccess/source/ui/querydesign/ConnectionLineData.cxx
+++ b/dbaccess/source/ui/querydesign/ConnectionLineData.cxx
@@ -18,15 +18,16 @@
  */
 
 #include <ConnectionLineData.hxx>
+#include <utility>
 
 using namespace dbaui;
 OConnectionLineData::OConnectionLineData()
 {
 }
 
-OConnectionLineData::OConnectionLineData( const OUString& rSourceFieldName, const OUString& rDestFieldName )
-    :m_aSourceFieldName( rSourceFieldName )
-    ,m_aDestFieldName( rDestFieldName )
+OConnectionLineData::OConnectionLineData( OUString sSourceFieldName, OUString sDestFieldName )
+    :m_aSourceFieldName(std::move( sSourceFieldName ))
+    ,m_aDestFieldName(std::move( sDestFieldName ))
 {
 }
 
diff --git a/dbaccess/source/ui/querydesign/TableConnection.cxx b/dbaccess/source/ui/querydesign/TableConnection.cxx
index ada4b5990890..2bfceb1b6c69 100644
--- a/dbaccess/source/ui/querydesign/TableConnection.cxx
+++ b/dbaccess/source/ui/querydesign/TableConnection.cxx
@@ -21,6 +21,7 @@
 #include <ConnectionLine.hxx>
 #include <TableConnectionData.hxx>
 #include <JoinTableView.hxx>
+#include <utility>
 
 using namespace dbaui;
 using namespace comphelper;
@@ -29,9 +30,9 @@ using namespace ::com::sun::star::accessibility;
 
 namespace dbaui
 {
-    OTableConnection::OTableConnection( OJoinTableView* _pContainer,const TTableConnectionData::value_type& _pTabConnData )
+    OTableConnection::OTableConnection( OJoinTableView* _pContainer, TTableConnectionData::value_type  _aTabConnData )
         :Window(_pContainer)
-        ,m_pData( _pTabConnData )
+        ,m_pData(std::move( _aTabConnData ))
         ,m_pParent( _pContainer )
         ,m_bSelected( false )
     {
diff --git a/dbaccess/source/ui/querydesign/TableConnectionData.cxx b/dbaccess/source/ui/querydesign/TableConnectionData.cxx
index 107d8a9d1a48..aa524a5ac7f9 100644
--- a/dbaccess/source/ui/querydesign/TableConnectionData.cxx
+++ b/dbaccess/source/ui/querydesign/TableConnectionData.cxx
@@ -18,6 +18,7 @@
  */
 
 #include <TableConnectionData.hxx>
+#include <utility>
 #include <osl/diagnose.h>
 
 using namespace dbaui;
@@ -27,10 +28,10 @@ OTableConnectionData::OTableConnectionData()
     Init();
 }
 
-OTableConnectionData::OTableConnectionData(const TTableWindowData::value_type& _pReferencingTable
-                                          ,const TTableWindowData::value_type& _pReferencedTable )
- :m_pReferencingTable(_pReferencingTable)
- ,m_pReferencedTable(_pReferencedTable)
+OTableConnectionData::OTableConnectionData(TTableWindowData::value_type _pReferencingTable
+                                          ,TTableWindowData::value_type _pReferencedTable )
+ :m_pReferencingTable(std::move(_pReferencingTable))
+ ,m_pReferencedTable(std::move(_pReferencedTable))
 {
     Init();
 }
diff --git a/dbaccess/source/ui/querydesign/TableWindow.cxx b/dbaccess/source/ui/querydesign/TableWindow.cxx
index b8afc951d2d8..24793a92fb45 100644
--- a/dbaccess/source/ui/querydesign/TableWindow.cxx
+++ b/dbaccess/source/ui/querydesign/TableWindow.cxx
@@ -25,6 +25,7 @@
 #include <JoinTableView.hxx>
 #include <JoinDesignView.hxx>
 #include <osl/diagnose.h>
+#include <utility>
 #include <vcl/svapp.hxx>
 #include <vcl/settings.hxx>
 #include <vcl/commandevent.hxx>
@@ -86,11 +87,11 @@ void Draw3DBorder(vcl::RenderContext& rRenderContext, const tools::Rectangle& rR
 
 }
 
-OTableWindow::OTableWindow( vcl::Window* pParent, const TTableWindowData::value_type& pTabWinData )
+OTableWindow::OTableWindow( vcl::Window* pParent, TTableWindowData::value_type pTabWinData )
     : ::comphelper::OContainerListener(m_aMutex)
     , Window( pParent, WB_3DLOOK|WB_MOVEABLE )
     , m_xTitle( VclPtr<OTableWindowTitle>::Create(this) )
-    , m_pData( pTabWinData )
+    , m_pData(std::move( pTabWinData ))
     , m_nMoveCount(0)
     , m_nMoveIncrement(1)
     , m_nSizingFlags( SizingFlags::NONE )
diff --git a/dbaccess/source/ui/querydesign/TableWindowData.cxx b/dbaccess/source/ui/querydesign/TableWindowData.cxx
index d07c8214a17c..3cbead6e406c 100644
--- a/dbaccess/source/ui/querydesign/TableWindowData.cxx
+++ b/dbaccess/source/ui/querydesign/TableWindowData.cxx
@@ -26,6 +26,7 @@
 #include <com/sun/star/container/XNameAccess.hpp>
 #include <com/sun/star/container/XIndexAccess.hpp>
 #include <com/sun/star/lang/XComponent.hpp>
+#include <utility>
 
 using namespace dbaui;
 using namespace ::com::sun::star::lang;
@@ -37,13 +38,13 @@ using namespace ::com::sun::star::beans;
 using namespace ::com::sun::star::container;
 
 OTableWindowData::OTableWindowData( const Reference< XPropertySet>& _xTable
-                                   ,const OUString& _rComposedName
-                                   ,const OUString& rTableName
-                                   ,const OUString& rWinName )
+                                   ,OUString _sComposedName
+                                   ,OUString sTableName
+                                   ,OUString sWinName )
     :m_xTable(_xTable)
-    ,m_aTableName( rTableName )
-    ,m_aWinName( rWinName )
-    ,m_sComposedName(_rComposedName)
+    ,m_aTableName(std::move( sTableName ))
+    ,m_aWinName(std::move( sWinName ))
+    ,m_sComposedName(std::move(_sComposedName))
     ,m_aPosition( Point(-1,-1) )
     ,m_aSize( Size(-1,-1) )
     ,m_bShowAll( true )
diff --git a/dbaccess/source/ui/querydesign/querycontroller.cxx b/dbaccess/source/ui/querydesign/querycontroller.cxx
index 230cb7f6eac8..677c44787e07 100644
--- a/dbaccess/source/ui/querydesign/querycontroller.cxx
+++ b/dbaccess/source/ui/querydesign/querycontroller.cxx
@@ -64,6 +64,7 @@
 #include <toolkit/helper/vclunohelper.hxx>
 #include <tools/diagnose_ex.h>
 #include <osl/diagnose.h>
+#include <utility>
 #include <vcl/stdtext.hxx>
 #include <vcl/svapp.hxx>
 #include <vcl/weld.hxx>
@@ -1381,8 +1382,8 @@ struct CommentStrip
 {
     OUString maComment;
     bool            mbLastOnLine;
-    CommentStrip( const OUString& rComment, bool bLastOnLine )
-        : maComment( rComment), mbLastOnLine( bLastOnLine) {}
+    CommentStrip( OUString sComment, bool bLastOnLine )
+        : maComment(std::move( sComment)), mbLastOnLine( bLastOnLine) {}
 };
 
 }
diff --git a/dbaccess/source/ui/tabledesign/TableUndo.cxx b/dbaccess/source/ui/tabledesign/TableUndo.cxx
index d60756a2e697..3769e5fd08cb 100644
--- a/dbaccess/source/ui/tabledesign/TableUndo.cxx
+++ b/dbaccess/source/ui/tabledesign/TableUndo.cxx
@@ -25,6 +25,7 @@
 #include <TableDesignView.hxx>
 #include <FieldDescriptions.hxx>
 #include <svx/svxids.hrc>
+#include <utility>
 
 using namespace dbaui;
 using namespace ::svt;
@@ -116,11 +117,11 @@ OTableEditorUndoAct::~OTableEditorUndoAct()
 {
 }
 
-OTableEditorTypeSelUndoAct::OTableEditorTypeSelUndoAct( OTableEditorCtrl* pOwner, sal_Int32 nRowID, sal_uInt16 nColumn, const TOTypeInfoSP& _pOldType )
+OTableEditorTypeSelUndoAct::OTableEditorTypeSelUndoAct( OTableEditorCtrl* pOwner, sal_Int32 nRowID, sal_uInt16 nColumn, TOTypeInfoSP _pOldType )
     :OTableEditorUndoAct( pOwner ,STR_TABED_UNDO_TYPE_CHANGED)
     ,m_nCol( nColumn )
     ,m_nRow( nRowID )
-    ,m_pOldType( _pOldType )
+    ,m_pOldType(std::move( _pOldType ))
 {
 }
 
diff --git a/dbaccess/source/ui/tabledesign/TableUndo.hxx b/dbaccess/source/ui/tabledesign/TableUndo.hxx
index 5ad23b84fb0f..1863555f7556 100644
--- a/dbaccess/source/ui/tabledesign/TableUndo.hxx
+++ b/dbaccess/source/ui/tabledesign/TableUndo.hxx
@@ -78,7 +78,7 @@ namespace dbaui
         virtual void    Undo() override;
         virtual void    Redo() override;
     public:
-        OTableEditorTypeSelUndoAct( OTableEditorCtrl* pOwner, sal_Int32 nRowID, sal_uInt16 nColumn, const TOTypeInfoSP& _pOldType );
+        OTableEditorTypeSelUndoAct( OTableEditorCtrl* pOwner, sal_Int32 nRowID, sal_uInt16 nColumn, TOTypeInfoSP _pOldType );
         virtual ~OTableEditorTypeSelUndoAct() override;
     };