Tooling: Use QHash/QMap's constFind() to avoid unnecessary detaches

Use QHash/QMap's constFind() instead of non-const find() where
applicable to avoid unnecessary detaches.

Change-Id: I3906c7ff0601b2b4c336c6e4312319b76fa0c183
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>

1 files changed, 4 insertions, 4 deletions

diff --git a/src/qml/debugger/qqmldebugservice.cpp b/src/qml/debugger/qqmldebugservice.cpp
index 034aa55ce7..56cffc169c 100644
--- a/src/qml/debugger/qqmldebugservice.cpp
+++ b/src/qml/debugger/qqmldebugservice.cpp
@@ -103,8 +103,8 @@ Q_GLOBAL_STATIC(ObjectReferenceHash, objectReferenceHash)
 
 void ObjectReferenceHash::remove(QObject *obj)
 {
-    QHash<QObject *, int>::Iterator iter = objects.find(obj);
-    if (iter != objects.end()) {
+    const auto iter = objects.constFind(obj);
+    if (iter != objects.cend()) {
         ids.remove(iter.value());
         objects.erase(iter);
     }
@@ -120,9 +120,9 @@ int QQmlDebugService::idForObject(QObject *object)
         return -1;
 
     ObjectReferenceHash *hash = objectReferenceHash();
-    QHash<QObject *, int>::Iterator iter = hash->objects.find(object);
+    auto iter = hash->objects.constFind(object);
 
-    if (iter == hash->objects.end()) {
+    if (iter == hash->objects.cend()) {
         int id = hash->nextId++;
         hash->ids.insert(id, object);
         iter = hash->objects.insert(object, id);