quick3d: eradicate Q_FOREACH loops [low-risk]

... by replacing them with C++11 range-for loops.

This is the batch with low-risk changes. They operate
on local containers or the loop body clearly does not
cause the container to change.

Sprinkled in a reserve() or two.

Change-Id: I5a3da485213791ae58c82f4a81829cfba558a3d6
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/quick3d/quick3drender/items/quick3dtexture.cpp b/src/quick3d/quick3drender/items/quick3dtexture.cpp
index ea0ac3f41..9c528996c 100644
--- a/src/quick3d/quick3drender/items/quick3dtexture.cpp
+++ b/src/quick3d/quick3drender/items/quick3dtexture.cpp
@@ -85,10 +85,11 @@ int Quick3DTextureExtension::textureImageCount(QQmlListProperty<QAbstractTexture
 
 void Quick3DTextureExtension::clearTextureImageList(QQmlListProperty<QAbstractTextureImage> *list)
 {
-    Quick3DTextureExtension *self = qobject_cast<Quick3DTextureExtension *>(list->object);
-    if (self)
-        Q_FOREACH (QAbstractTextureImage *img, self->parentTexture()->textureImages())
+    if (Quick3DTextureExtension *self = qobject_cast<Quick3DTextureExtension *>(list->object)) {
+        const auto images = self->parentTexture()->textureImages();
+        for (QAbstractTextureImage *img : images)
             self->parentTexture()->removeTextureImage(img);
+    }
 }
 
 } // namespace Quick