diff options
author | J-P Nurmi <jpnurmi@qt.io> | 2017-10-10 16:03:24 +0200 |
---|---|---|
committer | J-P Nurmi <jpnurmi@qt.io> | 2017-10-10 20:05:52 +0000 |
commit | d72cafb2fcc35df22036843ac71054db8650f64d (patch) | |
tree | 209aac2b5b2ea50351c9caadf58fbaf00814f07e /src/imports/platform/qquickplatformmenuitemgroup.cpp | |
parent | 9cbc5be241bcd8abb0cafc7a7e4b04746e9fe291 (diff) |
Platform: fix clazy-range-loop findings
Avoid accidental detach with C++11 range for loops.
Change-Id: I2a105ef5a2505d26ee086974177f8f1e4040b522
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
Diffstat (limited to 'src/imports/platform/qquickplatformmenuitemgroup.cpp')
-rw-r--r-- | src/imports/platform/qquickplatformmenuitemgroup.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/imports/platform/qquickplatformmenuitemgroup.cpp b/src/imports/platform/qquickplatformmenuitemgroup.cpp index af901bb8..f07d0a1b 100644 --- a/src/imports/platform/qquickplatformmenuitemgroup.cpp +++ b/src/imports/platform/qquickplatformmenuitemgroup.cpp @@ -164,7 +164,7 @@ void QQuickPlatformMenuItemGroup::setEnabled(bool enabled) m_enabled = enabled; emit enabledChanged(); - for (QQuickPlatformMenuItem *item : m_items) { + for (QQuickPlatformMenuItem *item : qAsConst(m_items)) { if (item->m_enabled) { item->sync(); emit item->enabledChanged(); @@ -193,7 +193,7 @@ void QQuickPlatformMenuItemGroup::setVisible(bool visible) m_visible = visible; emit visibleChanged(); - for (QQuickPlatformMenuItem *item : m_items) { + for (QQuickPlatformMenuItem *item : qAsConst(m_items)) { if (item->m_visible) { item->sync(); emit item->visibleChanged(); @@ -222,7 +222,7 @@ void QQuickPlatformMenuItemGroup::setExclusive(bool exclusive) m_exclusive = exclusive; emit exclusiveChanged(); - for (QQuickPlatformMenuItem *item : m_items) + for (QQuickPlatformMenuItem *item : qAsConst(m_items)) item->sync(); } @@ -317,7 +317,7 @@ void QQuickPlatformMenuItemGroup::clear() if (m_items.isEmpty()) return; - for (QQuickPlatformMenuItem *item : m_items) { + for (QQuickPlatformMenuItem *item : qAsConst(m_items)) { item->setGroup(nullptr); disconnect(item, &QQuickPlatformMenuItem::checkedChanged, this, &QQuickPlatformMenuItemGroup::updateCurrent); disconnect(item, &QQuickPlatformMenuItem::triggered, this, &QQuickPlatformMenuItemGroup::activateItem); |