render/framegraph: eradicate Q_FOREACH loops [low-risk]

... by replacing them with C++11 range-for loops. To
avoid detaches of these mutable Qt containers, wrap
the container in qAsConst(), where needed.

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.

Saves 900B in text size on optimized GCC 5.3 Linux
AMD64 builds.

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

1 files changed, 4 insertions, 2 deletions

diff --git a/src/render/framegraph/renderpassfilternode.cpp b/src/render/framegraph/renderpassfilternode.cpp
index e497ebd9c..451f824d3 100644
--- a/src/render/framegraph/renderpassfilternode.cpp
+++ b/src/render/framegraph/renderpassfilternode.cpp
@@ -61,9 +61,11 @@ void RenderPassFilter::updateFromPeer(Qt3DCore::QNode *peer)
     m_filters.clear();
     m_parameterPack.clear();
     setEnabled(filter->isEnabled());
-    Q_FOREACH (QAnnotation *criterion, filter->includes())
+    const auto criteria = filter->includes();
+    for (QAnnotation *criterion : criteria)
         appendFilter(criterion->id());
-    Q_FOREACH (QParameter *p, filter->parameters())
+    const auto parameters = filter->parameters();
+    for (QParameter *p : parameters)
         m_parameterPack.appendParameter(p->id());
 }