From f223c83a30c943983283f5e9eacc5bbcc8278329 Mon Sep 17 00:00:00 2001 From: Montel Laurent Date: Sat, 1 Jun 2013 16:05:53 +0200 Subject: [PATCH] const'ref --- src/kdenlivedoc.cpp | 6 +++--- src/kdenlivedoc.h | 6 +++--- src/keyframehelper.cpp | 12 ++++++++---- 3 files changed, 14 insertions(+), 10 deletions(-) diff --git a/src/kdenlivedoc.cpp b/src/kdenlivedoc.cpp index 07471af0..05851246 100644 --- a/src/kdenlivedoc.cpp +++ b/src/kdenlivedoc.cpp @@ -844,7 +844,7 @@ void KdenliveDoc::setProjectFolder(KUrl url) updateProjectFolderPlacesEntry(); } -void KdenliveDoc::moveProjectData(KUrl url) +void KdenliveDoc::moveProjectData(const KUrl &url) { QList list = m_clipManager->documentClipList(); KUrl::List cacheUrls; @@ -1258,7 +1258,7 @@ void KdenliveDoc::deleteClip(const QString &clipId) emit signalDeleteProjectClip(clipId); } -void KdenliveDoc::slotAddClipList(const KUrl::List urls, stringMap data) +void KdenliveDoc::slotAddClipList(const KUrl::List &urls, const stringMap &data) { m_clipManager->slotAddClipList(urls, data); //emit selectLastAddedClip(QString::number(m_clipManager->lastClipId())); @@ -1266,7 +1266,7 @@ void KdenliveDoc::slotAddClipList(const KUrl::List urls, stringMap data) } -void KdenliveDoc::slotAddClipFile(const KUrl &url, stringMap data) +void KdenliveDoc::slotAddClipFile(const KUrl &url, const stringMap &data) { m_clipManager->slotAddClipFile(url, data); emit selectLastAddedClip(QString::number(m_clipManager->lastClipId())); diff --git a/src/kdenlivedoc.h b/src/kdenlivedoc.h index 0851011c..610d4a90 100644 --- a/src/kdenlivedoc.h +++ b/src/kdenlivedoc.h @@ -84,7 +84,7 @@ public: * * If the clip wasn't added before, it tries to add it to the project. */ bool addClipInfo(QDomElement elem, QDomElement orig, const QString &clipId); - void slotAddClipList(const KUrl::List urls, stringMap data = stringMap()); + void slotAddClipList(const KUrl::List &urls, const stringMap &data = stringMap()); void deleteClip(const QString &clipId); int getFramePos(QString duration); DocClipBase *getBaseClip(const QString &clipId); @@ -196,7 +196,7 @@ private: QList m_tracksList; void setNewClipResource(const QString &id, const QString &path); QString searchFileRecursively(const QDir &dir, const QString &matchSize, const QString &matchHash) const; - void moveProjectData(KUrl url); + void moveProjectData(const KUrl &url); bool checkDocumentClips(QDomNodeList infoproducers); /** @brief Creates a new project. */ @@ -222,7 +222,7 @@ public slots: * @param mod (optional) true if the document has to be saved */ void setModified(bool mod = true); void checkProjectClips(bool displayRatioChanged = false, bool fpsChanged = false); - void slotAddClipFile(const KUrl &url, stringMap data); + void slotAddClipFile(const KUrl &url, const stringMap &data); private slots: void slotAutoSave(); diff --git a/src/keyframehelper.cpp b/src/keyframehelper.cpp index 20277f9d..b0bbb391 100644 --- a/src/keyframehelper.cpp +++ b/src/keyframehelper.cpp @@ -167,10 +167,12 @@ void KeyframeHelper::mouseMoveEvent(QMouseEvent * event) m_extraGeometries[i]->remove(m_movingItem.frame()); } int pos = qBound(0, (int)(xPos / m_scale), frameLength); - if (KdenliveSettings::snaptopoints() && qAbs(pos - m_position) < 5) pos = m_position; + if (KdenliveSettings::snaptopoints() && qAbs(pos - m_position) < 5) + pos = m_position; m_movingItem.frame(pos); for (int i = 0; i < m_extraMovingItems.count(); ++i) { - if (m_extraMovingItems.at(i)) m_extraMovingItems[i]->frame(pos); + if (m_extraMovingItems.at(i)) + m_extraMovingItems[i]->frame(pos); } update(); return; @@ -211,7 +213,8 @@ void KeyframeHelper::mouseReleaseEvent(QMouseEvent * event) m_movingKeyframe = false; for (int i = 0; i < m_extraGeometries.count(); ++i) { - if (m_extraMovingItems.at(i)) m_extraGeometries[i]->insert(m_extraMovingItems.at(i)); + if (m_extraMovingItems.at(i)) + m_extraGeometries[i]->insert(m_extraMovingItems.at(i)); } emit keyframeMoved(m_position); @@ -317,7 +320,8 @@ int KeyframeHelper::value() const void KeyframeHelper::setValue(const int pos) { - if (pos == m_position || m_geom == NULL) return; + if (pos == m_position || m_geom == NULL) + return; if (pos == m_seekPosition) { m_seekPosition = SEEK_INACTIVE; } -- 2.39.2