diff options
author | Noel Grandin <noel@peralex.com> | 2014-03-06 15:29:08 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2014-03-18 08:32:26 +0200 |
commit | 86a32589e90ee983159fb5b2c6a594428ab7d422 (patch) | |
tree | 6de946fe2b9b25614f0d197af95e9d3aadcd1bd9 /forms/source/misc | |
parent | Avoid possible resource leaks by boost::scoped_array (diff) | |
download | core-86a32589e90ee983159fb5b2c6a594428ab7d422.tar.gz core-86a32589e90ee983159fb5b2c6a594428ab7d422.zip |
Find places where OUString and OString are passed by value.
It's not very efficient, because we generally end up copying it twice -
once into the parameter and again into the destination OUString.
So I create a clang plugin that finds such places and generates a
warning so that we can convert them to pass-by-reference.
Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
Diffstat (limited to 'forms/source/misc')
-rw-r--r-- | forms/source/misc/services.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/forms/source/misc/services.cxx b/forms/source/misc/services.cxx index 4b7416c5e32a..2228bc06d164 100644 --- a/forms/source/misc/services.cxx +++ b/forms/source/misc/services.cxx @@ -99,7 +99,7 @@ static Sequence<sal_Int64> s_aFactories; void registerClassInfo( - OUString _rClassImplName, // the ImplName of the class + const OUString& _rClassImplName, // the ImplName of the class const Sequence< OUString >& _rServiceNames, // the services supported by this class ::cppu::ComponentInstantiation _pCreateFunction // the method for instantiating such a class ) |