diff options
| author | Noel Grandin <noel@peralex.com> | 2014-03-17 09:17:48 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2014-03-17 13:30:52 +0200 |
| commit | 12bb4fdddd3c72a9fc9209288cc8e827bf353920 (patch) | |
| tree | 8ef0f79c52fc7207bc13f7e56a70389605335f7a /cui/source/options | |
| parent | 7e8af4b249132764d9eca559b4a6e2449d4d26f8 (diff) | |
cui: prefer passing OUString by reference
Change-Id: I1a4d36a329a0eac8b9bb3d9b0a5434ffc843bde2
Diffstat (limited to 'cui/source/options')
| -rw-r--r-- | cui/source/options/optaboutconfig.cxx | 2 | ||||
| -rw-r--r-- | cui/source/options/optaboutconfig.hxx | 2 | ||||
| -rw-r--r-- | cui/source/options/optlingu.cxx | 7 | ||||
| -rw-r--r-- | cui/source/options/optsave.cxx | 2 |
4 files changed, 7 insertions, 6 deletions
diff --git a/cui/source/options/optaboutconfig.cxx b/cui/source/options/optaboutconfig.cxx index 6e0a0f345f7e..80504a5395fb 100644 --- a/cui/source/options/optaboutconfig.cxx +++ b/cui/source/options/optaboutconfig.cxx @@ -381,7 +381,7 @@ void CuiAboutConfigTabPage::FillItems(const Reference< XNameAccess >& xNameAcces } } -Reference< XNameAccess > CuiAboutConfigTabPage::getConfigAccess( OUString sNodePath, sal_Bool bUpdate ) +Reference< XNameAccess > CuiAboutConfigTabPage::getConfigAccess( const OUString& sNodePath, sal_Bool bUpdate ) { uno::Reference< uno::XComponentContext > xContext( ::comphelper::getProcessComponentContext() ); diff --git a/cui/source/options/optaboutconfig.hxx b/cui/source/options/optaboutconfig.hxx index 0f8027219b1b..135fc21e39c5 100644 --- a/cui/source/options/optaboutconfig.hxx +++ b/cui/source/options/optaboutconfig.hxx @@ -62,7 +62,7 @@ public: void InsertEntry(const OUString& rProp, const OUString& rStatus, const OUString& rType, const OUString& rValue); void Reset(/* const SfxItemSet&*/ ); void FillItems(const com::sun::star::uno::Reference<com::sun::star::container::XNameAccess>& xNameAccess); - com::sun::star::uno::Reference< com::sun::star::container::XNameAccess > getConfigAccess( OUString sNodePath, sal_Bool bUpdate ); + com::sun::star::uno::Reference< com::sun::star::container::XNameAccess > getConfigAccess( const OUString& sNodePath, sal_Bool bUpdate ); virtual sal_Bool FillItemSet( /* SfxItemSet& rSet*/ ); virtual Size GetOptimalSize() const; diff --git a/cui/source/options/optlingu.cxx b/cui/source/options/optlingu.cxx index 8297b609965b..d0cac4e3cc4d 100644 --- a/cui/source/options/optlingu.cxx +++ b/cui/source/options/optlingu.cxx @@ -130,10 +130,11 @@ static sal_Int32 lcl_SeqGetEntryPos( return i < nLen ? i : -1; } -static void lcl_OpenURL( OUString sURL ) +static void lcl_OpenURL( const OUString& _sURL ) { - if ( !sURL.isEmpty() ) + if ( !_sURL.isEmpty() ) { + OUString sURL = _sURL; localizeWebserviceURI(sURL); try { @@ -200,7 +201,7 @@ class ModuleUserData_Impl OUString sImplName; public: - ModuleUserData_Impl( OUString sImpName, sal_Bool bIsParent, sal_Bool bChecked, sal_uInt8 nSetType, sal_uInt8 nSetIndex ) : + ModuleUserData_Impl( const OUString& sImpName, sal_Bool bIsParent, sal_Bool bChecked, sal_uInt8 nSetType, sal_uInt8 nSetIndex ) : bParent(bIsParent), bIsChecked(bChecked), nType(nSetType), diff --git a/cui/source/options/optsave.cxx b/cui/source/options/optsave.cxx index e4b6e58a6706..b9490de6c6a4 100644 --- a/cui/source/options/optsave.cxx +++ b/cui/source/options/optsave.cxx @@ -345,7 +345,7 @@ sal_Bool SfxSaveTabPage::FillItemSet( SfxItemSet& rSet ) -sal_Bool isODFFormat( OUString sFilter ) +sal_Bool isODFFormat( const OUString& sFilter ) { static const char* aODFFormats[] = { |