diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2016-01-26 14:38:54 +0100 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2016-04-25 05:11:28 +0000 |
commit | 54d26804d788d0bbe500a0d3f4049d25d5be244b (patch) | |
tree | 13537c2f71ad02ffa7650159c4216b24227f19d9 /src/plugins | |
parent | 0909a94af2d274840697e4ae25cc14b3f5eb6197 (diff) |
Android: eradicate Q_FOREACH loops [already const]
... (or trivially marked const) local variables,
parameters, or data members with trivial loop bodies,
by replacing them with C++11 range-for loops.
Change-Id: I25e9e2148abd7676305ebd18754dc647f217e48b
Reviewed-by: Olivier Goffart (Woboq GmbH) <ogoffart@woboq.com>
Diffstat (limited to 'src/plugins')
5 files changed, 8 insertions, 8 deletions
diff --git a/src/plugins/platforms/android/androidjnimain.cpp b/src/plugins/platforms/android/androidjnimain.cpp index 51849aa688..3847f55d3a 100644 --- a/src/plugins/platforms/android/androidjnimain.cpp +++ b/src/plugins/platforms/android/androidjnimain.cpp @@ -494,7 +494,7 @@ static jboolean startQtApplication(JNIEnv *env, jobject /*object*/, jstring para const char *nativeString = env->GetStringUTFChars(environmentString, 0); const QList<QByteArray> envVars = QByteArray(nativeString).split('\t'); env->ReleaseStringUTFChars(environmentString, nativeString); - foreach (const QByteArray &envVar, envVars) { + for (const QByteArray &envVar : envVars) { const QList<QByteArray> envVarPair = envVar.split('='); if (envVarPair.size() == 2 && ::setenv(envVarPair[0], envVarPair[1], 1) != 0) qWarning() << "Can't set environment" << envVarPair; diff --git a/src/plugins/platforms/android/androidjnimenu.cpp b/src/plugins/platforms/android/androidjnimenu.cpp index 551931b688..778a444837 100644 --- a/src/plugins/platforms/android/androidjnimenu.cpp +++ b/src/plugins/platforms/android/androidjnimenu.cpp @@ -257,7 +257,7 @@ namespace QtAndroidMenu if (menus.size() == 1) { // Expand the menu order = addAllMenuItemsToMenu(env, menu, static_cast<QAndroidPlatformMenu *>(menus.front())); } else { - foreach (QAndroidPlatformMenu *item, menus) { + for (QAndroidPlatformMenu *item : menus) { QString itemText = removeAmpersandEscapes(item->text()); jstring jtext = env->NewString(reinterpret_cast<const jchar *>(itemText.data()), itemText.length()); diff --git a/src/plugins/platforms/android/qandroidplatformmenu.cpp b/src/plugins/platforms/android/qandroidplatformmenu.cpp index 83e79eb76e..06b297a1ad 100644 --- a/src/plugins/platforms/android/qandroidplatformmenu.cpp +++ b/src/plugins/platforms/android/qandroidplatformmenu.cpp @@ -159,7 +159,7 @@ QPlatformMenuItem *QAndroidPlatformMenu::menuItemAt(int position) const QPlatformMenuItem *QAndroidPlatformMenu::menuItemForTag(quintptr tag) const { - foreach (QPlatformMenuItem *menuItem, m_menuItems) { + for (QPlatformMenuItem *menuItem : m_menuItems) { if (menuItem->tag() == tag) return menuItem; } diff --git a/src/plugins/platforms/android/qandroidplatformmenubar.cpp b/src/plugins/platforms/android/qandroidplatformmenubar.cpp index 56ccbe1afe..35930f0628 100644 --- a/src/plugins/platforms/android/qandroidplatformmenubar.cpp +++ b/src/plugins/platforms/android/qandroidplatformmenubar.cpp @@ -86,7 +86,7 @@ void QAndroidPlatformMenuBar::handleReparent(QWindow *newParentWindow) QPlatformMenu *QAndroidPlatformMenuBar::menuForTag(quintptr tag) const { - foreach (QPlatformMenu *menu, m_menus) { + for (QPlatformMenu *menu : m_menus) { if (menu->tag() == tag) return menu; } diff --git a/src/plugins/platforms/android/qandroidplatformscreen.cpp b/src/plugins/platforms/android/qandroidplatformscreen.cpp index aa4fa94f0a..a3b918a1a4 100644 --- a/src/plugins/platforms/android/qandroidplatformscreen.cpp +++ b/src/plugins/platforms/android/qandroidplatformscreen.cpp @@ -117,7 +117,7 @@ QAndroidPlatformScreen::~QAndroidPlatformScreen() QWindow *QAndroidPlatformScreen::topWindow() const { - foreach (QAndroidPlatformWindow *w, m_windowStack) { + for (QAndroidPlatformWindow *w : m_windowStack) { if (w->window()->type() == Qt::Window || w->window()->type() == Qt::Popup || w->window()->type() == Qt::Dialog) { @@ -129,7 +129,7 @@ QWindow *QAndroidPlatformScreen::topWindow() const QWindow *QAndroidPlatformScreen::topLevelAt(const QPoint &p) const { - foreach (QAndroidPlatformWindow *w, m_windowStack) { + for (QAndroidPlatformWindow *w : m_windowStack) { if (w->geometry().contains(p, false) && w->window()->isVisible()) return w->window(); } @@ -363,8 +363,8 @@ void QAndroidPlatformScreen::doRedraw() || !window->isRaster()) continue; - QVector<QRect> visibleRects = visibleRegion.rects(); - foreach (const QRect &rect, visibleRects) { + const QVector<QRect> visibleRects = visibleRegion.rects(); + for (const QRect &rect : visibleRects) { QRect targetRect = window->geometry(); targetRect &= rect; |