When creating methods that pass QString, use const QString&

This removes unecessary creation and destruction of the object
making the code shine a bit more and be more fluffy. :)

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

1 files changed, 6 insertions, 6 deletions

diff --git a/qthelper.h b/qthelper.h
index 648a68090..9d31f5a12 100644
--- a/qthelper.h
+++ b/qthelper.h
@@ -29,12 +29,12 @@ class DiveComputerList {
 public:
 	DiveComputerList();
 	~DiveComputerList();
-	const DiveComputerNode *getExact(QString m, uint32_t d);
-	const DiveComputerNode *get(QString m);
-	void addDC(QString m, uint32_t d, QString n = "", QString s = "", QString f = "");
-	void rmDC(QString m, uint32_t d);
-	DiveComputerNode matchDC(QString m, uint32_t d);
-	DiveComputerNode matchModel(QString m);
+	const DiveComputerNode *getExact(const QString& m, uint32_t d);
+	const DiveComputerNode *get(const QString& m);
+	void addDC(const QString& m, uint32_t d,const QString& n = QString(),const QString& s = QString(), const QString& f = QString());
+	void rmDC(const QString& m, uint32_t d);
+	DiveComputerNode matchDC(const QString& m, uint32_t d);
+	DiveComputerNode matchModel(const QString& m);
 	QMultiMap<QString, DiveComputerNode> dcMap;
 	QMultiMap<QString, DiveComputerNode> dcWorkingMap;
 };