diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-17 10:55:36 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-17 13:33:48 +0200 |
commit | f71a35500088ba332f21896f373de0fa903e7bcc (patch) | |
tree | 92206ccbb46be2098ae37e069b4fa55f8b79a9fd /unotools | |
parent | 49bc18c0472b4c7758bfcbdf55383de9e9ef3027 (diff) |
loplugin:constparams in configmgr,unotools
Change-Id: Ief3df5487eb20b1f6d0e8aed89cfef4464d23a88
Reviewed-on: https://gerrit.libreoffice.org/40048
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools')
-rw-r--r-- | unotools/source/config/bootstrap.cxx | 16 | ||||
-rw-r--r-- | unotools/source/config/configitem.cxx | 2 | ||||
-rw-r--r-- | unotools/source/config/configmgr.cxx | 2 | ||||
-rw-r--r-- | unotools/source/config/configvaluecontainer.cxx | 4 | ||||
-rw-r--r-- | unotools/source/config/options.cxx | 2 | ||||
-rw-r--r-- | unotools/source/misc/ZipPackageHelper.cxx | 6 | ||||
-rw-r--r-- | unotools/source/misc/closeveto.cxx | 2 | ||||
-rw-r--r-- | unotools/source/misc/desktopterminationobserver.cxx | 2 |
8 files changed, 18 insertions, 18 deletions
diff --git a/unotools/source/config/bootstrap.cxx b/unotools/source/config/bootstrap.cxx index 51d678b3ed69..7934e171e326 100644 --- a/unotools/source/config/bootstrap.cxx +++ b/unotools/source/config/bootstrap.cxx @@ -108,8 +108,8 @@ public: // construction and initialization const OUString& getImplName() const { return m_aImplName; } private: // implementation - bool initBaseInstallationData(rtl::Bootstrap& _rData); - bool initUserInstallationData(rtl::Bootstrap& _rData); + bool initBaseInstallationData(rtl::Bootstrap const & _rData); + bool initUserInstallationData(rtl::Bootstrap const & _rData); }; namespace @@ -312,7 +312,7 @@ static PathStatus getDerivedPath( OUString& _rURL, OUString const& _aBaseURL, PathStatus _aBaseStatus, OUString const& _sRelativeURL, - rtl::Bootstrap& _rData, OUString const& _sBootstrapParameter + rtl::Bootstrap const & _rData, OUString const& _sBootstrapParameter ) { OUString sDerivedURL; @@ -358,7 +358,7 @@ static inline PathStatus getDerivedPath( OUString& _rURL, Bootstrap::Impl::PathData const& _aBaseData, OUString const& _sRelativeURL, - rtl::Bootstrap& _rData, OUString const& _sBootstrapParameter + rtl::Bootstrap const & _rData, OUString const& _sBootstrapParameter ) { return getDerivedPath(_rURL,_aBaseData.path,_aBaseData.status,_sRelativeURL,_rData,_sBootstrapParameter); @@ -394,14 +394,14 @@ static inline Bootstrap::PathStatus updateStatus(Bootstrap::Impl::PathData & _rR return _rResult.status = checkStatusAndNormalizeURL(_rResult.path); } -static Bootstrap::PathStatus implGetBootstrapFile(rtl::Bootstrap& _rData, Bootstrap::Impl::PathData & _rBootstrapFile) +static Bootstrap::PathStatus implGetBootstrapFile(rtl::Bootstrap const & _rData, Bootstrap::Impl::PathData & _rBootstrapFile) { _rData.getIniName(_rBootstrapFile.path); return updateStatus(_rBootstrapFile); } -static Bootstrap::PathStatus implGetVersionFile(rtl::Bootstrap& _rData, Bootstrap::Impl::PathData & _rVersionFile) +static Bootstrap::PathStatus implGetVersionFile(rtl::Bootstrap const & _rData, Bootstrap::Impl::PathData & _rVersionFile) { _rData.getFrom(BOOTSTRAP_ITEM_VERSIONFILE, _rVersionFile.path); @@ -648,7 +648,7 @@ Bootstrap::Status Bootstrap::checkBootstrapStatus(OUString& _rDiagnosticMessage, // class Bootstrap::Impl -bool Bootstrap::Impl::initBaseInstallationData(rtl::Bootstrap& _rData) +bool Bootstrap::Impl::initBaseInstallationData(rtl::Bootstrap const & _rData) { OUString const csBaseInstallItem( BOOTSTRAP_ITEM_BASEINSTALLATION ); OUString const csBaseInstallDefault( BOOTSTRAP_DEFAULT_BASEINSTALL ); @@ -662,7 +662,7 @@ bool Bootstrap::Impl::initBaseInstallationData(rtl::Bootstrap& _rData) return bResult; } -bool Bootstrap::Impl::initUserInstallationData(rtl::Bootstrap& _rData) +bool Bootstrap::Impl::initUserInstallationData(rtl::Bootstrap const & _rData) { OUString const csUserInstallItem( BOOTSTRAP_ITEM_USERINSTALLATION ); diff --git a/unotools/source/config/configitem.cxx b/unotools/source/config/configitem.cxx index 3d290573699b..d0d17510ebe4 100644 --- a/unotools/source/config/configitem.cxx +++ b/unotools/source/config/configitem.cxx @@ -698,7 +698,7 @@ bool ConfigItem::ClearNodeSet(const OUString& rNode) return bRet; } -bool ConfigItem::ClearNodeElements(const OUString& rNode, Sequence< OUString >& rElements) +bool ConfigItem::ClearNodeElements(const OUString& rNode, Sequence< OUString > const & rElements) { ValueCounter_Impl aCounter(m_nInValueChange); bool bRet = false; diff --git a/unotools/source/config/configmgr.cxx b/unotools/source/config/configmgr.cxx index 959fc772684d..f8b5e891cb02 100644 --- a/unotools/source/config/configmgr.cxx +++ b/unotools/source/config/configmgr.cxx @@ -145,7 +145,7 @@ utl::ConfigManager & utl::ConfigManager::getConfigManager() { } css::uno::Reference< css::container::XHierarchicalNameAccess > -utl::ConfigManager::acquireTree(utl::ConfigItem & item) { +utl::ConfigManager::acquireTree(utl::ConfigItem const & item) { css::uno::Sequence< css::uno::Any > args(1); args[0] <<= css::beans::NamedValue( "nodepath", diff --git a/unotools/source/config/configvaluecontainer.cxx b/unotools/source/config/configvaluecontainer.cxx index d19c87c7b9c5..22d7d98d9285 100644 --- a/unotools/source/config/configvaluecontainer.cxx +++ b/unotools/source/config/configvaluecontainer.cxx @@ -160,7 +160,7 @@ namespace utl public: UpdateFromConfig( const OConfigurationNode& _rRootNode, ::osl::Mutex& _rMutex ) : SubNodeAccess( _rRootNode, _rMutex ) { } - void operator() ( NodeValueAccessor& _rAccessor ) + void operator() ( NodeValueAccessor const & _rAccessor ) { ::utl::lcl_copyData( _rAccessor, m_rRootNode.getNodeValue( _rAccessor.getPath( ) ), m_rMutex ); } @@ -171,7 +171,7 @@ namespace utl public: UpdateToConfig( const OConfigurationNode& _rRootNode, ::osl::Mutex& _rMutex ) : SubNodeAccess( _rRootNode, _rMutex ) { } - void operator() ( NodeValueAccessor& _rAccessor ) + void operator() ( NodeValueAccessor const & _rAccessor ) { Any aNewValue; lcl_copyData( aNewValue, _rAccessor, m_rMutex ); diff --git a/unotools/source/config/options.cxx b/unotools/source/config/options.cxx index 593c431580d2..292b254649d5 100644 --- a/unotools/source/config/options.cxx +++ b/unotools/source/config/options.cxx @@ -50,7 +50,7 @@ void ConfigurationBroadcaster::AddListener( utl::ConfigurationListener* pListene mpList->push_back( pListener ); } -void ConfigurationBroadcaster::RemoveListener( utl::ConfigurationListener* pListener ) +void ConfigurationBroadcaster::RemoveListener( utl::ConfigurationListener const * pListener ) { if ( mpList ) { for ( IMPL_ConfigurationListenerList::iterator it = mpList->begin(); diff --git a/unotools/source/misc/ZipPackageHelper.cxx b/unotools/source/misc/ZipPackageHelper.cxx index 7d03b43a9279..6d532e37b171 100644 --- a/unotools/source/misc/ZipPackageHelper.cxx +++ b/unotools/source/misc/ZipPackageHelper.cxx @@ -100,7 +100,7 @@ Reference< XInterface >& ZipPackageHelper::getRootFolder() return mxRootFolder; } -Reference< XInterface > ZipPackageHelper::addFolder( Reference< XInterface >& xRootFolder, +Reference< XInterface > ZipPackageHelper::addFolder( Reference< XInterface > const & xRootFolder, const OUString& rName ) { if ( rName == ".." || rName == "." ) @@ -123,7 +123,7 @@ Reference< XInterface > ZipPackageHelper::addFolder( Reference< XInterface >& xR return xFolder; } -void ZipPackageHelper::addFolderWithContent( Reference< XInterface >& xRootFolder, const OUString& rDirURL ) +void ZipPackageHelper::addFolderWithContent( Reference< XInterface > const & xRootFolder, const OUString& rDirURL ) { if (rDirURL.isEmpty()) return; @@ -159,7 +159,7 @@ void ZipPackageHelper::addFolderWithContent( Reference< XInterface >& xRootFolde } } -void ZipPackageHelper::addFile( css::uno::Reference< css::uno::XInterface >& xRootFolder, +void ZipPackageHelper::addFile( css::uno::Reference< css::uno::XInterface > const & xRootFolder, const OUString& rSourceFile ) { OUString aFileURL( rSourceFile ); diff --git a/unotools/source/misc/closeveto.cxx b/unotools/source/misc/closeveto.cxx index ed668fc12848..2aa74b142fb0 100644 --- a/unotools/source/misc/closeveto.cxx +++ b/unotools/source/misc/closeveto.cxx @@ -104,7 +104,7 @@ namespace utl i_data.xCloseable->addCloseListener( i_data.pListener.get() ); } - void lcl_deinit( CloseVeto_Data& i_data ) + void lcl_deinit( CloseVeto_Data const & i_data ) { if ( !i_data.xCloseable.is() ) return; diff --git a/unotools/source/misc/desktopterminationobserver.cxx b/unotools/source/misc/desktopterminationobserver.cxx index f7f3c43ccbc0..b84745951e17 100644 --- a/unotools/source/misc/desktopterminationobserver.cxx +++ b/unotools/source/misc/desktopterminationobserver.cxx @@ -182,7 +182,7 @@ namespace utl OObserverImpl::ensureObservation(); } - void DesktopTerminationObserver::revokeTerminationListener( ITerminationListener* _pListener ) + void DesktopTerminationObserver::revokeTerminationListener( ITerminationListener const * _pListener ) { ::osl::MutexGuard aGuard( ::osl::Mutex::getGlobalMutex() ); Listeners& rListeners = getListenerAdminData().aListeners; |