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 /pyuno | |
parent | bb17844099ba98a77c8e5d7a25c0c416a4b0641e (diff) |
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 'pyuno')
-rw-r--r-- | pyuno/source/module/pyuno_module.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pyuno/source/module/pyuno_module.cxx b/pyuno/source/module/pyuno_module.cxx index 5513f0e051ce..47b57aa694c0 100644 --- a/pyuno/source/module/pyuno_module.cxx +++ b/pyuno/source/module/pyuno_module.cxx @@ -106,7 +106,7 @@ public: { PyDict_SetItem(used, key, Py_True); } - void setInitialised(OUString key, sal_Int32 pos = -1) + void setInitialised(const OUString& key, sal_Int32 pos = -1) { if (initialised[key]) { @@ -126,7 +126,7 @@ public: if ( pos >= 0 ) ++nPosConsumed; } - bool isInitialised(OUString key) + bool isInitialised(const OUString& key) { return initialised[key]; } |