From 432df9b38e733d5f8146696936cfbbda2e0a1112 Mon Sep 17 00:00:00 2001 From: leper Date: Sat, 28 Nov 2015 08:27:42 +0000 Subject: [PATCH] Use two range based for loops to fix two TODOs. This was SVN commit r17303. --- source/renderer/WaterManager.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/source/renderer/WaterManager.cpp b/source/renderer/WaterManager.cpp index 6080e8e933..75e2583755 100644 --- a/source/renderer/WaterManager.cpp +++ b/source/renderer/WaterManager.cpp @@ -144,10 +144,8 @@ WaterManager::~WaterManager() // Cleanup if the caller messed up UnloadWaterTextures(); - // TODO: when c++11 is around, use lambdas or something because short Korea is best Korea. - for (size_t i = 0; i < m_ShoreWaves.size(); ++i) + for (WaveObject* const& obj : m_ShoreWaves) { - WaveObject* obj = m_ShoreWaves[i]; if (obj->m_VBvertices) g_VBMan.Release(obj->m_VBvertices); delete obj; @@ -495,10 +493,8 @@ void WaterManager::CreateWaveMeshes() size_t SideSize = m_MapSize*2; CTerrain* terrain = g_Game->GetWorld()->GetTerrain(); - // TODO: when c++11 is around, use lambdas or something because short Korea is best Korea. - for (size_t i = 0; i < m_ShoreWaves.size(); ++i) + for (WaveObject* const& obj : m_ShoreWaves) { - WaveObject* obj = m_ShoreWaves[i]; if (obj->m_VBvertices) g_VBMan.Release(obj->m_VBvertices); delete obj;