diff options
author | Noel Grandin <noel@peralex.com> | 2016-05-05 09:46:12 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-05-06 06:48:38 +0000 |
commit | f3d9aab8410c00298f29ca0194c5d33d53c63ff2 (patch) | |
tree | 370d24d49547d8eb2cdbcb293992d9b9a4a670ed /reportdesign | |
parent | 654c98064d3fd2bd1e13ae2bda5f84e8d51d0071 (diff) |
teach passstuffbyref plugin to check for..
unnecessarily passing primitives by const ref.
Suggested by Tor Lillqvist
Change-Id: I445e220542969ca3e252581e5953fb01cb2b2be6
Reviewed-on: https://gerrit.libreoffice.org/24672
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'reportdesign')
-rw-r--r-- | reportdesign/inc/ReportDefinition.hxx | 2 | ||||
-rw-r--r-- | reportdesign/source/core/api/Group.cxx | 2 | ||||
-rw-r--r-- | reportdesign/source/core/api/ReportDefinition.cxx | 2 | ||||
-rw-r--r-- | reportdesign/source/core/api/Tools.cxx | 2 | ||||
-rw-r--r-- | reportdesign/source/core/inc/Group.hxx | 2 | ||||
-rw-r--r-- | reportdesign/source/core/inc/Tools.hxx | 2 |
6 files changed, 6 insertions, 6 deletions
diff --git a/reportdesign/inc/ReportDefinition.hxx b/reportdesign/inc/ReportDefinition.hxx index e0a437479aca..e8c404b34a50 100644 --- a/reportdesign/inc/ReportDefinition.hxx +++ b/reportdesign/inc/ReportDefinition.hxx @@ -97,7 +97,7 @@ namespace reportdesign OReportDefinition& operator=(const OReportDefinition&) = delete; void setSection( const OUString& _sProperty - ,const bool& _bOn + ,bool _bOn ,const OUString& _sName ,css::uno::Reference< css::report::XSection>& _member); diff --git a/reportdesign/source/core/api/Group.cxx b/reportdesign/source/core/api/Group.cxx index fb62e431630d..7f75c28dc901 100644 --- a/reportdesign/source/core/api/Group.cxx +++ b/reportdesign/source/core/api/Group.cxx @@ -293,7 +293,7 @@ void SAL_CALL OGroup::removeVetoableChangeListener( const OUString& PropertyName } void OGroup::setSection( const OUString& _sProperty - ,const bool& _bOn + ,bool _bOn ,const OUString& _sName ,uno::Reference< report::XSection>& _member) { diff --git a/reportdesign/source/core/api/ReportDefinition.cxx b/reportdesign/source/core/api/ReportDefinition.cxx index 8626105cbba9..6addf6d7905c 100644 --- a/reportdesign/source/core/api/ReportDefinition.cxx +++ b/reportdesign/source/core/api/ReportDefinition.cxx @@ -1051,7 +1051,7 @@ uno::Reference< util::XCloneable > SAL_CALL OReportDefinition::createClone( ) t } void OReportDefinition::setSection( const OUString& _sProperty - ,const bool& _bOn + ,bool _bOn ,const OUString& _sName ,uno::Reference< report::XSection>& _member) { diff --git a/reportdesign/source/core/api/Tools.cxx b/reportdesign/source/core/api/Tools.cxx index e4350305d84c..2762e8acb013 100644 --- a/reportdesign/source/core/api/Tools.cxx +++ b/reportdesign/source/core/api/Tools.cxx @@ -42,7 +42,7 @@ uno::Reference< report::XSection> lcl_getSection(const uno::Reference< uno::XInt void throwIllegallArgumentException( const OUString& _sTypeName ,const uno::Reference< uno::XInterface >& ExceptionContext_ - ,const ::sal_Int16& ArgumentPosition_ + ,sal_Int16 ArgumentPosition_ ,const css::uno::Reference< css::uno::XComponentContext >& Context_ ) { diff --git a/reportdesign/source/core/inc/Group.hxx b/reportdesign/source/core/inc/Group.hxx index 2232701211e2..90995800a13c 100644 --- a/reportdesign/source/core/inc/Group.hxx +++ b/reportdesign/source/core/inc/Group.hxx @@ -85,7 +85,7 @@ namespace reportdesign l.notify(); } void setSection( const OUString& _sProperty - ,const bool& _bOn + ,bool _bOn ,const OUString& _sName ,css::uno::Reference< css::report::XSection>& _member); protected: diff --git a/reportdesign/source/core/inc/Tools.hxx b/reportdesign/source/core/inc/Tools.hxx index e64bb93c3aec..5b8036683fba 100644 --- a/reportdesign/source/core/inc/Tools.hxx +++ b/reportdesign/source/core/inc/Tools.hxx @@ -59,7 +59,7 @@ namespace reportdesign */ void throwIllegallArgumentException(const OUString& _sTypeName ,const css::uno::Reference< css::uno::XInterface >& ExceptionContext_ - ,const ::sal_Int16& ArgumentPosition_ + ,sal_Int16 ArgumentPosition_ ,const css::uno::Reference< css::uno::XComponentContext >& Context_); /** clones the given object |