diff options
| author | Sergio Martins <sergio.martins@kdab.com> | 2016-12-10 17:09:13 +0000 |
|---|---|---|
| committer | Sean Harmer <sean.harmer@kdab.com> | 2017-04-02 13:02:32 +0000 |
| commit | 172287f824b740a7e2ce75e811c0858cbe0c40c1 (patch) | |
| tree | e0cba921f07f8cb5de9aea7b8132a1b8596cbc32 /src/plugins/sceneparsers/gltf/gltfimporter.cpp | |
| parent | 0ba3824ac24cb435c095713e24250c87d184fd97 (diff) | |
Don't allocate temporary containers while iterating
Fixes all clazy-container-anti-pattern warnings.
Change-Id: I399e333ef9b4731667cd698cd3882e0fdd63a857
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>
Diffstat (limited to 'src/plugins/sceneparsers/gltf/gltfimporter.cpp')
| -rw-r--r-- | src/plugins/sceneparsers/gltf/gltfimporter.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/sceneparsers/gltf/gltfimporter.cpp b/src/plugins/sceneparsers/gltf/gltfimporter.cpp index c026594bf..4544bc5c3 100644 --- a/src/plugins/sceneparsers/gltf/gltfimporter.cpp +++ b/src/plugins/sceneparsers/gltf/gltfimporter.cpp @@ -1072,7 +1072,7 @@ void GLTFImporter::cleanup() m_shaderPaths.clear(); delete_if_without_parent(m_programs); m_programs.clear(); - for (auto params : m_techniqueParameters.values()) + for (auto params : qAsConst(m_techniqueParameters)) delete_if_without_parent(params); m_techniqueParameters.clear(); delete_if_without_parent(m_techniques); |