]> git.sesse.net Git - kdenlive/commitdiff
const'ref.
authorMontel Laurent <montel@kde.org>
Tue, 14 May 2013 18:10:54 +0000 (20:10 +0200)
committerMontel Laurent <montel@kde.org>
Tue, 14 May 2013 18:10:54 +0000 (20:10 +0200)
48 files changed:
src/documentchecker.cpp
src/documentchecker.h
src/effectslistview.cpp
src/effectslistview.h
src/effectslistwidget.cpp
src/effectslistwidget.h
src/effectstackedit.cpp
src/effectstackedit.h
src/geometryval.cpp
src/geometryval.h
src/geometrywidget.cpp
src/geometrywidget.h
src/guide.cpp
src/guide.h
src/headertrack.cpp
src/headertrack.h
src/jogshuttle.cpp
src/jogshuttle.h
src/keyframehelper.h
src/markerdialog.cpp
src/markerdialog.h
src/mltdevicecapture.h
src/parameterplotter.cpp
src/parameterplotter.h
src/positionedit.cpp
src/positionedit.h
src/projectlist.cpp
src/projectlist.h
src/projectsettings.cpp
src/projectsettings.h
src/renderer.h
src/renderwidget.cpp
src/renderwidget.h
src/slideshowclip.cpp
src/slideshowclip.h
src/timecode.cpp
src/timecode.h
src/titlewidget.cpp
src/titlewidget.h
src/tracksconfigdialog.cpp
src/trackview.cpp
src/trackview.h
src/transitionsettings.cpp
src/transitionsettings.h
src/unicodedialog.cpp
src/unicodedialog.h
src/videoglwidget.cpp
src/videoglwidget.h

index 7504af20ed8c71071f76922672d70ad02e8137d4..3f82996ef5fe6e13ae3167d52e55696e981ab6d5 100644 (file)
@@ -339,7 +339,6 @@ bool DocumentChecker::hasErrorInClips()
 
     for (int i = 0; i < max; i++) {
         e = missingProxies.at(i).toElement();
-        QString clipType;
         QString realPath = e.attribute("resource");
         QString id = e.attribute("id");
         // Tell Kdenlive to recreate proxy
@@ -448,7 +447,7 @@ QString DocumentChecker::getProperty(QDomElement effect, const QString &name)
     return QString();
 }
 
-void DocumentChecker::setProperty(QDomElement effect, const QString &name, const QString value)
+void DocumentChecker::setProperty(QDomElement effect, const QString &name, const QString &value)
 {
     QDomNodeList params = effect.elementsByTagName("property");
     for (int i = 0; i < params.count(); i++) {
@@ -916,7 +915,7 @@ void DocumentChecker::slotDeleteSelected()
     }
 }
 
-void DocumentChecker::checkMissingImagesAndFonts(QStringList images, QStringList fonts, const QString &id, const QString &baseClip)
+void DocumentChecker::checkMissingImagesAndFonts(const QStringList &images, const QStringList &fonts, const QString &id, const QString &baseClip)
 {
     QDomDocument doc;
     foreach(const QString &img, images) {
index 03c91ec1f093354c5de71faf5f5a219be0dbb0a5..60c4fd4fa194fc7dcbf901636ab4c9cb45e95a2a 100644 (file)
@@ -45,10 +45,10 @@ private slots:
     void slotPlaceholders();
     void slotDeleteSelected();
     QString getProperty(QDomElement effect, const QString &name);
-    void setProperty(QDomElement effect, const QString &name, const QString value);
+    void setProperty(QDomElement effect, const QString &name, const QString &value);
     QString searchLuma(const QDir &dir, const QString &file) const;
     /** @brief Check if images and fonts in this clip exists, returns a list of images that do exist so we don't check twice. */
-    void checkMissingImagesAndFonts(QStringList images, QStringList fonts, const QString &id, const QString &baseClip);
+    void checkMissingImagesAndFonts(const QStringList &images, const QStringList &fonts, const QString &id, const QString &baseClip);
     void slotCheckButtons();
     /** @brief Fix duration mismatch issues. */
     void slotFixDuration();
index e4c9d6e04ef4de0aa25f0bb6e140317e14277ed8..01768474c847295bd27fe924c0ab87493f656394 100644 (file)
@@ -184,7 +184,7 @@ void EffectsListView::slotUpdateSearch(QTreeWidgetItem *item, bool hidden)
     }
 }
 
-void EffectsListView::slotAutoExpand(QString text)
+void EffectsListView::slotAutoExpand(const QString &text)
 {
     search_effect->updateSearch();
     bool selected = false;
index c744d080d8f8c1f502abe7a946d62b55d2cd0bfc..0efbc74e04c6a53e92bf83b1faa024b970517f64 100644 (file)
@@ -87,7 +87,7 @@ private slots:
 
     /** @brief Expands folders that match our search.
     * @param text Current search string */
-    void slotAutoExpand(QString text);
+    void slotAutoExpand(const QString &text);
 
 signals:
     void addEffect(const QDomElement);
index 2e6af1f76c518b5ffdd1520aadc52a0aefad29a0..2e1229a8af1f5580d7229b4e44603b0baa482254 100644 (file)
@@ -300,7 +300,7 @@ const QDomElement EffectsListWidget::itemEffect(QTreeWidgetItem *item) const
 }
 
 
-QString EffectsListWidget::currentInfo()
+QString EffectsListWidget::currentInfo() const
 {
     QTreeWidgetItem *item = currentItem();
     if (!item || item->data(0, TypeRole).toInt() == (int)EFFECT_FOLDER) return QString();
index c44d0f65f36db0f66b8a253d9cae2082de6f2114..b3fd80b02b37e01b477f0eda604f95ecc8e9e63b 100644 (file)
@@ -37,7 +37,7 @@ public:
     explicit EffectsListWidget(QMenu *contextMenu, QWidget *parent = 0);
     virtual ~EffectsListWidget();
     const QDomElement currentEffect() const;
-    QString currentInfo();
+    QString currentInfo() const;
     const QDomElement itemEffect(QTreeWidgetItem *item) const;
     void initList(QMenu *effectsMenu, KActionCategory *effectActions);
 
index be6681497097c0f4ba5ec802e034a9f0903c2cb9..c15c0fe6b74106ee8d9967eb845cb9957d80214d 100644 (file)
@@ -69,7 +69,7 @@ Monitor *EffectStackEdit::monitor()
     return m_metaInfo.monitor;
 }
 
-void EffectStackEdit::updateProjectFormat(MltVideoProfile profile, Timecode t)
+void EffectStackEdit::updateProjectFormat(const MltVideoProfile &profile, const Timecode &t)
 {
     m_metaInfo.profile = profile;
     m_metaInfo.timecode = t;
index c210b32e416322baaf1a396d70b1265da7b12a32..dd3bddfe5db165e01a205835739bf4accf032b99 100644 (file)
@@ -38,7 +38,7 @@ class EffectStackEdit : public QScrollArea
 public:
     explicit EffectStackEdit(Monitor *monitor, QWidget *parent = 0);
     ~EffectStackEdit();
-    void updateProjectFormat(MltVideoProfile profile, Timecode t);
+    void updateProjectFormat(const MltVideoProfile &profile, const Timecode &t);
     static QMap<QString, QImage> iconCache;
     /** @brief Sets attribute @param name to @param value.
     *
index 0d706894d028e7f283cb5f434830fe352c66266b..313751bb303c3adaf50e36838523148ac13f0ffe 100644 (file)
@@ -29,7 +29,7 @@
 #include <QTimer>
 
 
-Geometryval::Geometryval(const MltVideoProfile &profile, Timecode t, const QPoint &frame_size, int startPoint, QWidget* parent) :
+Geometryval::Geometryval(const MltVideoProfile &profile, const Timecode &t, const QPoint &frame_size, int startPoint, QWidget* parent) :
         QWidget(parent),
         m_profile(profile),
         m_paramRect(NULL),
index 0e00447de808cb533ba1685eaa898d66ce6e1884..96c9f5bd9fc3917445e887d396419dbbe2dfc215 100644 (file)
@@ -40,7 +40,7 @@ class Geometryval : public QWidget, public Ui::Geometryval
 {
     Q_OBJECT
 public:
-    explicit Geometryval(const MltVideoProfile &profile, Timecode t, const QPoint &frame_size, int startPoint = 0, QWidget* parent = 0);
+    explicit Geometryval(const MltVideoProfile &profile, const Timecode &t, const QPoint &frame_size, int startPoint = 0, QWidget* parent = 0);
     virtual ~Geometryval();
     QDomElement getParamDesc();
     QString getValue() const;
index eab0061e4da12c4dc955a5c750dafcc388f765de..b1e0c509e074ac02a6261a81813e944912d8e6a6 100644 (file)
@@ -38,7 +38,7 @@
 
 
 
-GeometryWidget::GeometryWidget(Monitor* monitor, Timecode timecode, int clipPos, bool isEffect, bool showRotation, QWidget* parent):
+GeometryWidget::GeometryWidget(Monitor* monitor, const Timecode &timecode, int clipPos, bool isEffect, bool showRotation, QWidget* parent):
     QWidget(parent),
     m_monitor(monitor),
     m_timePos(new TimecodeDisplay(timecode)),
@@ -378,7 +378,7 @@ void GeometryWidget::setupParam(const QDomElement elem, int minframe, int maxfra
     slotPositionChanged(0, false);
 }
 
-void GeometryWidget::addParameter(const QDomElement elem)
+void GeometryWidget::addParameter(const QDomElement &elem)
 {
     Mlt::Geometry *geometry = new Mlt::Geometry(elem.attribute("value").toUtf8().data(), m_outPoint - m_inPoint, m_monitor->render->frameRenderWidth(), m_monitor->render->renderHeight());
     m_extraGeometries.append(geometry);
index aaa82b9745759a3002430f7de778cf5eeaa3ccfa..fc5e245abe73b38d322b00b2ed4650d4359da96f 100644 (file)
@@ -48,7 +48,7 @@ public:
     * @param isEffect true if used in an effect, false if used in a transition
     * @param showRotation Should we show or hide the rotation sliders
     * @param parent (optional) Parent widget */
-    GeometryWidget(Monitor *monitor, Timecode timecode, int clipPos, bool isEffect, bool showRotation, QWidget* parent = 0);
+    GeometryWidget(Monitor *monitor, const Timecode &timecode, int clipPos, bool isEffect, bool showRotation, QWidget* parent = 0);
     virtual ~GeometryWidget();
     /** @brief Gets the geometry as a serialized string. */
     QString getValue() const;
@@ -57,7 +57,7 @@ public:
     void updateTimecodeFormat();
     /** @brief Sets the size of the original clip. */
     void setFrameSize(QPoint size);
-    void addParameter(const QDomElement elem);
+    void addParameter(const QDomElement &elem);
     void importKeyframes(const QString &data, int maximum);
     int currentPosition() const;
 
index 3d44c627fad372ae530b15087f6b4c7f04094fe2..ba05ec6aef126841aa744ae0cbe494b013b5c1d9 100644 (file)
@@ -29,7 +29,7 @@
 #include <QGraphicsView>
 #include <QScrollBar>
 
-Guide::Guide(CustomTrackView *view, GenTime pos, const QString &label, double height) :
+Guide::Guide(CustomTrackView *view, const GenTime &pos, const QString &label, double height) :
         QGraphicsLineItem(),
         m_position(pos),
         m_label(label),
index 5e77c85763cfe2452083e54ec4cf73682ffe144c..6bd392ae654103839c72122962aec159b52123cf 100644 (file)
@@ -34,7 +34,7 @@ class Guide : public QGraphicsLineItem
 {
 
 public:
-    Guide(CustomTrackView *view, GenTime pos, const QString &label, double height);
+    Guide(CustomTrackView *view, const GenTime &pos, const QString &label, double height);
     GenTime position() const;
     void updateGuide(const GenTime &newPos, const QString &comment = QString());
     QString label() const;
index d1d8f0f75a997814dcb4a16bec96a8ef23b689a8..cbf8a6ad06aced30facb20a90bd1381904436833 100644 (file)
@@ -96,7 +96,7 @@ HeaderTrack::HeaderTrack(int index, TrackInfo info, int height, const QList <QAc
 {
 }*/
 
-void HeaderTrack::updateEffectLabel(QStringList effects)
+void HeaderTrack::updateEffectLabel(const QStringList &effects)
 {
     if (!effects.isEmpty()) {
         effect_label->setHidden(false);
index e262bb9cd619c4234d5d92b34ba42f047078ff87..c2ca783c7ffd782d1cb64a2b646ddc2bbcf2358a 100644 (file)
@@ -38,7 +38,7 @@ public:
     void adjustSize(int height);
     void setSelectedIndex(int ix);
     /** @brief Update the track label to show if current track has effects or not.*/
-    void updateEffectLabel(QStringList effects);
+    void updateEffectLabel(const QStringList &effects);
 
 protected:
     virtual void mousePressEvent(QMouseEvent * event);
index 0e92cb42f12783e410a1013f8984e864ed909b53..e9f5fcb1caac0a161bc510ab1cb62db6bdd71bd1 100644 (file)
@@ -265,7 +265,7 @@ void ShuttleThread::jog(unsigned int value)
 }
 
 
-JogShuttle::JogShuttle(QString device, QObject *parent) :
+JogShuttle::JogShuttle(const QString &device, QObject *parent) :
         QObject(parent)
 {
     initDevice(device);
@@ -276,7 +276,7 @@ JogShuttle::~JogShuttle()
        stopDevice();
 }
 
-void JogShuttle::initDevice(QString device)
+void JogShuttle::initDevice(const QString &device)
 {
     if (m_shuttleProcess.isRunning()) {
         if (device == m_shuttleProcess.m_device) return;
index 55b4472a1306db2c767856e14c15fd48481b83f4..6b70be86d5c086f025630281a29704541e66041e 100644 (file)
@@ -55,10 +55,10 @@ private:
 class JogShuttle: public QObject
 {
 Q_OBJECT public:
-    explicit JogShuttle(QString device, QObject * parent = 0);
+    explicit JogShuttle(const QString &device, QObject * parent = 0);
     ~JogShuttle();
     void stopDevice();
-    void initDevice(QString device);
+    void initDevice(const QString &device);
 
 protected:
     virtual void customEvent(QEvent * e);
index a3d7b28d964b6614545056581ad1ccac47c67d91..6d1f9a38a8f87d11572b19e47fa3e1b9252d6d05 100644 (file)
@@ -31,7 +31,7 @@ class KeyframeHelper : public QWidget
 {
     Q_OBJECT
 public:
-    KeyframeHelper(QWidget *parent = 0);
+    explicit KeyframeHelper(QWidget *parent = 0);
     int value() const;
     int frameLength;
 
index a99d93fff108b81ae5e150f336d8c87cf35725a2..1dc8356bb2c1046c323e20a069f0654b9877a226 100644 (file)
@@ -26,7 +26,7 @@
 #include <KDebug>
 
 
-MarkerDialog::MarkerDialog(DocClipBase *clip, CommentedTime t, Timecode tc, const QString &caption, QWidget * parent) :
+MarkerDialog::MarkerDialog(DocClipBase *clip, const CommentedTime &t, const Timecode &tc, const QString &caption, QWidget * parent) :
         QDialog(parent)
         , m_producer(NULL)
         , m_profile(NULL)
index 710c6348b56448daff558f8a8ea1d383618fe632..5927868755bfe327027348c718dc61fb31397f60 100644 (file)
@@ -44,7 +44,7 @@ class MarkerDialog : public QDialog, public Ui::MarkerDialog_UI
     Q_OBJECT
 
 public:
-    MarkerDialog(DocClipBase *clip, CommentedTime t, Timecode tc, const QString &caption, QWidget * parent = 0);
+    MarkerDialog(DocClipBase *clip, const CommentedTime &t, const Timecode &tc, const QString &caption, QWidget * parent = 0);
     ~MarkerDialog();
     CommentedTime newMarker();
     QImage markerImage() const;
index f572636b91a2db4add72ee316f872a7a7d6ce3c6..032de2ceda90534d49f57c1c58c86f6fc3f35bfe 100644 (file)
@@ -51,7 +51,7 @@ Q_OBJECT public:
     /** @brief Build a MLT Renderer
      *  @param winid The parent widget identifier (required for SDL display). Set to 0 for OpenGL rendering
      *  @param profile The MLT profile used for the capture (default one will be used if empty). */
-    MltDeviceCapture(QString profile, VideoSurface *surface, QWidget *parent = 0);
+    explicit MltDeviceCapture(QString profile, VideoSurface *surface, QWidget *parent = 0);
 
     /** @brief Destroy the MLT Renderer. */
     ~MltDeviceCapture();
index c123be23e2925267a86cca089fe85bc2eb339c9b..45eff43897ea114818123e0739636b00cb6ea303 100644 (file)
@@ -282,22 +282,22 @@ void ParameterPlotter::setNewPoints(bool b)
     m_newPoints = b;
 }
 
-bool ParameterPlotter::isMoveX()
+bool ParameterPlotter::isMoveX() const
 {
     return m_moveX;
 }
 
-bool ParameterPlotter::isMoveY()
+bool ParameterPlotter::isMoveY() const
 {
     return m_moveY;
 }
 
-bool ParameterPlotter::isMoveTimeline()
+bool ParameterPlotter::isMoveTimeline() const
 {
     return m_moveTimeline;
 }
 
-bool ParameterPlotter::isNewPoints()
+bool ParameterPlotter::isNewPoints() const
 {
     return m_newPoints;
 }
index 2961dcab1373a1d1f5f4ac0513e84c341c20b141..358bed254e834d6c9a0b53395cf09e5128c1774e 100644 (file)
@@ -32,10 +32,10 @@ public:
     void setMoveY(bool);
     void setMoveTimeLine(bool);
     void setNewPoints(bool);
-    bool isMoveX();
-    bool isMoveY();
-    bool isMoveTimeline();
-    bool isNewPoints();
+    bool isMoveX() const;
+    bool isMoveY() const;
+    bool isMoveTimeline() const;
+    bool isNewPoints() const;
     void replot(const QString& name = QString());
 private:
     KPlotPoint* m_movepoint;
index 78bcdd37c9eac0e240e35f701c01ba6f977b9307..04c5d60f9e5b373f8eb1183aa4b6c9cf48a1928c 100644 (file)
@@ -25,7 +25,7 @@
 #include <QSlider>
 #include <QHBoxLayout>
 
-PositionEdit::PositionEdit(const QString &name, int pos, int min, int max, const Timecode tc, QWidget* parent) :
+PositionEdit::PositionEdit(const QString &name, int pos, int min, int max, const Timecode&tc, QWidget* parent) :
         QWidget(parent)
 {
     QHBoxLayout *layout = new QHBoxLayout(this);
index ed22ba6ca0cc0df79df593b1b46bf578303cc44f..2666a310f29f48cf06fe93382d7432c261c0436e 100644 (file)
@@ -30,7 +30,7 @@ class PositionEdit : public QWidget
 {
     Q_OBJECT
 public:
-    explicit PositionEdit(const QString &name, int pos, int min, int max, const Timecode tc, QWidget* parent = 0);
+    explicit PositionEdit(const QString &name, int pos, int min, int max, const Timecode& tc, QWidget* parent = 0);
     virtual ~PositionEdit();
     int getPosition() const;
     void setPosition(int pos);
index 9c74ef09852424a7bb58f98c20c3c73534e04d57..f42b1863b62ac471dc1745f6f82bfb838156af80 100644 (file)
@@ -2750,7 +2750,7 @@ void ProjectList::removeClipCut(const QString &id, int in, int out)
     emit projectModified();
 }
 
-SubProjectItem *ProjectList::getSubItem(ProjectItem *clip, QPoint zone)
+SubProjectItem *ProjectList::getSubItem(ProjectItem *clip, const QPoint &zone)
 {
     SubProjectItem *sub = NULL;
     if (clip) {
@@ -2778,7 +2778,7 @@ void ProjectList::slotUpdateClipCut(QPoint p)
     m_commandStack->push(command);
 }
 
-void ProjectList::doUpdateClipCut(const QString &id, const QPoint oldzone, const QPoint zone, const QString &comment)
+void ProjectList::doUpdateClipCut(const QString &id, const QPoint &oldzone, const QPoint &zone, const QString &comment)
 {
     ProjectItem *clip = getItemById(id);
     SubProjectItem *sub = getSubItem(clip, oldzone);
@@ -2798,7 +2798,7 @@ void ProjectList::slotForceProcessing(const QString &id)
     m_render->forceProcessing(id);
 }
 
-void ProjectList::slotAddOrUpdateSequence(const QString frameName)
+void ProjectList::slotAddOrUpdateSequence(const QString &frameName)
 {
     QString fileName = KUrl(frameName).fileName().section('_', 0, -2);
     QStringList list;
@@ -2860,7 +2860,7 @@ QMap <QString, QString> ProjectList::getProxies()
     return list;
 }
 
-void ProjectList::slotCreateProxy(const QString id)
+void ProjectList::slotCreateProxy(const QString &id)
 {
     ProjectItem *item = getItemById(id);
     if (!item || hasPendingJob(item, PROXYJOB) || item->referencedClip()->isPlaceHolder()) return;
@@ -3073,7 +3073,7 @@ void ProjectList::slotCheckJobProcess()
     if (m_jobThreads.futures().isEmpty() || m_jobThreads.futures().count() < KdenliveSettings::proxythreads()) m_jobThreads.addFuture(QtConcurrent::run(this, &ProjectList::slotProcessJobs));
 }
 
-void ProjectList::slotAbortProxy(const QString id, const QString path)
+void ProjectList::slotAbortProxy(const QString &id, const QString &path)
 {
     Q_UNUSED(path)
 
@@ -3228,7 +3228,7 @@ void ProjectList::updateProxyConfig()
     else delete command;
 }
 
-void ProjectList::slotProcessLog(const QString id, int progress, int type, const QString message)
+void ProjectList::slotProcessLog(const QString &id, int progress, int type, const QString &message)
 {
     ProjectItem *item = getItemById(id);
     setJobStatus(item, (JOBTYPE) type, JOBWORKING, progress, message);
index 04a55f6c2c7767dbe460fc3f6d4d46b9c889ed6d..32c0ade5c0ca7957dcfbbb8c086fa900275b0e1e 100644 (file)
@@ -252,8 +252,8 @@ public:
     void addClipCut(const QString &id, int in, int out, const QString desc, bool newItem);
     void removeClipCut(const QString &id, int in, int out);
     void focusTree() const;
-    SubProjectItem *getSubItem(ProjectItem *clip, QPoint zone);
-    void doUpdateClipCut(const QString &id, const QPoint oldzone, const QPoint zone, const QString &comment);
+    SubProjectItem *getSubItem(ProjectItem *clip, const QPoint &zone);
+    void doUpdateClipCut(const QString &id, const QPoint &oldzone, const QPoint &zone, const QString &comment);
     bool hasMissingClips();
     void deleteProjectFolder(QMap <QString, QString> map);
     void selectItemById(const QString &clipId);
@@ -460,16 +460,16 @@ private slots:
     /** @brief Try to find a matching profile for given item. */
     bool adjustProjectProfileToItem(ProjectItem *item = NULL);
     /** @brief Add a sequence from the stopmotion widget. */
-    void slotAddOrUpdateSequence(const QString frameName);
+    void slotAddOrUpdateSequence(const QString &frameName);
     /** @brief A proxy clip was created, update display. */
     void slotGotProxy(const QString &proxyPath);
     void slotGotProxy(ProjectItem *item);
     /** @brief Enable / disable proxy for current clip. */
     void slotProxyCurrentItem(bool doProxy, ProjectItem *itemToProxy = NULL);
     /** @brief Put clip in the proxy waiting list. */
-    void slotCreateProxy(const QString id);
+    void slotCreateProxy(const QString &id);
     /** @brief Stop creation of this clip's proxy. */
-    void slotAbortProxy(const QString id, const QString path);
+    void slotAbortProxy(const QString &id, const QString& path);
     /** @brief Start creation of clip jobs. */
     void slotProcessJobs();
     /** @brief Discard running and pending clip jobs. */
@@ -477,7 +477,7 @@ private slots:
     /** @brief Discard a running clip jobs. */
     void slotCancelRunningJob(const QString id, stringMap);
     /** @brief Update a clip's job status. */
-    void slotProcessLog(const QString, int progress, int, const QString = QString());
+    void slotProcessLog(const QString&, int progress, int, const QString & tmp= QString());
     /** @brief A clip job crashed, inform user. */
     void slotUpdateJobStatus(const QString id, int type, int status, const QString label, const QString actionName, const QString details);
     void slotUpdateJobStatus(ProjectItem *item, int type, int status, const QString &label, const QString &actionName = QString(), const QString details = QString());
index e963b288e131a4dd454513a7dd4743d6ef2fa7ae..1778e6b0b39b9467cc68f3c16b69d9d58fa298eb 100644 (file)
@@ -35,7 +35,7 @@
 #include <QDir>
 #include <kmessagebox.h>
 
-ProjectSettings::ProjectSettings(ProjectList *projectlist, QMap <QString, QString> metadata, QStringList lumas, int videotracks, int audiotracks, const QString &projectPath, bool readOnlyTracks, bool savedProject, QWidget * parent) :
+ProjectSettings::ProjectSettings(ProjectList *projectlist, QMap <QString, QString> metadata, const QStringList &lumas, int videotracks, int audiotracks, const QString &projectPath, bool readOnlyTracks, bool savedProject, QWidget * parent) :
     QDialog(parent), m_savedProject(savedProject), m_projectList(projectlist), m_lumas(lumas)
 {
     setupUi(this);
@@ -482,7 +482,7 @@ QString ProjectSettings::proxyExtension() const
 }
 
 //static
-QStringList ProjectSettings::extractPlaylistUrls(QString path)
+QStringList ProjectSettings::extractPlaylistUrls(const QString &path)
 {
     QStringList urls;
     QDomDocument doc;
@@ -535,7 +535,7 @@ QStringList ProjectSettings::extractPlaylistUrls(QString path)
 
 
 //static
-QStringList ProjectSettings::extractSlideshowUrls(KUrl url)
+QStringList ProjectSettings::extractSlideshowUrls(const KUrl &url)
 {
     QStringList urls;
     QString path = url.directory(KUrl::AppendTrailingSlash);
index 7ff27296b4e67b9b843abbaeec3695e3f88873f7..06cdc8ae7410cfd63baf86d437905689a08c6268 100644 (file)
@@ -32,7 +32,7 @@ class ProjectSettings : public QDialog, public Ui::ProjectSettings_UI
     Q_OBJECT
 
 public:
-    ProjectSettings(ProjectList *projectlist, QMap <QString, QString> metadata, QStringList lumas, int videotracks, int audiotracks, const QString& projectPath, bool readOnlyTracks, bool unsavedProject, QWidget * parent = 0);
+    ProjectSettings(ProjectList *projectlist, QMap <QString, QString> metadata, const QStringList &lumas, int videotracks, int audiotracks, const QString& projectPath, bool readOnlyTracks, bool unsavedProject, QWidget * parent = 0);
     QString selectedProfile() const;
     KUrl selectedFolder() const;
     QPoint tracks();
@@ -46,8 +46,8 @@ public:
     QString proxyParams() const;
     QString proxyExtension() const;
     const QMap <QString, QString> metadata() const;
-    static QStringList extractPlaylistUrls(QString path);
-    static QStringList extractSlideshowUrls(KUrl url);
+    static QStringList extractPlaylistUrls(const QString &path);
+    static QStringList extractSlideshowUrls(const KUrl &url);
 
 public slots:
     virtual void accept();
index d0c7d34a25475494fa3564bf365a2341923de213..e44e59a3ace2f7a4f4269aa0bf3410441730f79b 100644 (file)
@@ -82,7 +82,7 @@ bool operator==(const requestClipInfo &a)
 class MltErrorEvent : public QEvent
 {
 public:
-    MltErrorEvent(QString message) : QEvent(QEvent::User), m_message(message) {}
+    MltErrorEvent(const QString &message) : QEvent(QEvent::User), m_message(message) {}
     QString message() const {
         return m_message;
     }
index fbc9c11bec40b5ffa5e7cf885fba15896cd1fa07..9bba7f58d9a8196be432b8de52245a4484cb4ef1 100644 (file)
@@ -134,7 +134,7 @@ const QString RenderJobItem::metadata() const
 }
 
 
-RenderWidget::RenderWidget(const QString &projectfolder, bool enableProxy, MltVideoProfile profile, QWidget * parent) :
+RenderWidget::RenderWidget(const QString &projectfolder, bool enableProxy, const MltVideoProfile &profile, QWidget * parent) :
         QDialog(parent),
         m_projectFolder(projectfolder),
         m_profile(profile),
@@ -1183,7 +1183,7 @@ int RenderWidget::waitingJobsCount() const
     return count;
 }
 
-void RenderWidget::setProfile(MltVideoProfile profile)
+void RenderWidget::setProfile(const MltVideoProfile &profile)
 {
     m_view.scanning_list->setCurrentIndex(0);
     m_view.rescale_width->setValue(KdenliveSettings::defaultrescalewidth());
@@ -2019,7 +2019,7 @@ void RenderWidget::slotHideLog()
     m_view.error_box->setVisible(false);
 }
 
-void RenderWidget::setRenderProfile(QMap <QString, QString> props)
+void RenderWidget::setRenderProfile(const QMap<QString, QString> &props)
 {
     m_view.scanning_list->setCurrentIndex(props.value("renderscanning").toInt());
     int exportAudio = props.value("renderexportaudio").toInt();
index 2c0002eb3604d0f7b95093d6b7edca574d77cc17..6e74ffa70709e0cf9d8dc415a93bb3469b08e26d 100644 (file)
@@ -114,16 +114,16 @@ class RenderWidget : public QDialog
     Q_OBJECT
 
 public:
-    explicit RenderWidget(const QString &projectfolder, bool enableProxy, MltVideoProfile profile, QWidget * parent = 0);
+    explicit RenderWidget(const QString &projectfolder, bool enableProxy, const MltVideoProfile &profile, QWidget * parent = 0);
     virtual ~RenderWidget();
     void setGuides(QDomElement guidesxml, double duration);
     void focusFirstVisibleItem(const QString &profile = QString());
-    void setProfile(MltVideoProfile profile);
+    void setProfile(const MltVideoProfile& profile);
     void setRenderJob(const QString &dest, int progress = 0);
     void setRenderStatus(const QString &dest, int status, const QString &error);
     void setDocumentPath(const QString &path);
     void reloadProfiles();
-    void setRenderProfile(QMap <QString, QString> props);
+    void setRenderProfile(const QMap <QString, QString>& props);
     int waitingJobsCount() const;
     QString getFreeScriptName(const KUrl &projectName = KUrl(), const QString &prefix = QString());
     bool startWaitingRenderJobs();
index b9ce196f4d5dd86188f3325bacadf47eae342e66..71d2c2b62077d3ef392680892a49c6c87c2faf35 100644 (file)
@@ -269,7 +269,7 @@ QString SlideshowClip::selectedPath()
 }
 
 // static
-int SlideshowClip::getFrameNumberFromPath(KUrl path)
+int SlideshowClip::getFrameNumberFromPath(const KUrl &path)
 {
     QString filter = path.fileName();
     filter = filter.section('.', 0, -2);
@@ -281,7 +281,7 @@ int SlideshowClip::getFrameNumberFromPath(KUrl path)
 }
 
 // static
-QString SlideshowClip::selectedPath(KUrl url, bool isMime, QString extension, QStringList *list)
+QString SlideshowClip::selectedPath(const KUrl &url, bool isMime, QString extension, QStringList *list)
 {
     QString folder;
     if (isMime) {
index b90eda34d29c5d4853a1836ded792dfc91ad9b64..fe0134fb8dfb44771b75c648502fe2f61f659eb4 100644 (file)
@@ -50,9 +50,9 @@ public:
     QString animation() const;
 
     /** @brief Get the image frame number from a file path, for example image_047.jpg will return 47. */
-    static int getFrameNumberFromPath(KUrl path);
+    static int getFrameNumberFromPath(const KUrl &path);
     /** @brief return the url pattern for selected slideshow. */
-    static QString selectedPath(KUrl url, bool isMime, QString extension, QStringList *list);
+    static QString selectedPath(const KUrl &url, bool isMime, QString extension, QStringList *list);
     /** @brief Convert the selection animation style into an affine geometry string. */
     static QString animationToGeometry(const QString &animation, int &ttl);
 
index 08d524d13b274eabfa2bc52c8a39eb8e91b2a13d..789b286103d2bf0659513ec0c6fc04da94b64cf9 100644 (file)
@@ -117,7 +117,7 @@ bool Timecode::df() const
     return m_dropFrameTimecode;
 }
 
-const QString Timecode::mask(GenTime t) const
+const QString Timecode::mask(const GenTime& t) const
 {
     if (t < GenTime()) {
         if (m_dropFrameTimecode) return "#99:99:99,99";
index 35a6f41e87df791254ceb6f7950fe68e0cee2be7..23469756bbb8f3f4daec0f8e623a63243100da1c 100644 (file)
@@ -55,7 +55,7 @@ public:
     const QString getTimecodeFromFrames(int frames) const;
     double fps() const;
     bool df() const;
-    const QString mask(GenTime t = GenTime()) const;
+    const QString mask(const GenTime &t = GenTime()) const;
     QString reformatSeparators(QString duration) const;
 
 private:
index a42d0a4db6f65b96858dd07667b588bbbd19cb59..2bda535620d59ffa18310f16bdfbecf617e715d8 100644 (file)
@@ -64,7 +64,7 @@ const int BLUREFFECT = 1;
 const int SHADOWEFFECT = 2;
 const int TYPEWRITEREFFECT = 3;
 
-TitleWidget::TitleWidget(const KUrl &url, Timecode tc, const QString &projectTitlePath, Render *render, QWidget *parent) :
+TitleWidget::TitleWidget(const KUrl &url, const Timecode &tc, const QString &projectTitlePath, Render *render, QWidget *parent) :
         QDialog(parent),
         Ui::TitleWidget_UI(),
         m_startViewport(NULL),
@@ -1544,7 +1544,7 @@ void TitleWidget::slotInsertUnicode()
     m_unicodeDialog->exec();
 }
 
-void TitleWidget::slotInsertUnicodeString(QString text)
+void TitleWidget::slotInsertUnicodeString(const QString &text)
 {
     QList<QGraphicsItem *> l = graphicsView->scene()->selectedItems();
     if (l.size() > 0) {
index 16afcae5e84e39023bd701ad6f356cb5cc4aad87..4c44b631da33b069d8b2fa405d98c04552f7702f 100644 (file)
@@ -63,7 +63,7 @@ public:
      * @param projectPath default path to save to or load from title documents
      * @param render project renderer
      * @param parent (optional) parent widget */
-    explicit TitleWidget(const KUrl &url, Timecode tc, const QString &projectTitlePath, Render *render, QWidget *parent = 0);
+    explicit TitleWidget(const KUrl &url, const Timecode &tc, const QString &projectTitlePath, Render *render, QWidget *parent = 0);
     virtual ~TitleWidget();
     QDomDocument xml();
     void setXml(const QDomDocument& doc);
@@ -312,7 +312,7 @@ private slots:
     /** Called whenever text properties change (font e.g.) */
     void slotUpdateText();
     void slotInsertUnicode();
-    void slotInsertUnicodeString(QString);
+    void slotInsertUnicodeString(const QString&);
 
     void displayBackgroundFrame();
 
index ad0665f1ab4bb63f5bcb4e3782bc5aa4b024e7d1..7d40b96c1ecd526f81ee70aaabb9df7e2657557b 100644 (file)
@@ -44,7 +44,7 @@ void TracksDelegate::setEditorData(QWidget *editor, const QModelIndex &index) co
     QComboBox *comboBox = qobject_cast<QComboBox *>(editor);
     if (!comboBox)
         return;
-    int pos = comboBox->findText(index.model()->data(index).toString(), Qt::MatchExactly);
+    const int pos = comboBox->findText(index.model()->data(index).toString(), Qt::MatchExactly);
     comboBox->setCurrentIndex(pos);
 }
 
@@ -106,7 +106,7 @@ const QList <TrackInfo> TracksConfigDialog::tracksList()
 {
     QList <TrackInfo> tracks;
     TrackInfo info;
-    for (int i = table->rowCount() - 1; i >= 0; i--) {
+    for (int i = table->rowCount() - 1; i >= 0; --i) {
         info.trackName = table->item(i, 0)->text();
         QTableWidgetItem *item = table->item(i, 1);
         if (item->text() == i18n("Audio")) {
@@ -134,7 +134,7 @@ void TracksConfigDialog::setupOriginal(int selected)
 
     QStringList numbers;
     TrackInfo info;
-    for (int i = m_doc->tracksCount() - 1; i >= 0; i--) {
+    for (int i = m_doc->tracksCount() - 1; i >= 0; --i) {
         numbers << QString::number(i);
         info = m_doc->trackInfoAt(m_doc->tracksCount() - i - 1);
         table->setItem(i, 0, new QTableWidgetItem(info.trackName));
index b2303ecb6d429f756bbbbba555afd30777fcad9f..6bc3d6409e82143246890578d36eb0f8de7b4a0f 100644 (file)
@@ -179,7 +179,7 @@ int TrackView::outPoint() const
     return m_ruler->outPoint();
 }
 
-void TrackView::slotSetZone(QPoint p, bool updateDocumentProperties)
+void TrackView::slotSetZone(const QPoint &p, bool updateDocumentProperties)
 {
     m_ruler->setZone(p);
     if (updateDocumentProperties) m_doc->setZone(p.x(), p.y());
@@ -1011,7 +1011,7 @@ QDomElement TrackView::getEffectByTag(const QString &effecttag, const QString &e
 }
 
 
-DocClipBase *TrackView::getMissingProducer(const QString id) const
+DocClipBase *TrackView::getMissingProducer(const QString &id) const
 {
     QDomElement missingXml;
     QDomDocument doc = m_doc->toXml();
@@ -1130,7 +1130,7 @@ void TrackView::updateProjectFps()
     m_trackview->updateProjectFps();
 }
 
-void TrackView::slotRenameTrack(int ix, QString name)
+void TrackView::slotRenameTrack(int ix, const QString &name)
 {
     int tracknumber = m_doc->tracksCount() - ix;
     QList <TrackInfo> tracks = m_doc->tracksList();
@@ -1168,7 +1168,7 @@ void TrackView::slotUpdateTrackEffectState(int ix)
     widgets.at(m_doc->tracksCount() - ix - 1)->updateEffectLabel(m_doc->trackInfoAt(ix).effectsList.effectNames());
 }
 
-void TrackView::slotSaveTimelinePreview(const QString path)
+void TrackView::slotSaveTimelinePreview(const QString &path)
 {
     QImage img(width(), height(), QImage::Format_ARGB32_Premultiplied);
     img.fill(palette().base().color().rgb());
index b46ec00c0eef484d1af3a1cc922b2cbe58fd3e97..49399b6c22e2602a7c192224bcf7426e72ae22f9 100644 (file)
@@ -86,9 +86,9 @@ public slots:
     void slotDeleteClip(const QString &clipId);
     void slotChangeZoom(int horizontal, int vertical = -1);
     void setDuration(int dur);
-    void slotSetZone(QPoint p, bool updateDocumentProperties = true);
+    void slotSetZone(const QPoint &p, bool updateDocumentProperties = true);
     /** @brief Save a snapshot image of current timeline view */
-    void slotSaveTimelinePreview(const QString path);
+    void slotSaveTimelinePreview(const QString &path);
 private:
     CustomRuler *m_ruler;
     CustomTrackView *m_trackview;
@@ -107,7 +107,7 @@ private:
     
     void parseDocument(const QDomDocument &doc);
     int slotAddProjectTrack(int ix, QDomElement xml, bool locked, QDomNodeList producers);
-    DocClipBase *getMissingProducer(const QString id) const;
+    DocClipBase *getMissingProducer(const QString &id) const;
     void adjustTrackHeaders();
     /** @brief Add effects from the xml. Returns true if some effect was upgraded, false if everything went fine.*/
     void slotAddProjectEffects(QDomNodeList effects, QDomElement parentNode, ClipItem *clip, int trackIndex);
@@ -132,7 +132,7 @@ private slots:
     /** @brief Changes the name of a track.
     * @param ix Number of the track
     * @param name New name */
-    void slotRenameTrack(int ix, QString name);
+    void slotRenameTrack(int ix, const QString &name);
     void slotRepaintTracks();
 
     /** @brief Adjusts the margins of the header area.
index 55655f3117d129d2b2d959a271e15a344a65efe2..0d45c205e99688cc0fb64dc6bea6e1a3524de8d7 100644 (file)
@@ -69,7 +69,7 @@ TransitionSettings::TransitionSettings(Monitor *monitor, QWidget* parent) :
     connect(m_effectEdit, SIGNAL(parameterChanged(QDomElement,QDomElement,int)), this , SLOT(slotUpdateEffectParams(QDomElement,QDomElement)));
 }
 
-void TransitionSettings::updateProjectFormat(MltVideoProfile profile, Timecode t, const QList<TrackInfo> &info)
+void TransitionSettings::updateProjectFormat(const MltVideoProfile &profile, const Timecode &t, const QList<TrackInfo> &info)
 {
     m_effectEdit->updateProjectFormat(profile, t);
     m_tracks = info;
@@ -159,7 +159,7 @@ void TransitionSettings::slotTransitionTrackChanged()
     m_effectEdit->updateParameter("transition_btrack", QString::number(ix));
 }
 
-void TransitionSettings::slotTransitionItemSelected(Transition* t, int nextTrack, QPoint p, bool update)
+void TransitionSettings::slotTransitionItemSelected(Transition* t, int nextTrack, const QPoint &p, bool update)
 {
     setEnabled(t != NULL);
     m_effectEdit->setFrameSize(p);
index 769e94ba5ba5b1b2ce86b8db93051fe0a3c2aa84..ddda5c9de10bd103fc3e7b69e96fbfda9236fd9c 100644 (file)
@@ -34,7 +34,7 @@ class TransitionSettings : public QWidget, public Ui::TransitionSettings_UI
 public:
     explicit TransitionSettings(Monitor *monitor, QWidget* parent = 0);
     void raiseWindow(QWidget*);
-    void updateProjectFormat(MltVideoProfile profile, Timecode t, const QList <TrackInfo>& info);
+    void updateProjectFormat(const MltVideoProfile &profile, const Timecode &t, const QList <TrackInfo>& info);
     void updateTimecodeFormat();
     void setKeyframes(const QString &data, int maximum);
 
@@ -48,7 +48,7 @@ private:
     void updateTrackList();
 
 public slots:
-    void slotTransitionItemSelected(Transition* t, int nextTrack, QPoint p, bool update);
+    void slotTransitionItemSelected(Transition* t, int nextTrack, const QPoint &p, bool update);
     void slotTransitionChanged(bool reinit = true, bool updateCurrent = false);
     void slotUpdateEffectParams(const QDomElement, const QDomElement);
 
index 6f0dae0ccf12508c0dd293813bdacc42e87c4fed..acea4857e14c5a7a4160095101a33639dca9dc81 100644 (file)
@@ -289,7 +289,7 @@ void UnicodeDialog::writeChoices()
 /**
  * \brief Validates the entered Unicode number and displays its Unicode character.
  */
-void UnicodeDialog::slotTextChanged(QString text)
+void UnicodeDialog::slotTextChanged(const QString &text)
 {
     unicodeNumber->blockSignals(true);
 
index 74f2cec04805d85ec90ce60e74856012858a5ffe..b34f3c01a9240c5656df70e58ee66763809345a5 100644 (file)
@@ -72,7 +72,7 @@ signals:
     void charSelected(const QString&);
 
 private slots:
-    void slotTextChanged(QString text);
+    void slotTextChanged(const QString &text);
     void slotReturnPressed();
     void slotNextUnicode();
     void slotPrevUnicode();
index c613feb03f1f3e27060484d31922b65050f9c107..617ef1917b41fc6aa86dcf804b4f2935364fa9ea 100644 (file)
@@ -129,7 +129,7 @@ void VideoGLWidget::paintGL()
     }
 }
 
-void VideoGLWidget::showImage(const QImage image)
+void VideoGLWidget::showImage(const QImage &image)
 {
     m_image_width = image.width();
     m_image_height = image.height();
index a7cd3b215afcdce87caf7612adc9f6360c49bb48..c9016e64d2b880fa50234b09ade38e56aba6d748 100644 (file)
@@ -28,7 +28,7 @@ private:
     Qt::WindowFlags m_baseFlags;
 
 public slots:
-    void showImage(const QImage image);
+    void showImage(const QImage &image);
 
 protected:
     void initializeGL();