diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2016-02-29 01:01:50 +0100 |
---|---|---|
committer | Sean Harmer <sean.harmer@kdab.com> | 2016-05-14 16:02:26 +0000 |
commit | 8f680c83447d9986be2709cc796ade05788c2135 (patch) | |
tree | 82b732b6474d11efbcddfc54cf049b5d1adb195a | |
parent | e95d7845c84f5af5055e09f0471564ac975a9172 (diff) |
plugins: eradicate all Q_FOREACH loops
... by replacing them with C++11 range-for loops.
In assimp, there were only these low-risk changes.
Change-Id: I114d6bd62bb37b6347f5d3bda41125198a282f7e
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
-rw-r--r-- | src/plugins/sceneparsers/assimp/assimpio.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/plugins/sceneparsers/assimp/assimpio.cpp b/src/plugins/sceneparsers/assimp/assimpio.cpp index 01e247ca0..3240b8891 100644 --- a/src/plugins/sceneparsers/assimp/assimpio.cpp +++ b/src/plugins/sceneparsers/assimp/assimpio.cpp @@ -169,16 +169,20 @@ QString texturePath(const aiString &path) QParameter *findNamedParameter(const QString &name, QMaterial *material) { // Does the material contain the parameter ? - foreach (QParameter *p , material->parameters()) + const auto params = material->parameters(); + for (QParameter *p : params) { if (p->name() == name) return p; + } // Does the material's effect contain the parameter ? if (material->effect()) { const QEffect *e = material->effect(); - foreach (QParameter *p, e->parameters()) + const auto params = e->parameters(); + for (QParameter *p : params) { if (p->name() == name) return p; + } } // Create and add parameter to material |