QmlDesigner: Refactor allQmlItemsRecursive

Change-Id: If8b98e008e59de1eae87108a3256005fd85981b9
Reviewed-by: Thomas Hartmann <Thomas.Hartmann@digia.com>

1 files changed, 8 insertions, 10 deletions

diff --git a/src/plugins/qmldesigner/designercore/model/qmlobjectnode.cpp b/src/plugins/qmldesigner/designercore/model/qmlobjectnode.cpp
index fc1709b38c..99fd9678b1 100644
--- a/src/plugins/qmldesigner/designercore/model/qmlobjectnode.cpp
+++ b/src/plugins/qmldesigner/designercore/model/qmlobjectnode.cpp
@@ -322,20 +322,18 @@ QList<QmlModelStateOperation> QmlObjectNode::allAffectingStatesOperations() cons
 
 static QList<QmlItemNode> allQmlItemsRecursive(const QmlItemNode &qmlItemNode)
 {
-    QList<QmlItemNode> returnList;
+    QList<QmlItemNode> qmlItemNodeList;
 
     if (qmlItemNode.isValid()) {
-        returnList.append(qmlItemNode);
-        QList<QmlItemNode> allChildNodes;
-        foreach (const ModelNode &node, qmlItemNode.modelNode().allDirectSubModelNodes()) {
-            if (QmlItemNode::isValidQmlItemNode(node))
-                allChildNodes.append(node);
-        }
-        foreach (const QmlItemNode &node, allChildNodes) {
-            returnList.append(allQmlItemsRecursive(node));
+        qmlItemNodeList.append(qmlItemNode);
+
+        foreach (const ModelNode &modelNode, qmlItemNode.modelNode().allDirectSubModelNodes()) {
+            if (QmlItemNode::isValidQmlItemNode(modelNode))
+                qmlItemNodeList.append(allQmlItemsRecursive(modelNode));
         }
     }
-    return returnList;
+
+    return qmlItemNodeList;
 }
 
 QList<QmlModelState> QmlObjectNode::allDefinedStates() const