X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Frender%2Fsimplescene.cpp;h=fbdb004f9a0e8f8917b5ae42a8a2554d84b54d22;hb=f853ca0a365ae8b43ba8b2f4d6f21cd1c2bd4bd5;hp=d1fd3c1cea2a4c4ad0349fb1a30017cfa2b52920;hpb=7aaec9a70b8d7733429bec043f8e33e02956f266;p=libs%2Fgl.git

diff --git a/source/render/simplescene.cpp b/source/render/simplescene.cpp
index d1fd3c1c..fbdb004f 100644
--- a/source/render/simplescene.cpp
+++ b/source/render/simplescene.cpp
@@ -1,3 +1,4 @@
+#include <msp/core/algorithm.h>
 #include "renderer.h"
 #include "simplescene.h"
 
@@ -6,52 +7,42 @@ namespace GL {
 
 void SimpleScene::add(Renderable &r)
 {
-	// Add to cache as well if the cache is valid
-	if(renderables.insert(&r).second && !cache.empty())
-		cache.push_back(&r);
+	auto i = lower_bound(renderables, &r);
+	if(i==renderables.end() || *i!=&r)
+		renderables.insert(i, &r);
 }
 
 void SimpleScene::remove(Renderable &r)
 {
-	renderables.erase(&r);
-	cache.clear();
-}
-
-void SimpleScene::populate_cache() const
-{
-	if(cache.empty() && !renderables.empty())
-	{
-		cache.reserve(renderables.size());
-		cache.insert(cache.end(), renderables.begin(), renderables.end());
-	}
+	auto i = find(renderables, &r);
+	if(i!=renderables.end())
+		renderables.erase(i);
 }
 
 void SimpleScene::setup_frame(Renderer &renderer)
 {
-	populate_cache();
-	for(RenderableArray::const_iterator i=cache.begin(); i!=cache.end(); ++i)
-		(*i)->setup_frame(renderer);
+	for(Renderable *r: renderables)
+		r->setup_frame(renderer);
 }
 
 void SimpleScene::finish_frame()
 {
-	for(RenderableArray::const_iterator i=cache.begin(); i!=cache.end(); ++i)
-		(*i)->finish_frame();
+	for(Renderable *r: renderables)
+		r->finish_frame();
 }
 
-void SimpleScene::render(Renderer &renderer, const Tag &tag) const
+void SimpleScene::render(Renderer &renderer, Tag tag) const
 {
-	populate_cache();
 	if(setup_frustum(renderer))
 	{
-		for(RenderableArray::const_iterator i=cache.begin(); i!=cache.end(); ++i)
-			if(!frustum_cull(**i))
-				renderer.render(**i, tag);
+		for(Renderable *r: renderables)
+			if(!frustum_cull(*r))
+				r->render(renderer, tag);
 	}
 	else
 	{
-		for(RenderableArray::const_iterator i=cache.begin(); i!=cache.end(); ++i)
-			renderer.render(**i, tag);
+		for(Renderable *r: renderables)
+			r->render(renderer, tag);
 	}
 }