diff options
| author | Noel Grandin <noel@peralex.com> | 2016-04-14 10:20:52 +0200 |
|---|---|---|
| committer | Noel Grandin <noel@peralex.com> | 2016-04-14 10:24:47 +0200 |
| commit | b8eb2946511ce617323b13dffe2b1d9704e0be60 (patch) | |
| tree | d5aa4f91e75d654a1cdfcd75f26dd4b813a73ff7 /unotools | |
| parent | d8644c8edb405abd9d71e62e43e898c1d2a28fd2 (diff) | |
loplugin:passstuffbyref in various
Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
Diffstat (limited to 'unotools')
| -rw-r--r-- | unotools/source/config/bootstrap.cxx | 2 | ||||
| -rw-r--r-- | unotools/source/config/moduleoptions.cxx | 8 | ||||
| -rw-r--r-- | unotools/source/config/securityoptions.cxx | 6 |
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; |