diff options
author | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2017-01-02 17:09:54 +0100 |
---|---|---|
committer | Friedemann Kleint <Friedemann.Kleint@qt.io> | 2017-01-10 12:30:19 +0000 |
commit | 5a1ce04cb71263ab078921f5efc8cf30f95a2eb5 (patch) | |
tree | c0c4119ab225a0031f4c7a4ac8af8f625c44a0bd /src | |
parent | 86515f2a650c8e6acc6906d5308061c8cd064408 (diff) |
Remove foreach
Use range based for.
Change-Id: If91077be5bb13aa1447866c4243511db72d4f46c
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Diffstat (limited to 'src')
-rw-r--r-- | src/activeqt/container/qaxbase.cpp | 16 | ||||
-rw-r--r-- | src/activeqt/container/qaxscript.cpp | 22 | ||||
-rw-r--r-- | src/activeqt/container/qaxselect.cpp | 3 | ||||
-rw-r--r-- | src/activeqt/control/qaxfactory.h | 4 | ||||
-rw-r--r-- | src/activeqt/control/qaxserver.cpp | 16 | ||||
-rw-r--r-- | src/activeqt/control/qaxserverbase.cpp | 13 | ||||
-rw-r--r-- | src/activeqt/shared/qaxtypes.cpp | 2 | ||||
-rw-r--r-- | src/activeqt/shared/qaxutils.cpp | 3 |
8 files changed, 40 insertions, 39 deletions
diff --git a/src/activeqt/container/qaxbase.cpp b/src/activeqt/container/qaxbase.cpp index f2deedb..36bf4e6 100644 --- a/src/activeqt/container/qaxbase.cpp +++ b/src/activeqt/container/qaxbase.cpp @@ -1000,7 +1000,7 @@ bool QAxBase::setControl(const QString &c) if (search.isEmpty()) { controls.beginGroup(QLatin1String("/CLSID")); const QStringList clsids = controls.childGroups(); - foreach (const QString &clsid, clsids) { + for (const QString &clsid : clsids) { const QString name = controls.value(clsid + QLatin1String("/Default")).toString(); if (name == c) { search = clsid; @@ -1100,8 +1100,8 @@ void QAxBase::disableClassInfo() */ void QAxBase::clear() { - foreach (QAxEventSink *eventSink, d->eventSink) { - if (eventSink) { + for (auto it = d->eventSink.cbegin(), end = d->eventSink.cend(); it != end; ++it) { + if (QAxEventSink *eventSink = it.value()) { eventSink->unadvise(); eventSink->Release(); } @@ -2167,8 +2167,8 @@ void MetaObjectGenerator::readClassInfo() QString tlfile; if (!tlid.isEmpty()) { controls.beginGroup(QLatin1String("/Classes/TypeLib/") + tlid); - QStringList versions = controls.childGroups(); - foreach (const QString &version, versions) { + const QStringList versions = controls.childGroups(); + for (const QString &version : versions) { tlfile = controls.value(QLatin1Char('/') + version + QLatin1String("/0/win32/.")).toString(); if (!tlfile.isEmpty()) break; @@ -2928,7 +2928,7 @@ QMetaObject *MetaObjectGenerator::tryCache() IConnectionPointContainer *cpoints = 0; d->ptr->QueryInterface(IID_IConnectionPointContainer, reinterpret_cast<void **>(&cpoints)); if (cpoints) { - foreach (const QUuid &iid, d->metaobj->connectionInterfaces) { + for (const QUuid &iid : qAsConst(d->metaobj->connectionInterfaces)) { IConnectionPoint *cpoint = 0; cpoints->FindConnectionPoint(iid, &cpoint); if (cpoint) { @@ -3151,8 +3151,8 @@ QMetaObject *MetaObjectGenerator::metaObject(const QMetaObject *parentObject, co if (!cacheKey.isEmpty()) { mo_cache.insert(cacheKey, d->metaobj); d->cachedMetaObject = true; - foreach (QAxEventSink *sink, d->eventSink) { - if (sink) { + for (auto it = d->eventSink.cbegin(), end = d->eventSink.cend(); it != end; ++it) { + if (QAxEventSink *sink = it.value()) { QUuid ciid = sink->connectionInterface(); d->metaobj->connectionInterfaces.append(ciid); diff --git a/src/activeqt/container/qaxscript.cpp b/src/activeqt/container/qaxscript.cpp index 7c84e4b..fef39fa 100644 --- a/src/activeqt/container/qaxscript.cpp +++ b/src/activeqt/container/qaxscript.cpp @@ -713,7 +713,7 @@ bool QAxScript::load(const QString &code, const QString &language) if (code.contains(QLatin1String("End Sub"), Qt::CaseInsensitive)) lang = QLatin1String("VBScript"); - foreach (const QAxEngineDescriptor &engine, engines) { + for (const QAxEngineDescriptor &engine : qAsConst(engines)) { if (!engine.code.isEmpty() && code.contains(engine.code)) { lang = engine.name; break; @@ -939,8 +939,8 @@ QStringList QAxScriptManager::functions(QAxScript::FunctionFlags flags) const { QStringList functions; functions.reserve(d->scriptDict.size()); - foreach (const QAxScript *script, d->scriptDict) - functions.append(script->functions(flags)); + for (auto it = d->scriptDict.cbegin(), end = d->scriptDict.cend(); it != end; ++it) + functions.append(it.value()->functions(flags)); return functions; } @@ -1055,7 +1055,7 @@ QAxScript *QAxScriptManager::load(const QString &file, const QString &name) if (file.endsWith(QLatin1String(".js"))) { language = QLatin1String("JScript"); } else { - foreach (const QAxEngineDescriptor &engine, engines) { + for (const QAxEngineDescriptor &engine : qAsConst(engines)) { if (!engine.extension.isEmpty() && file.endsWith(engine.extension)) { language = engine.name; break; @@ -1178,7 +1178,7 @@ QString QAxScriptManager::scriptFileFilter() QString specialFiles = QLatin1String(";;VBScript Files (*.vbs *.dsm)" ";;JavaScript Files (*.js)"); - foreach (const QAxEngineDescriptor &engine, engines) { + for (const QAxEngineDescriptor &engine : qAsConst(engines)) { if (!engine.extension.isEmpty()) { allFiles += QLatin1String(" *") + engine.extension; specialFiles += QLatin1String(";;") + engine.name @@ -1214,18 +1214,18 @@ QAxScript *QAxScriptManager::scriptForFunction(const QString &function) const { // check full prototypes if included if (function.contains(QLatin1Char('('))) { - foreach (QAxScript *script, d->scriptDict) { - if (script->functions(QAxScript::FunctionSignatures).contains(function)) - return script; + for (auto it = d->scriptDict.cbegin(), end = d->scriptDict.cend(); it != end; ++it) { + if (it.value()->functions(QAxScript::FunctionSignatures).contains(function)) + return it.value(); } } QString funcName = function; funcName.truncate(funcName.indexOf(QLatin1Char('('))); // second try, checking only names, not prototypes - foreach (QAxScript *script, d->scriptDict) { - if (script->functions(QAxScript::FunctionNames).contains(funcName)) - return script; + for (auto it = d->scriptDict.cbegin(), end = d->scriptDict.cend(); it != end; ++it) { + if (it.value()->functions(QAxScript::FunctionNames).contains(funcName)) + return it.value(); } return 0; diff --git a/src/activeqt/container/qaxselect.cpp b/src/activeqt/container/qaxselect.cpp index 0fc7d62..073ad03 100644 --- a/src/activeqt/container/qaxselect.cpp +++ b/src/activeqt/container/qaxselect.cpp @@ -203,7 +203,8 @@ public: { m_controls = readControls(L"CLSID", unsigned(QSysInfo::WordSize)); if (QSysInfo::WordSize == 64) { // Append the 32bit controls as disabled items. - foreach (const Control &c, readControls(L"Wow6432Node\\CLSID", 32u)) { + const QList<Control> controls = readControls(L"Wow6432Node\\CLSID", 32u); + for (const Control &c : controls) { if (std::find_if(m_controls.constBegin(), m_controls.constEnd(), FindByClsidPredicate(c.clsid)) == m_controls.constEnd()) m_controls.append(c); } diff --git a/src/activeqt/control/qaxfactory.h b/src/activeqt/control/qaxfactory.h index c7444d3..2f7bed1 100644 --- a/src/activeqt/control/qaxfactory.h +++ b/src/activeqt/control/qaxfactory.h @@ -251,7 +251,7 @@ private: factory = new QAxClass<Class>(typeLibID().toString(), appID().toString()); \ qRegisterMetaType<Class*>(#Class"*"); \ keys = factory->featureList(); \ - Q_FOREACH (const QString &key, keys) { \ + for (const QString &key : qAsConst(keys)) { \ factoryKeys += key; \ factories.insert(key, factory); \ creatable.insert(key, true); \ @@ -261,7 +261,7 @@ private: factory = new QAxClass<Class>(typeLibID().toString(), appID().toString()); \ qRegisterMetaType<Class*>(#Class"*"); \ keys = factory->featureList(); \ - Q_FOREACH (const QString &key, keys) { \ + for (const QString &key : qAsConst(keys)) { \ factoryKeys += key; \ factories.insert(key, factory); \ creatable.insert(key, false); \ diff --git a/src/activeqt/control/qaxserver.cpp b/src/activeqt/control/qaxserver.cpp index b02c38b..97756c3 100644 --- a/src/activeqt/control/qaxserver.cpp +++ b/src/activeqt/control/qaxserver.cpp @@ -262,8 +262,8 @@ HRESULT UpdateRegistry(BOOL bRegister) settings->setValue(QLatin1String("/AppID/") + module + QLatin1String(".EXE/AppID"), appId); } - QStringList keys = qAxFactory()->featureList(); - foreach (const QString &classNameIn, keys) { + const QStringList keys = qAxFactory()->featureList(); + for (const QString &classNameIn : keys) { QObject *object = qAxFactory()->createObject(classNameIn); const QMetaObject *mo = qAxFactory()->metaObject(classNameIn); const QString classId = qAxFactory()->classID(classNameIn).toString().toUpper(); @@ -368,8 +368,8 @@ HRESULT UpdateRegistry(BOOL bRegister) settings->remove(QLatin1String("/AppID/") + appId + QLatin1String("/.")); settings->remove(QLatin1String("/AppID/") + module + QLatin1String(".EXE")); } - QStringList keys = qAxFactory()->featureList(); - foreach (const QString &classNameIn, keys) { + const QStringList keys = qAxFactory()->featureList(); + for (const QString &classNameIn : keys) { const QMetaObject *mo = qAxFactory()->metaObject(classNameIn); const QString classId = qAxFactory()->classID(classNameIn).toString().toUpper(); const QString className = qax_clean_type(classNameIn, mo); @@ -1160,7 +1160,7 @@ extern "C" HRESULT __stdcall DumpIDL(const QString &outfile, const QString &ver) out << "\timportlib(\"stdole32.tlb\");" << endl; out << "\timportlib(\"stdole2.tlb\");" << endl << endl; - QStringList keys = qAxFactory()->featureList(); + const QStringList keys = qAxFactory()->featureList(); out << "\t/************************************************************************" << endl; out << "\t** If this causes a compile error in MIDL you need to upgrade the" << endl; @@ -1205,7 +1205,7 @@ extern "C" HRESULT __stdcall DumpIDL(const QString &outfile, const QString &ver) out << "\t/* Forward declaration of classes that might be used as parameters */" << endl << endl; int res = S_OK; - foreach (const QString &className, keys) { + for (const QString &className : keys) { const QMetaObject *mo = qAxFactory()->metaObject(className); // We have meta object information for this type. Forward declare it. if (mo) { @@ -1222,7 +1222,7 @@ extern "C" HRESULT __stdcall DumpIDL(const QString &outfile, const QString &ver) } out << endl; - foreach (const QString &className, keys) { + for (const QString &className : keys) { const QMetaObject *mo = qAxFactory()->metaObject(className); // We have meta object information for this type. Define it. if (mo) { @@ -1238,7 +1238,7 @@ extern "C" HRESULT __stdcall DumpIDL(const QString &outfile, const QString &ver) if (res != S_OK) goto ErrorInClass; - foreach (const QString &className, keys) { + for (const QString &className : keys) { QObject *o = qAxFactory()->createObject(className); if (!o) continue; diff --git a/src/activeqt/control/qaxserverbase.cpp b/src/activeqt/control/qaxserverbase.cpp index 61645fd..7be78d2 100644 --- a/src/activeqt/control/qaxserverbase.cpp +++ b/src/activeqt/control/qaxserverbase.cpp @@ -637,7 +637,7 @@ public: connections = old.connections; that = old.that; iid = old.iid; - foreach (const CONNECTDATA &connection, connections) + for (const CONNECTDATA &connection : qAsConst(connections)) connection.pUnk->AddRef(); } virtual ~QAxConnection() @@ -840,8 +840,8 @@ public: InitializeCriticalSection(&refCountSection); // COM only knows the CLSID, but QAxFactory is class name based... - QStringList keys = qAxFactory()->featureList(); - foreach (const QString &key, keys) { + const QStringList keys = qAxFactory()->featureList(); + for (const QString &key : keys) { if (qAxFactory()->classID(key) == clsid) { className = key; break; @@ -1121,9 +1121,8 @@ QAxServerBase::~QAxServerBase() #endif revokeActiveObject(); - - foreach (IConnectionPoint *point, points) { - if (point) + for (auto it = points.cbegin(), end = points.cend(); it != end; ++it) { + if (IConnectionPoint *point = it.value()) point->Release(); } delete aggregatedObject; @@ -1339,7 +1338,7 @@ bool QAxServerBase::internalCreate() if (isWidget) { qt.object->installEventFilter(this); const QWidgetList children = qt.object->findChildren<QWidget*>(); - foreach (QWidget *child, children) + for (QWidget *child : children) child->installEventFilter(this); } return true; diff --git a/src/activeqt/shared/qaxtypes.cpp b/src/activeqt/shared/qaxtypes.cpp index df3b655..99ab4c8 100644 --- a/src/activeqt/shared/qaxtypes.cpp +++ b/src/activeqt/shared/qaxtypes.cpp @@ -1222,7 +1222,7 @@ QVariant VARIANTToQVariant(const VARIANT &arg, const QByteArray &typeName, uint bool allStrings = true; QStringList strings; const QVariantList list(var.toList()); - foreach (const QVariant &variant, list) { + for (const QVariant &variant : list) { if (variant.canConvert(QVariant::String)) strings << variant.toString(); else diff --git a/src/activeqt/shared/qaxutils.cpp b/src/activeqt/shared/qaxutils.cpp index 51c0a1b..fde7dc6 100644 --- a/src/activeqt/shared/qaxutils.cpp +++ b/src/activeqt/shared/qaxutils.cpp @@ -95,7 +95,8 @@ HRGN qaxHrgnFromQRegion(QRegion region, const QWindow *window) addRectToHrgn(hRegion, region.boundingRect()); return hRegion; } - foreach (const QRect &rect, region.rects()) + const QVector<QRect> &rects = region.rects(); + for (const QRect &rect : rects) addRectToHrgn(hRegion, rect); return hRegion; } |