aboutsummaryrefslogtreecommitdiffstats
path: root/src/imports/platform/qquickplatformmenuitemgroup.cpp
diff options
context:
space:
mode:
authorJ-P Nurmi <jpnurmi@qt.io>2017-10-10 16:03:24 +0200
committerJ-P Nurmi <jpnurmi@qt.io>2017-10-10 20:05:52 +0000
commitd72cafb2fcc35df22036843ac71054db8650f64d (patch)
tree209aac2b5b2ea50351c9caadf58fbaf00814f07e /src/imports/platform/qquickplatformmenuitemgroup.cpp
parent9cbc5be241bcd8abb0cafc7a7e4b04746e9fe291 (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.cpp8
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);