summaryrefslogtreecommitdiff
path: root/unotools/source/config
diff options
context:
space:
mode:
authorNoel Grandin <noel@peralex.com>2016-04-14 10:20:52 +0200
committerNoel Grandin <noel@peralex.com>2016-04-14 10:24:47 +0200
commitb8eb2946511ce617323b13dffe2b1d9704e0be60 (patch)
treed5aa4f91e75d654a1cdfcd75f26dd4b813a73ff7 /unotools/source/config
parentd8644c8edb405abd9d71e62e43e898c1d2a28fd2 (diff)
loplugin:passstuffbyref in various
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
Diffstat (limited to 'unotools/source/config')
-rw-r--r--unotools/source/config/bootstrap.cxx2
-rw-r--r--unotools/source/config/moduleoptions.cxx8
-rw-r--r--unotools/source/config/securityoptions.cxx6
3 files changed, 8 insertions, 8 deletions
diff --git a/unotools/source/config/bootstrap.cxx b/unotools/source/config/bootstrap.cxx
index d25d12981d20..42a3081efbec 100644
--- a/unotools/source/config/bootstrap.cxx
+++ b/unotools/source/config/bootstrap.cxx
@@ -109,7 +109,7 @@ namespace utl
OUString getBootstrapValue(OUString const& _sName, OUString const& _sDefault) const;
static bool getVersionValue(OUString const& _sName, OUString& _rValue, OUString const& _sDefault);
- OUString getImplName() const { return m_aImplName; }
+ const OUString& getImplName() const { return m_aImplName; }
private: // implementation
bool initBaseInstallationData(rtl::Bootstrap& _rData);
diff --git a/unotools/source/config/moduleoptions.cxx b/unotools/source/config/moduleoptions.cxx
index 71703899d50f..4f476294d78a 100644
--- a/unotools/source/config/moduleoptions.cxx
+++ b/unotools/source/config/moduleoptions.cxx
@@ -187,11 +187,11 @@ struct FactoryInfo
// That's why we can't make our member public. We must use get/set/init methods
// to control access on it!
bool getInstalled () const { return bInstalled; };
- OUString getFactory () const { return sFactory; };
- OUString getTemplateFile () const { return sTemplateFile; };
- OUString getDefaultFilter () const { return sDefaultFilter; };
+ const OUString& getFactory () const { return sFactory; };
+ const OUString& getTemplateFile () const { return sTemplateFile; };
+ const OUString& getDefaultFilter () const { return sDefaultFilter; };
bool isDefaultFilterReadonly() const { return bDefaultFilterReadonly; }
- sal_Int32 getIcon () const { return nIcon; };
+ sal_Int32 getIcon () const { return nIcon; };
// If you call set-methods - we check for changes of values and mark it.
// But if you wish to set it without that... you must initialize it!
diff --git a/unotools/source/config/securityoptions.cxx b/unotools/source/config/securityoptions.cxx
index 59de838a005d..d048db008bab 100644
--- a/unotools/source/config/securityoptions.cxx
+++ b/unotools/source/config/securityoptions.cxx
@@ -131,14 +131,14 @@ class SvtSecurityOptions_Impl : public ConfigItem
bool IsReadOnly ( SvtSecurityOptions::EOption eOption ) const;
- Sequence< OUString > GetSecureURLs ( ) const { return m_seqSecureURLs;}
- void SetSecureURLs ( const Sequence< OUString >& seqURLList );
+ const Sequence< OUString >& GetSecureURLs( ) const { return m_seqSecureURLs;}
+ void SetSecureURLs ( const Sequence< OUString >& seqURLList );
inline sal_Int32 GetMacroSecurityLevel ( ) const;
void SetMacroSecurityLevel ( sal_Int32 _nLevel );
inline bool IsMacroDisabled ( ) const;
- Sequence< SvtSecurityOptions::Certificate > GetTrustedAuthors ( ) const { return m_seqTrustedAuthors;}
+ const Sequence< SvtSecurityOptions::Certificate >& GetTrustedAuthors( ) const { return m_seqTrustedAuthors;}
void SetTrustedAuthors ( const Sequence< SvtSecurityOptions::Certificate >& rAuthors );
bool IsOptionSet ( SvtSecurityOptions::EOption eOption ) const;