From 28be432041a8735d085b9740d32a422e51a01b74 Mon Sep 17 00:00:00 2001 From: John Firebaugh Date: Sat, 12 Aug 2017 16:12:58 -0700 Subject: [core] Eliminate temporary sources set --- src/mbgl/renderer/renderer_impl.cpp | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) (limited to 'src/mbgl') diff --git a/src/mbgl/renderer/renderer_impl.cpp b/src/mbgl/renderer/renderer_impl.cpp index 1a8af29485..62fbba039f 100644 --- a/src/mbgl/renderer/renderer_impl.cpp +++ b/src/mbgl/renderer/renderer_impl.cpp @@ -254,13 +254,6 @@ void Renderer::Impl::render(const UpdateParameters& updateParameters) { parameters.context.setDirtyState(); } - std::unordered_set sources; - for (const auto& entry : renderSources) { - if (entry.second->isEnabled()) { - sources.insert(entry.second.get()); - } - } - Color backgroundColor; class RenderItem { @@ -397,8 +390,10 @@ void Renderer::Impl::render(const UpdateParameters& updateParameters) { MBGL_DEBUG_GROUP(parameters.context, "clip"); // Update all clipping IDs. - for (const auto& source : sources) { - source->startRender(parameters); + for (const auto& entry : renderSources) { + if (entry.second->isEnabled()) { + entry.second->startRender(parameters); + } } MBGL_DEBUG_GROUP(parameters.context, "clipping masks"); @@ -532,8 +527,10 @@ void Renderer::Impl::render(const UpdateParameters& updateParameters) { // This guarantees that we have at least one function per tile called. // When only rendering layers via the stylesheet, it's possible that we don't // ever visit a tile during rendering. - for (const auto& source : sources) { - source->finishRender(parameters); + for (const auto& entry : renderSources) { + if (entry.second->isEnabled()) { + entry.second->finishRender(parameters); + } } } -- cgit v1.2.1