From e8f43a6f96b987e8476674484346fcb7d9e2eb6c Mon Sep 17 00:00:00 2001 From: Montel Laurent Date: Wed, 15 May 2013 20:08:32 +0200 Subject: [PATCH] Still const'ref. Minor optimization --- src/abstractclipitem.cpp | 2 +- src/abstractgroupitem.cpp | 6 +++--- src/abstractgroupitem.h | 4 ++-- src/clipitem.cpp | 8 ++++---- src/clipitem.h | 8 ++++---- src/clipmanager.cpp | 4 ++-- src/clipmanager.h | 4 ++-- src/customtrackview.cpp | 2 +- src/customtrackview.h | 2 +- src/databackup/backupwidget.h | 2 -- src/geometrywidget.cpp | 2 +- src/geometrywidget.h | 2 +- src/onmonitoritems/rotoscoping/splineitem.cpp | 2 +- src/onmonitoritems/rotoscoping/splineitem.h | 2 +- src/simplekeyframes/simplekeyframewidget.cpp | 2 +- src/simplekeyframes/simplekeyframewidget.h | 3 +-- 16 files changed, 26 insertions(+), 29 deletions(-) diff --git a/src/abstractclipitem.cpp b/src/abstractclipitem.cpp index f56d05b0..966604cf 100644 --- a/src/abstractclipitem.cpp +++ b/src/abstractclipitem.cpp @@ -279,7 +279,7 @@ void AbstractClipItem::drawKeyFrames(QPainter *painter, const QTransform &transf x2 = br.right(); if (limitedKeyFrames) { QMap::const_iterator end = m_keyframes.constEnd(); - end--; + --end; x2 = x1 + maxw * (end.key() - start); x1 += maxw * (m_keyframes.constBegin().key() - start); } diff --git a/src/abstractgroupitem.cpp b/src/abstractgroupitem.cpp index d3c873ad..e52a8a33 100644 --- a/src/abstractgroupitem.cpp +++ b/src/abstractgroupitem.cpp @@ -107,7 +107,7 @@ QPainterPath AbstractGroupItem::transitionGroupShape(QPointF offset) const return groupShape(TRANSITIONWIDGET, offset); } -QPainterPath AbstractGroupItem::groupShape(GRAPHICSRECTITEM type, QPointF offset) const +QPainterPath AbstractGroupItem::groupShape(GRAPHICSRECTITEM type, const QPointF &offset) const { QPainterPath path; QList children = childItems(); @@ -130,7 +130,7 @@ QPainterPath AbstractGroupItem::groupShape(GRAPHICSRECTITEM type, QPointF offset return path; } -QPainterPath AbstractGroupItem::spacerGroupShape(GRAPHICSRECTITEM type, QPointF offset) const +QPainterPath AbstractGroupItem::spacerGroupShape(GRAPHICSRECTITEM type, const QPointF &offset) const { QPainterPath path; QList children = childItems(); @@ -212,7 +212,7 @@ QVariant AbstractGroupItem::itemChange(GraphicsItemChange change, const QVariant int xpos = projectScene()->getSnapPointForPos((int)(start.x() + newPos.x() - pos().x()), KdenliveSettings::snaptopoints()); xpos = qMax(xpos, 0); - //kDebug()<<"GRP XPOS:"< m_resizeInfos; }; diff --git a/src/clipitem.cpp b/src/clipitem.cpp index fb62f053..269e7a76 100644 --- a/src/clipitem.cpp +++ b/src/clipitem.cpp @@ -648,7 +648,7 @@ void ClipItem::slotGetEndThumb() } -void ClipItem::slotSetStartThumb(QImage img) +void ClipItem::slotSetStartThumb(const QImage &img) { if (!img.isNull() && img.format() == QImage::Format_ARGB32) { QPixmap pix = QPixmap::fromImage(img); @@ -659,7 +659,7 @@ void ClipItem::slotSetStartThumb(QImage img) } } -void ClipItem::slotSetEndThumb(QImage img) +void ClipItem::slotSetEndThumb(const QImage &img) { if (!img.isNull() && img.format() == QImage::Format_ARGB32) { QPixmap pix = QPixmap::fromImage(img); @@ -690,12 +690,12 @@ void ClipItem::slotThumbReady(int frame, QImage img) } } -void ClipItem::slotSetStartThumb(const QPixmap pix) +void ClipItem::slotSetStartThumb(const QPixmap &pix) { m_startPix = pix; } -void ClipItem::slotSetEndThumb(const QPixmap pix) +void ClipItem::slotSetEndThumb(const QPixmap &pix) { m_endPix = pix; } diff --git a/src/clipitem.h b/src/clipitem.h index da52dbde..d71ebb5e 100644 --- a/src/clipitem.h +++ b/src/clipitem.h @@ -251,16 +251,16 @@ private slots: void slotGotAudioData(); void slotPrepareAudioThumb(double pixelForOneFrame, int startpixel, int endpixel, int channels, int pixelHeight); void animate(qreal value); - void slotSetStartThumb(QImage img); - void slotSetEndThumb(QImage img); + void slotSetStartThumb(const QImage &img); + void slotSetEndThumb(const QImage &img); void slotThumbReady(int frame, QImage img); /** @brief The thumbnailer has finished to cache all required thumbs. */ void slotGotThumbsCache(); public slots: void slotFetchThumbs(); - void slotSetStartThumb(const QPixmap pix); - void slotSetEndThumb(const QPixmap pix); + void slotSetStartThumb(const QPixmap &pix); + void slotSetEndThumb(const QPixmap &pix); void slotUpdateRange(); signals: diff --git a/src/clipmanager.cpp b/src/clipmanager.cpp index 52aa612a..80396991 100644 --- a/src/clipmanager.cpp +++ b/src/clipmanager.cpp @@ -527,7 +527,7 @@ void ClipManager::slotAddClip(KIO::Job *job, const KUrl &, const KUrl &dst) slotAddClipList(KUrl::List () << dst, data); } -void ClipManager::slotAddClipList(const KUrl::List urls, QMap data) +void ClipManager::slotAddClipList(const KUrl::List &urls, QMap data) { QUndoCommand *addClips = new QUndoCommand(); // Update list of removable volumes @@ -641,7 +641,7 @@ void ClipManager::slotAddClipList(const KUrl::List urls, QMap } } -void ClipManager::slotAddClipFile(const KUrl &url, QMap data) +void ClipManager::slotAddClipFile(const KUrl &url, const QMap &data) { slotAddClipList(KUrl::List(url), data); } diff --git a/src/clipmanager.h b/src/clipmanager.h index 2a06970b..b980f11c 100644 --- a/src/clipmanager.h +++ b/src/clipmanager.h @@ -88,14 +88,14 @@ Q_OBJECT public: * @param url file to add * @param group name of the group to insert the file in (can be empty) * @param groupId id of the group (if any) */ - void slotAddClipFile(const KUrl &url, QMap data); + void slotAddClipFile(const KUrl &url, const QMap &data); /** @brief Adds a list of files to the project. * @param urls files to add * @param group name of the group to insert the files in (can be empty) * @param groupId id of the group (if any) * It checks for duplicated items and asks to the user for instructions. */ - void slotAddClipList(const KUrl::List urls, QMap data); + void slotAddClipList(const KUrl::List &urls, QMap data); void slotAddTextClipFile(const QString &titleName, int out, const QString &xml, const QString &group, const QString &groupId); void slotAddTextTemplateClip(QString titleName, const KUrl &path, const QString &group, const QString &groupId); void slotAddXmlClipFile(const QString &name, const QDomElement &xml, const QString &group, const QString &groupId); diff --git a/src/customtrackview.cpp b/src/customtrackview.cpp index 649eb65a..b0ff7ba2 100644 --- a/src/customtrackview.cpp +++ b/src/customtrackview.cpp @@ -5669,7 +5669,7 @@ void CustomTrackView::slotLoadClipMarkers(const QString &id) else delete command; } -void CustomTrackView::addMarker(const QString &id, const CommentedTime marker) +void CustomTrackView::addMarker(const QString &id, const CommentedTime &marker) { DocClipBase *base = m_document->clipManager()->getClipById(id); if (base == NULL) return; diff --git a/src/customtrackview.h b/src/customtrackview.h index ecdf17b6..0242a211 100644 --- a/src/customtrackview.h +++ b/src/customtrackview.h @@ -76,7 +76,7 @@ public: void deleteClip(ItemInfo info, bool refresh = true); void slotDeleteClipMarker(const QString &comment, const QString &id, const GenTime &position); void slotDeleteAllClipMarkers(const QString &id); - void addMarker(const QString &id, const CommentedTime marker); + void addMarker(const QString &id, const CommentedTime &marker); void addData(const QString &id, const QString &key, const QString &data); void setScale(double scaleFactor, double verticalScale); void deleteClip(const QString &clipId); diff --git a/src/databackup/backupwidget.h b/src/databackup/backupwidget.h index bb6b87e1..1c71bbae 100644 --- a/src/databackup/backupwidget.h +++ b/src/databackup/backupwidget.h @@ -51,8 +51,6 @@ private slots: private: QString m_projectWildcard; - -signals: }; diff --git a/src/geometrywidget.cpp b/src/geometrywidget.cpp index e49327dc..661b388d 100644 --- a/src/geometrywidget.cpp +++ b/src/geometrywidget.cpp @@ -328,7 +328,7 @@ QString GeometryWidget::getExtraValue(const QString &name) const return val; } -void GeometryWidget::setupParam(const QDomElement elem, int minframe, int maxframe) +void GeometryWidget::setupParam(const QDomElement &elem, int minframe, int maxframe) { m_inPoint = minframe; m_outPoint = maxframe; diff --git a/src/geometrywidget.h b/src/geometrywidget.h index fc5e245a..dceadd6d 100644 --- a/src/geometrywidget.h +++ b/src/geometrywidget.h @@ -66,7 +66,7 @@ public slots: * @param elem DomElement representing this effect parameter * @param minframe In point of the clip * @param maxframe Out point of the clip */ - void setupParam(const QDomElement elem, int minframe, int maxframe); + void setupParam(const QDomElement &elem, int minframe, int maxframe); /** @brief Updates position of the local timeline to @param relTimelinePos. */ void slotSyncPosition(int relTimelinePos); void slotResetKeyframes(); diff --git a/src/onmonitoritems/rotoscoping/splineitem.cpp b/src/onmonitoritems/rotoscoping/splineitem.cpp index fa3c54f9..563cd376 100644 --- a/src/onmonitoritems/rotoscoping/splineitem.cpp +++ b/src/onmonitoritems/rotoscoping/splineitem.cpp @@ -252,7 +252,7 @@ void SplineItem::hoverMoveEvent(QGraphicsSceneHoverEvent* event) unsetCursor(); } -int SplineItem::getClosestPointOnCurve(QPointF point, double *tFinal) +int SplineItem::getClosestPointOnCurve(const QPointF &point, double *tFinal) { // TODO: proper minDiff qreal diff = 10000, param = 0; diff --git a/src/onmonitoritems/rotoscoping/splineitem.h b/src/onmonitoritems/rotoscoping/splineitem.h index d6b595c2..376c6223 100644 --- a/src/onmonitoritems/rotoscoping/splineitem.h +++ b/src/onmonitoritems/rotoscoping/splineitem.h @@ -51,7 +51,7 @@ protected: virtual void hoverMoveEvent(QGraphicsSceneHoverEvent *event); private: - int getClosestPointOnCurve(QPointF point, double *tFinal); + int getClosestPointOnCurve(const QPointF &point, double *tFinal); bool m_closed; bool m_editing; diff --git a/src/simplekeyframes/simplekeyframewidget.cpp b/src/simplekeyframes/simplekeyframewidget.cpp index 00fee2c9..af8e477e 100644 --- a/src/simplekeyframes/simplekeyframewidget.cpp +++ b/src/simplekeyframes/simplekeyframewidget.cpp @@ -98,7 +98,7 @@ void SimpleKeyframeWidget::slotSetPosition(int pos, bool update) emit positionChanged(pos); } -int SimpleKeyframeWidget::getPosition() +int SimpleKeyframeWidget::getPosition() const { return m_time->getValue(); } diff --git a/src/simplekeyframes/simplekeyframewidget.h b/src/simplekeyframes/simplekeyframewidget.h index 5564e0ac..edf6b3aa 100644 --- a/src/simplekeyframes/simplekeyframewidget.h +++ b/src/simplekeyframes/simplekeyframewidget.h @@ -37,7 +37,7 @@ public: explicit SimpleKeyframeWidget(const Timecode &t, int duration, QWidget* parent = 0); ~SimpleKeyframeWidget(); - int getPosition(); + int getPosition() const; void setKeyframes(const QList &keyframes); void addKeyframe(int pos = -1); @@ -60,7 +60,6 @@ private: QToolButton *m_buttonAddDelete; QToolButton *m_buttonPrevious; QToolButton *m_buttonNext; - //QToolButton *m_buttonSync; TimecodeDisplay *m_time; }; -- 2.39.2