QmlDesigner: performance-for-range-copy

Avoid copying the loop variable on each iteration if a const reference
would suffice.

For QPointers<>, this change only adds a const.

Change-Id: I5abe7d793cd46859a9a4f9304ec1bd41f899d72c
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>

2 files changed, 2 insertions, 2 deletions

diff --git a/src/plugins/qmldesigner/components/timelineeditor/timelineactions.cpp b/src/plugins/qmldesigner/components/timelineeditor/timelineactions.cpp
index 4d2523b3a3..c9271e490c 100644
--- a/src/plugins/qmldesigner/components/timelineeditor/timelineactions.cpp
+++ b/src/plugins/qmldesigner/components/timelineeditor/timelineactions.cpp
@@ -252,7 +252,7 @@ std::vector<std::tuple<ModelNode, qreal>> getFramesRelative(const ModelNode &par
 
     if (!sortedByTime.empty()) {
         qreal firstTime = getTime(sortedByTime.first());
-        for (ModelNode keyframe : sortedByTime)
+        for (const ModelNode &keyframe : sortedByTime)
             result.emplace_back(keyframe, getTime(keyframe) - firstTime);
     }
 
diff --git a/src/plugins/qmldesigner/components/timelineeditor/timelinesectionitem.cpp b/src/plugins/qmldesigner/components/timelineeditor/timelinesectionitem.cpp
index d8d34fe1d1..7e8bfb6f50 100644
--- a/src/plugins/qmldesigner/components/timelineeditor/timelinesectionitem.cpp
+++ b/src/plugins/qmldesigner/components/timelineeditor/timelinesectionitem.cpp
@@ -209,7 +209,7 @@ ModelNode TimelineSectionItem::targetNode() const
 QVector<qreal> TimelineSectionItem::keyframePositions() const
 {
     QVector<qreal> out;
-    for (auto frame : m_timeline.keyframeGroupsForTarget(m_targetNode))
+    for (const auto &frame : m_timeline.keyframeGroupsForTarget(m_targetNode))
         out.append(timelineScene()->keyframePositions(frame));
 
     return out;