diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2016-02-29 01:01:50 +0100 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@kdab.com> | 2016-04-28 05:07:29 +0000 |
| commit | 62dea78db3725b7d8dd32d0152216f4f3c8158c6 (patch) | |
| tree | de44bc1c9450aa8d58231382715c247ada2bd636 /src/render/backend/rendertarget.cpp | |
| parent | 77412385c687b2c000cf4afb90b709752d7d0a5d (diff) | |
render/backend: 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 8.3KiB (2.8%) in text size on optimized GCC 5.3
Linux AMD64 builds.
Change-Id: I7e4f8390b7530981185ad65d152b267e18a88c90
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'src/render/backend/rendertarget.cpp')
| -rw-r--r-- | src/render/backend/rendertarget.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/render/backend/rendertarget.cpp b/src/render/backend/rendertarget.cpp index bd411fc50..01f006c21 100644 --- a/src/render/backend/rendertarget.cpp +++ b/src/render/backend/rendertarget.cpp @@ -61,7 +61,8 @@ void RenderTarget::updateFromPeer(Qt3DCore::QNode *peer) { QRenderTarget *target = static_cast<QRenderTarget *>(peer); m_renderOutputs.clear(); - Q_FOREACH (QRenderTargetOutput *att, target->outputs()) + const auto outputs = target->outputs(); + for (QRenderTargetOutput *att : outputs) appendRenderOutput(att->id()); } |