diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-06-23 15:06:10 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-06-23 18:24:16 +0200 |
| commit | 74ff1a4355090c227f4bdcdddb20482a4abd6064 (patch) | |
| tree | 06ace5d2a3d8e5eb2a8d08c9835350f69d870e92 /package/source/xstor/xstorage.cxx | |
| parent | c7c2f6b37a834a3135e3683f1fc27f3ec938640c (diff) | |
clang-tidy modernize-pass-by-value in package
Change-Id: Id12d7b38d278c9fb18b30c6d921713a53168b048
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136337
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'package/source/xstor/xstorage.cxx')
| -rw-r--r-- | package/source/xstor/xstorage.cxx | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/package/source/xstor/xstorage.cxx b/package/source/xstor/xstorage.cxx index bdcd699a7eb7..bed733b10532 100644 --- a/package/source/xstor/xstorage.cxx +++ b/package/source/xstor/xstorage.cxx @@ -55,6 +55,7 @@ #include <comphelper/servicehelper.hxx> #include <comphelper/storagehelper.hxx> #include <comphelper/ofopxmlhelper.hxx> +#include <utility> #include <tools/diagnose_ex.h> #include "xstorage.hxx" @@ -127,8 +128,8 @@ static uno::Reference< io::XInputStream > GetSeekableTempCopy( const uno::Refere return xTempIn; } -SotElement_Impl::SotElement_Impl(const OUString& rName, bool bStor, bool bNew) - : m_aOriginalName(rName) +SotElement_Impl::SotElement_Impl(OUString aName, bool bStor, bool bNew) + : m_aOriginalName(std::move(aName)) , m_bIsRemoved(false) , m_bIsInserted(bNew) , m_bIsStorage(bStor) @@ -220,7 +221,7 @@ OStorage_Impl::OStorage_Impl( uno::Reference< io::XStream > const & xStream, OStorage_Impl::OStorage_Impl( OStorage_Impl* pParent, sal_Int32 nMode, uno::Reference< container::XNameContainer > const & xPackageFolder, - uno::Reference< lang::XSingleServiceFactory > const & xPackage, + uno::Reference< lang::XSingleServiceFactory > xPackage, uno::Reference< uno::XComponentContext > const & xContext, sal_Int32 nStorageType ) : m_xMutex( new comphelper::RefCountedMutex ) @@ -233,7 +234,7 @@ OStorage_Impl::OStorage_Impl( OStorage_Impl* pParent, , m_bListCreated( false ) , m_nModifiedListenerCount( 0 ) , m_xPackageFolder( xPackageFolder ) -, m_xPackage( xPackage ) +, m_xPackage(std::move( xPackage )) , m_xContext( xContext ) , m_bHasCommonEncryptionData( false ) , m_pParent( pParent ) // can be empty in case of temporary readonly substorages and relation storage |