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/qquickplatformmenubar.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/qquickplatformmenubar.cpp')
| -rw-r--r-- | src/imports/platform/qquickplatformmenubar.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/imports/platform/qquickplatformmenubar.cpp b/src/imports/platform/qquickplatformmenubar.cpp index 516bcc3e..4487b9a7 100644 --- a/src/imports/platform/qquickplatformmenubar.cpp +++ b/src/imports/platform/qquickplatformmenubar.cpp @@ -117,7 +117,7 @@ QQuickPlatformMenuBar::QQuickPlatformMenuBar(QObject *parent) QQuickPlatformMenuBar::~QQuickPlatformMenuBar() { - for (QQuickPlatformMenu *menu : m_menus) + for (QQuickPlatformMenu *menu : qAsConst(m_menus)) menu->setMenuBar(nullptr); delete m_handle; m_handle = nullptr; @@ -235,7 +235,7 @@ void QQuickPlatformMenuBar::clear() if (m_menus.isEmpty()) return; - for (QQuickPlatformMenu *menu : m_menus) { + for (QQuickPlatformMenu *menu : qAsConst(m_menus)) { m_data.removeOne(menu); if (m_handle) m_handle->removeMenu(menu->handle()); @@ -254,7 +254,7 @@ void QQuickPlatformMenuBar::classBegin() void QQuickPlatformMenuBar::componentComplete() { m_complete = true; - for (QQuickPlatformMenu *menu : m_menus) + for (QQuickPlatformMenu *menu : qAsConst(m_menus)) menu->sync(); if (!m_window) setWindow(findWindow()); |