Do not copy strings when you don't need it.

This patch removed the use of copy-constructors on the QString to use the
const-references. Even knowing that the QString is a refcounted class,
let's not get that bad habit.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>

1 files changed, 3 insertions, 3 deletions

diff --git a/qt-ui/tagwidget.h b/qt-ui/tagwidget.h
index 42c94b4e0..62fa36f30 100644
--- a/qt-ui/tagwidget.h
+++ b/qt-ui/tagwidget.h
@@ -12,15 +12,15 @@ public:
 	void setCompleter(QCompleter *completer);
 	QPair<int, int> getCursorTagPosition();
 	void highlight();
-	void setText(QString text);
+	void setText(const QString& text);
 	void clear();
 	void setCursorPosition(int position);
 	void wheelEvent(QWheelEvent *event);
 public
 slots:
 	void reparse();
-	void completionSelected(QString);
-	void completionHighlighted(QString);
+	void completionSelected(const QString& text);
+	void completionHighlighted(const QString& text);
 
 protected:
 	void keyPressEvent(QKeyEvent *e);