QmlDesigner: Improve return values

You can return by const reference if you the ownership of the return
value is non local. If it is local you should return by value. In that
case it should be non const because otherwise the compile cannot move
the return value.

Change-Id: I2a4b08dd0e67a2c2741a46bdd7def842c1db3fbf
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
Reviewed-by: Mahmoud Badri <mahmoud.badri@qt.io>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/plugins/qmldesigner/designercore/include/qmltimelinekeyframegroup.h b/src/plugins/qmldesigner/designercore/include/qmltimelinekeyframegroup.h
index c37be6934b..a20eeae55e 100644
--- a/src/plugins/qmldesigner/designercore/include/qmltimelinekeyframegroup.h
+++ b/src/plugins/qmldesigner/designercore/include/qmltimelinekeyframegroup.h
@@ -65,9 +65,9 @@ public:
 
     ModelNode keyframe(qreal position) const;
 
-    const QList<ModelNode> keyframes() const;
+    QList<ModelNode> keyframes() const;
 
-    const QList<ModelNode> keyframePositions() const;
+    QList<ModelNode> keyframePositions() const;
 
     static bool isValidKeyframe(const ModelNode &node);
     static bool checkKeyframesType(const ModelNode &node);