[Kdenlive-devel] [PATCH kdenlive 21/27] const parameter as reference

Mikko Rapeli mikko.rapeli at iki.fi
Sun Aug 28 10:25:01 UTC 2011


Found by cppcheck.
---
 src/monitormanager.cpp |    2 +-
 src/monitormanager.h   |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/monitormanager.cpp b/src/monitormanager.cpp
index 9bf6ce4..662748e 100644
--- a/src/monitormanager.cpp
+++ b/src/monitormanager.cpp
@@ -81,7 +81,7 @@ void MonitorManager::activateMonitor(QString name)
     emit checkColorScopes();
 }
 
-bool MonitorManager::isActive(const QString name) const
+bool MonitorManager::isActive(const QString &name) const
 {
     return m_activeMonitor ? m_activeMonitor->name() == name: false;
 }
diff --git a/src/monitormanager.h b/src/monitormanager.h
index 171ad0b..30e1864 100644
--- a/src/monitormanager.h
+++ b/src/monitormanager.h
@@ -47,7 +47,7 @@ public slots:
     /** @brief Activates a monitor.
      * @param name name of the monitor to activate */
     void activateMonitor(QString name = QString());
-    bool isActive(const QString name) const;
+    bool isActive(const QString &name) const;
     void slotPlay();
     void slotPause();
     void slotPlayZone();
-- 
1.7.5.4





More information about the Kdenlive mailing list