From: Montel Laurent <montel@kde.org>
Date: Sat, 1 Jun 2013 14:16:42 +0000 (+0200)
Subject: Const'ref. Fix include/forward class
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=bf036e23649237928fad4902a9cf1728e6d5c588;p=kdenlive

Const'ref. Fix include/forward class
---

diff --git a/src/renderer.cpp b/src/renderer.cpp
index 041f82ab..444fcc70 100644
--- a/src/renderer.cpp
+++ b/src/renderer.cpp
@@ -4090,7 +4090,7 @@ void Render::mltDeleteTransition(QString tag, int /*a_track*/, int b_track, GenT
     //if (m_isBlocked == 0) m_mltConsumer->set("refresh", 1);
 }
 
-QMap<QString, QString> Render::mltGetTransitionParamsFromXml(QDomElement xml)
+QMap<QString, QString> Render::mltGetTransitionParamsFromXml(const QDomElement &xml)
 {
     QDomNodeList attribs = xml.elementsByTagName("parameter");
     QMap<QString, QString> map;
@@ -4566,7 +4566,7 @@ void Render::updatePreviewSettings()
 }
 
 
-QString Render::updateSceneListFps(double current_fps, double new_fps, QString scene)
+QString Render::updateSceneListFps(double current_fps, double new_fps, const QString &scene)
 {
     // Update all frame positions to the new fps value
     //WARNING: there are probably some effects or other that hold a frame value
@@ -4733,7 +4733,7 @@ bool Render::getBlackMagicOutputDeviceList(KComboBox *devicelist, bool force)
     return true;
 }
 
-void Render::slotMultiStreamProducerFound(const QString path, QList<int> audio_list, QList<int> video_list, stringMap data)
+void Render::slotMultiStreamProducerFound(const QString &path, QList<int> audio_list, QList<int> video_list, stringMap data)
 { 
     if (KdenliveSettings::automultistreams()) {
         for (int i = 1; i < video_list.count(); ++i) {
diff --git a/src/renderer.h b/src/renderer.h
index 6656a17b..805d2086 100644
--- a/src/renderer.h
+++ b/src/renderer.h
@@ -303,7 +303,7 @@ class Render: public AbstractRender
     void setDropFrames(bool show);
     /** @brief Sets an MLT consumer property. */
     void setConsumerProperty(const QString &name, const QString &value);
-    QString updateSceneListFps(double current_fps, double new_fps, QString scene);
+    QString updateSceneListFps(double current_fps, double new_fps, const QString &scene);
 
     void showAudio(Mlt::Frame&);
     
@@ -403,7 +403,7 @@ private:
     void closeMlt();
     void mltCheckLength(Mlt::Tractor *tractor);
     void mltPasteEffects(Mlt::Producer *source, Mlt::Producer *dest);
-    QMap<QString, QString> mltGetTransitionParamsFromXml(QDomElement xml);
+    QMap<QString, QString> mltGetTransitionParamsFromXml(const QDomElement &xml);
     QMap<QString, Mlt::Producer *> m_slowmotionProducers;
     /** @brief The ids of the clips that are currently being loaded for info query */
     QStringList m_processingClipId;
@@ -432,7 +432,7 @@ private slots:
     /** @brief Process the clip info requests (in a separate thread). */
     void processFileProperties();
     /** @brief A clip with multiple video streams was found, ask what to do. */
-    void slotMultiStreamProducerFound(const QString path, QList<int> audio_list, QList<int> video_list, stringMap data);
+    void slotMultiStreamProducerFound(const QString &path, QList<int> audio_list, QList<int> video_list, stringMap data);
     void showFrame(Mlt::Frame *);
     void slotCheckSeeking();
 
diff --git a/src/renderwidget.cpp b/src/renderwidget.cpp
index 766dfb59..3d287db6 100644
--- a/src/renderwidget.cpp
+++ b/src/renderwidget.cpp
@@ -46,6 +46,7 @@
 #include <QDBusInterface>
 #include <QThread>
 #include <QScriptEngine>
+#include <QKeyEvent>
 
 #include "locale.h"
 
diff --git a/src/renderwidget.h b/src/renderwidget.h
index 2c0c9cee..388a3835 100644
--- a/src/renderwidget.h
+++ b/src/renderwidget.h
@@ -29,12 +29,12 @@
 #include <QPushButton>
 #include <QPainter>
 #include <QStyledItemDelegate>
-#include <QKeyEvent>
 
 #include "definitions.h"
 #include "ui_renderwidget_ui.h"
 
 class QDomElement;
+class QKeyEvent;
 
 
 // RenderViewDelegate is used to draw the progress bars.