From 3cec1c93808c28644c97907a5698568df0971b3d Mon Sep 17 00:00:00 2001
From: Montel Laurent <montel@kde.org>
Date: Fri, 17 May 2013 20:53:18 +0200
Subject: [PATCH] Minor optimization

---
 src/abstractmonitor.h        |  2 +-
 src/managecapturesdialog.cpp | 10 +++++-----
 src/monitor.cpp              |  2 +-
 src/monitor.h                |  2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/abstractmonitor.h b/src/abstractmonitor.h
index b55fbd0f..db7ab3a5 100644
--- a/src/abstractmonitor.h
+++ b/src/abstractmonitor.h
@@ -89,7 +89,7 @@ class AbstractMonitor : public QWidget
     Q_OBJECT
 public:
     AbstractMonitor(Kdenlive::MONITORID id, MonitorManager *manager, QWidget *parent = 0);
-    Kdenlive::MONITORID id() {return m_id;};
+    Kdenlive::MONITORID id() {return m_id;}
     virtual ~AbstractMonitor();
     virtual AbstractRender *abstractRender() = 0;
     bool isActive() const;
diff --git a/src/managecapturesdialog.cpp b/src/managecapturesdialog.cpp
index 8a724ce4..5467460c 100644
--- a/src/managecapturesdialog.cpp
+++ b/src/managecapturesdialog.cpp
@@ -70,7 +70,7 @@ ManageCapturesDialog::~ManageCapturesDialog() {}
 void ManageCapturesDialog::slotCheckItemIcon()
 {
     int ct = 0;
-    int count = m_view.treeWidget->topLevelItemCount();
+    const int count = m_view.treeWidget->topLevelItemCount();
     while (ct < count) {
         QTreeWidgetItem *item = m_view.treeWidget->topLevelItem(ct);
         //QTreeWidgetItem *item = m_view.treeWidget->currentItem();
@@ -88,7 +88,7 @@ void ManageCapturesDialog::slotCheckItemIcon()
 
 void ManageCapturesDialog::slotRefreshButtons()
 {
-    int count = m_view.treeWidget->topLevelItemCount();
+    const int count = m_view.treeWidget->topLevelItemCount();
     bool enabled = false;
     for (int i = 0; i < count; ++i) {
         QTreeWidgetItem *item = m_view.treeWidget->topLevelItem(i);
@@ -104,7 +104,7 @@ void ManageCapturesDialog::slotDeleteCurrent()
 {
     QTreeWidgetItem *item = m_view.treeWidget->currentItem();
     if (!item) return;
-    int i = m_view.treeWidget->indexOfTopLevelItem(item);
+    const int i = m_view.treeWidget->indexOfTopLevelItem(item);
     m_view.treeWidget->takeTopLevelItem(i);
     kDebug() << "DELETING FILE: " << item->text(0);
     //KIO::NetAccess::del(KUrl(item->text(0)), this);
@@ -116,7 +116,7 @@ void ManageCapturesDialog::slotDeleteCurrent()
 
 void ManageCapturesDialog::slotToggle()
 {
-    int count = m_view.treeWidget->topLevelItemCount();
+    const int count = m_view.treeWidget->topLevelItemCount();
     for (int i = 0; i < count; ++i) {
         QTreeWidgetItem *item = m_view.treeWidget->topLevelItem(i);
         if (item) {
@@ -130,7 +130,7 @@ KUrl::List ManageCapturesDialog::importFiles()
 {
     KUrl::List result;
 
-    int count = m_view.treeWidget->topLevelItemCount();
+    const int count = m_view.treeWidget->topLevelItemCount();
     for (int i = 0; i < count; ++i) {
         QTreeWidgetItem *item = m_view.treeWidget->topLevelItem(i);
         if (item && item->checkState(0) == Qt::Checked)
diff --git a/src/monitor.cpp b/src/monitor.cpp
index c6b14fb6..68f7b249 100644
--- a/src/monitor.cpp
+++ b/src/monitor.cpp
@@ -386,7 +386,7 @@ void Monitor::updateMarkers(DocClipBase *source)
     }
 }
 
-void Monitor::setMarkers(QList <CommentedTime> markers)
+void Monitor::setMarkers(const QList<CommentedTime> &markers)
 {
     m_ruler->setMarkers(markers);
 }
diff --git a/src/monitor.h b/src/monitor.h
index c7821273..5dc6f2a4 100644
--- a/src/monitor.h
+++ b/src/monitor.h
@@ -86,7 +86,7 @@ public:
     void checkOverlay();
     void updateTimecodeFormat();
     void updateMarkers(DocClipBase *source);
-    void setMarkers(QList <CommentedTime> markers);
+    void setMarkers(const QList <CommentedTime> &markers);
     MonitorEditWidget *getEffectEdit();
     QWidget *container();
     void reloadProducer(const QString &id);
-- 
2.39.2