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

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

7 files changed, 11 insertions, 11 deletions

diff --git a/include/unotools/charclass.hxx b/include/unotools/charclass.hxx
index ab695beb52cb..c76c7ae35f0b 100644
--- a/include/unotools/charclass.hxx
+++ b/include/unotools/charclass.hxx
@@ -69,11 +69,11 @@ public:
     /// Preferred ctor with service manager specified
     CharClass(
         const css::uno::Reference< css::uno::XComponentContext > & rxContext,
-        const LanguageTag& rLanguageTag );
+        LanguageTag aLanguageTag );
 
     /// Deprecated ctor, tries to get a process service manager or to load the
     /// library directly.
-    CharClass( const LanguageTag& rLanguageTag );
+    CharClass( LanguageTag aLanguageTag );
 
     ~CharClass();
 
diff --git a/include/unotools/configitem.hxx b/include/unotools/configitem.hxx
index 63bd38dce6b4..7692bd3d6c9e 100644
--- a/include/unotools/configitem.hxx
+++ b/include/unotools/configitem.hxx
@@ -109,7 +109,7 @@ namespace utl
             virtual void            ImplCommit() = 0;
 
         protected:
-            explicit ConfigItem(const OUString &rSubTree,
+            explicit ConfigItem(OUString aSubTree,
                         ConfigItemMode nMode = ConfigItemMode::NONE);
 
             void                    SetModified  (); // mark item as modified
diff --git a/include/unotools/intlwrapper.hxx b/include/unotools/intlwrapper.hxx
index 49d1f8bf5438..e86346a9552e 100644
--- a/include/unotools/intlwrapper.hxx
+++ b/include/unotools/intlwrapper.hxx
@@ -61,7 +61,7 @@ private:
     void                ImplNewCollator( bool bCaseSensitive ) const;
 
 public:
-    IntlWrapper(const LanguageTag& rLanguageTag);
+    IntlWrapper(LanguageTag aLanguageTag);
     ~IntlWrapper();
 
     const LocaleDataWrapper*    getLocaleData() const
diff --git a/include/unotools/localedatawrapper.hxx b/include/unotools/localedatawrapper.hxx
index 2c408b4a0cf9..5fbdcd0782b7 100644
--- a/include/unotools/localedatawrapper.hxx
+++ b/include/unotools/localedatawrapper.hxx
@@ -118,14 +118,14 @@ class UNOTOOLS_DLLPUBLIC LocaleDataWrapper
 public:
     LocaleDataWrapper(
         const css::uno::Reference< css::uno::XComponentContext > & rxContext,
-        const LanguageTag& rLanguageTag
+        LanguageTag aLanguageTag
         );
     /**
         @param rOverrideDateAcceptancePatterns Override locale's date acceptance patterns.
             An empty sequence resets the patterns to the locale's pattern sequence.
      */
     LocaleDataWrapper(
-        const LanguageTag& rLanguageTag,
+        LanguageTag aLanguageTag,
         const std::vector<OUString> & rOverrideDateAcceptancePatterns = {}
         );
     ~LocaleDataWrapper();
diff --git a/include/unotools/progresshandlerwrap.hxx b/include/unotools/progresshandlerwrap.hxx
index 207ba2af8c1b..4fcfd36da044 100644
--- a/include/unotools/progresshandlerwrap.hxx
+++ b/include/unotools/progresshandlerwrap.hxx
@@ -38,7 +38,7 @@ class UNLESS_MERGELIBS(UNOTOOLS_DLLPUBLIC) ProgressHandlerWrap final
     css::uno::Reference<css::task::XStatusIndicator> m_xStatusIndicator;
 
 public:
-    ProgressHandlerWrap(css::uno::Reference<css::task::XStatusIndicator> const& xSI);
+    ProgressHandlerWrap(css::uno::Reference<css::task::XStatusIndicator> xSI);
 
     // XProgressHandler
     virtual void SAL_CALL push(const css::uno::Any& Status) override;
diff --git a/include/unotools/streamhelper.hxx b/include/unotools/streamhelper.hxx
index 5d0a3792855b..a727869bfea6 100644
--- a/include/unotools/streamhelper.hxx
+++ b/include/unotools/streamhelper.hxx
@@ -26,6 +26,7 @@
 #include <cppuhelper/implbase.hxx>
 #include <tools/stream.hxx>
 #include <mutex>
+#include <utility>
 
 namespace utl
 {
@@ -41,9 +42,9 @@ class UNOTOOLS_DLLPUBLIC OInputStreamHelper final : public cppu::WeakImplHelper<
     sal_Int32       m_nAvailable;   // this is typically the chunk(buffer) size
 
 public:
-    OInputStreamHelper(const SvLockBytesRef& _xLockBytes,
+    OInputStreamHelper(SvLockBytesRef _xLockBytes,
                        sal_uInt32 _nAvailable)
-        :m_xLockBytes(_xLockBytes)
+        :m_xLockBytes(std::move(_xLockBytes))
         ,m_nActPos(0)
         ,m_nAvailable(_nAvailable){}
 
diff --git a/include/unotools/viewoptions.hxx b/include/unotools/viewoptions.hxx
index 734c9cada7bb..273e38817c58 100644
--- a/include/unotools/viewoptions.hxx
+++ b/include/unotools/viewoptions.hxx
@@ -113,8 +113,7 @@ class SAL_WARN_UNUSED UNOTOOLS_DLLPUBLIC SvtViewOptions final
             @onerror    An assertion is thrown in debug version. Otherwise we do nothing!
         *//*-*****************************************************************************************************/
 
-         SvtViewOptions(       EViewType        eType     ,
-                         const OUString& sViewName );
+         SvtViewOptions( EViewType eType, OUString sViewName );
 
         //  interface