diff options
author | Ulf Hermann <ulf.hermann@qt.io> | 2016-12-07 13:56:45 +0100 |
---|---|---|
committer | Ulf Hermann <ulf.hermann@qt.io> | 2016-12-09 12:34:56 +0000 |
commit | d7ef2aa02c9ba736104fe973bfce7d87e30f81af (patch) | |
tree | b633989433e223e7c61b943b0b04072128bb2837 /src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp | |
parent | 84e86b61bfd7650993324e64d72ac73635effc77 (diff) |
Qml Inspector: Avoid taking a QPointer of an object being deleted
As we manually remove items to be deleted from the list of selected
items, we don't need to keep them as QPointer in the first place.
Change-Id: Ie416b47d4f193cb3dc7cfb07e66ab2b36ad6721c
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Diffstat (limited to 'src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp')
-rw-r--r-- | src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp b/src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp index 107d54c626..ca7c76ab50 100644 --- a/src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp +++ b/src/plugins/qmltooling/qmldbg_inspector/globalinspector.cpp @@ -234,9 +234,7 @@ bool GlobalInspector::syncSelectedItems(const QList<QQuickItem *> &items) bool selectionChanged = false; // Disconnect and remove items that are no longer selected - foreach (const QPointer<QQuickItem> &item, m_selectedItems) { - if (!item) // Don't see how this can happen due to handling of destroyed() - continue; + foreach (QQuickItem *item, m_selectedItems) { if (items.contains(item)) continue; |