From eb9172d1a0caf929bb7787e7b2cf040fe5da42eb Mon Sep 17 00:00:00 2001 From: Jochen Becher Date: Sat, 13 Jan 2018 19:35:05 +0100 Subject: Modelinglib: Avoid temporary extensive copy Change-Id: I25fb6f671cc392836b829889987c516aa722bc92 Reviewed-by: Tobias Hunger --- .../qmt/model_controller/modelcontroller.cpp | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'src/libs/modelinglib/qmt/model_controller/modelcontroller.cpp') diff --git a/src/libs/modelinglib/qmt/model_controller/modelcontroller.cpp b/src/libs/modelinglib/qmt/model_controller/modelcontroller.cpp index ff74d61ff3..88a4520eb3 100644 --- a/src/libs/modelinglib/qmt/model_controller/modelcontroller.cpp +++ b/src/libs/modelinglib/qmt/model_controller/modelcontroller.cpp @@ -1060,9 +1060,9 @@ void ModelController::renewElementKey(MElement *element, QHash *renewe } auto object = dynamic_cast(element); if (object) { - foreach (const Handle &child, object->children()) + for (const Handle &child : object->children()) renewElementKey(child.target(), renewedKeys); - foreach (const Handle &relation, object->relations()) + for (const Handle &relation : object->relations()) renewElementKey(relation.target(), renewedKeys); } } @@ -1071,9 +1071,9 @@ void ModelController::renewElementKey(MElement *element, QHash *renewe void ModelController::updateRelationKeys(MElement *element, const QHash &renewedKeys) { if (auto object = dynamic_cast(element)) { - foreach (const Handle &handle, object->relations()) + for (const Handle &handle : object->relations()) updateRelationEndKeys(handle.target(), renewedKeys); - foreach (const Handle &child, object->children()) + for (const Handle &child : object->children()) updateRelationKeys(child.target(), renewedKeys); } else if (auto relation = dynamic_cast(element)) { updateRelationEndKeys(relation, renewedKeys); @@ -1097,9 +1097,9 @@ void ModelController::mapObject(MObject *object) if (object) { QMT_CHECK(!m_objectsMap.contains(object->uid())); m_objectsMap.insert(object->uid(), object); - foreach (const Handle &child, object->children()) + for (const Handle &child : object->children()) mapObject(child.target()); - foreach (const Handle &relation, object->relations()) + for (const Handle &relation : object->relations()) mapRelation(relation.target()); } } @@ -1108,9 +1108,9 @@ void ModelController::unmapObject(MObject *object) { if (object) { QMT_CHECK(m_objectsMap.contains(object->uid())); - foreach (const Handle &relation, object->relations()) + for (const Handle &relation : object->relations()) unmapRelation(relation.target()); - foreach (const Handle &child, object->children()) + for (const Handle &child : object->children()) unmapObject(child.target()); m_objectsMap.remove(object->uid()); } @@ -1209,7 +1209,7 @@ void ModelController::verifyModelIntegrity(const MObject *object, QHashcontains(object->uid()), return); objectsMap->insert(object->uid(), object); - foreach (const Handle &handle, object->relations()) { + for (const Handle &handle : object->relations()) { MRelation *relation = handle.target(); if (relation) { QMT_ASSERT(!relationsMap->contains(relation->uid()), return); @@ -1222,7 +1222,7 @@ void ModelController::verifyModelIntegrity(const MObject *object, QHashinsert(relation->endBUid(), relation); } } - foreach (const Handle &handle, object->children()) { + for (const Handle &handle : object->children()) { MObject *childObject = handle.target(); if (childObject) verifyModelIntegrity(childObject, objectsMap, relationsMap, objectRelationsMap); -- cgit v1.2.1