From: Jean-Baptiste Mardelle Date: Fri, 9 Sep 2011 23:39:41 +0000 (+0000) Subject: cppcheck fixes, patch by Mikko Rapeli [6/27] X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=d9f0349d94f8b232979dc4f5538a1eb04b127065;p=kdenlive cppcheck fixes, patch by Mikko Rapeli [6/27] svn path=/trunk/kdenlive/; revision=5878 --- diff --git a/src/definitions.h b/src/definitions.h index 6b0576b0..4384e8c6 100644 --- a/src/definitions.h +++ b/src/definitions.h @@ -101,14 +101,14 @@ struct MltVideoProfile { class EffectParameter { public: - EffectParameter(const QString name, const QString value): m_name(name), m_value(value) {} + EffectParameter(const QString &name, const QString &value): m_name(name), m_value(value) {} QString name() const { return m_name; } QString value() const { return m_value; } - void setValue(const QString value) { + void setValue(const QString &value) { m_value = value; } @@ -124,12 +124,12 @@ class EffectsParameterList: public QList < EffectParameter > { public: EffectsParameterList(): QList < EffectParameter >() {} - bool hasParam(const QString name) const { + bool hasParam(const QString &name) const { for (int i = 0; i < size(); i++) if (at(i).name() == name) return true; return false; } - QString paramValue(const QString name, QString defaultValue = QString()) const { + QString paramValue(const QString &name, QString defaultValue = QString()) const { for (int i = 0; i < size(); i++) { if (at(i).name() == name) return at(i).value(); } @@ -139,7 +139,7 @@ public: if (name.isEmpty()) return; append(EffectParameter(name, value)); } - void removeParam(const QString name) { + void removeParam(const QString &name) { for (int i = 0; i < size(); i++) if (at(i).name() == name) { removeAt(i); @@ -152,7 +152,7 @@ class CommentedTime { public: CommentedTime(): t(GenTime(0)) {} - CommentedTime(const GenTime time, QString comment) + CommentedTime(const GenTime &time, QString comment) : t(time), c(comment) { } QString comment() const {