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

2 files changed, 5 insertions, 5 deletions

diff --git a/starmath/inc/ElementsDockingWindow.hxx b/starmath/inc/ElementsDockingWindow.hxx
index a5985d38b59d..ccd9d7c89937 100644
--- a/starmath/inc/ElementsDockingWindow.hxx
+++ b/starmath/inc/ElementsDockingWindow.hxx
@@ -34,7 +34,7 @@ public:
     Point mBoxLocation;
     Size  mBoxSize;
 
-    SmElement(SmNodePointer pNode, OUString aText, OUString aHelpText);
+    SmElement(SmNodePointer pNode, const OUString& aText, const OUString& aHelpText);
     virtual ~SmElement();
 
     SmNodePointer getNode();
@@ -96,7 +96,7 @@ class SmElementsControl : public Control
     Size          maMaxElementDimensions;
     bool          mbVerticalMode;
 
-    void addElement(OUString aElementVisual, OUString aElementSource, OUString aHelpText);
+    void addElement(const OUString& aElementVisual, const OUString& aElementSource, const OUString& aHelpText);
 
     void addElements(const sal_uInt16 aElementsArray[][2], sal_uInt16 size);
 
diff --git a/starmath/inc/cursor.hxx b/starmath/inc/cursor.hxx
index b1f6e8b57929..ca3527dda94d 100644
--- a/starmath/inc/cursor.hxx
+++ b/starmath/inc/cursor.hxx
@@ -121,7 +121,7 @@ public:
     void DeletePrev(OutputDevice* pDev);
 
     /** Insert text at the current position */
-    void InsertText(OUString aString);
+    void InsertText(const OUString& aString);
 
     /** Insert an element into the formula */
     void InsertElement(SmFormulaElement element);
@@ -137,7 +137,7 @@ public:
      * This will work for stuff like "A intersection B". But stuff spaning multiple lines
      * or dependent on the context which position is placed in will not work!
      */
-    void InsertCommandText(OUString aCommandText);
+    void InsertCommandText(const OUString& aCommandText);
 
     /** Insert a special node created from aString
      *
@@ -148,7 +148,7 @@ public:
      * For more complex expressions use InsertCommandText, this method doesn't
      * use SmParser, this means that it's faster, but not as strong.
      */
-    void InsertSpecial(OUString aString);
+    void InsertSpecial(const OUString& aString);
 
     /** Create sub-/super script
      *