X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fpipeline.cpp;h=e468825748a640ed53cab215c6fc0ebee8c14fb3;hb=14e5bc2c02ef8cc468d5727065ad732bb422a89c;hp=59e6bbb843c40bbf470087eb63b7bf9a8fa9f987;hpb=812c37a8aef84f8720a3e1efedebecd993ec75b2;p=libs%2Fgl.git

diff --git a/source/pipeline.cpp b/source/pipeline.cpp
index 59e6bbb8..e4688257 100644
--- a/source/pipeline.cpp
+++ b/source/pipeline.cpp
@@ -40,8 +40,17 @@ void Pipeline::set_hdr(bool h)
 	if(h==hdr)
 		return;
 
+	bool old_hdr= hdr;
 	hdr = h;
-	create_targets(2);
+	try
+	{
+		create_targets(2);
+	}
+	catch(...)
+	{
+		hdr = old_hdr;
+		throw;
+	}
 }
 
 void Pipeline::set_multisample(unsigned s)
@@ -49,8 +58,17 @@ void Pipeline::set_multisample(unsigned s)
 	if(s==samples)
 		return;
 
+	unsigned old_samples = samples;
 	samples = s;
-	create_targets(1);
+	try
+	{
+		create_targets(1);
+	}
+	catch(...)
+	{
+		samples = old_samples;
+		throw;
+	}
 }
 
 void Pipeline::set_camera(const Camera *c)
@@ -60,7 +78,7 @@ void Pipeline::set_camera(const Camera *c)
 
 Pipeline::Pass &Pipeline::add_pass(const Tag &tag)
 {
-	passes.push_back(Pass(tag));
+	passes.push_back(Pass(tag, 0));
 	return passes.back();
 }
 
@@ -99,15 +117,32 @@ void Pipeline::remove_renderable(const Renderable &r)
 		}
 }
 
+Pipeline::Pass &Pipeline::add_pass(const Tag &tag, const Renderable &r)
+{
+	passes.push_back(Pass(tag, &r));
+	return passes.back();
+}
+
 void Pipeline::add_postprocessor(PostProcessor &pp)
 {
 	postproc.push_back(&pp);
-	create_targets(0);
+	try
+	{
+		create_targets(0);
+	}
+	catch(...)
+	{
+		postproc.pop_back();
+		throw;
+	}
 }
 
 void Pipeline::setup_frame() const
 {
 	in_frame = true;
+	for(PassList::const_iterator i=passes.begin(); i!=passes.end(); ++i)
+		if(const Renderable *renderable = i->get_renderable())
+			renderable->setup_frame();
 	for(vector<Slot>::const_iterator i=renderables.begin(); i!=renderables.end(); ++i)
 		i->renderable->setup_frame();
 }
@@ -115,6 +150,9 @@ void Pipeline::setup_frame() const
 void Pipeline::finish_frame() const
 {
 	in_frame = false;
+	for(PassList::const_iterator i=passes.begin(); i!=passes.end(); ++i)
+		if(const Renderable *renderable = i->get_renderable())
+			renderable->finish_frame();
 	for(vector<Slot>::const_iterator i=renderables.begin(); i!=renderables.end(); ++i)
 		i->renderable->finish_frame();
 }
@@ -137,19 +175,37 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
 	if(!in_frame)
 		setup_frame();
 
+	const Framebuffer *out_fbo = Framebuffer::current();
+	// These is a no-ops but will ensure the related state gets restored
+	BindRestore restore_fbo(out_fbo);
+	BindRestore restore_depth_test(DepthTest::current());
+	BindRestore restore_blend(Blend::current());
+
 	if(target[0])
 	{
 		Framebuffer &fbo = (samples ? target_ms->fbo : target[0]->fbo);
-		// XXX exception safety
 		fbo.bind();
 		fbo.clear(COLOR_BUFFER_BIT|DEPTH_BUFFER_BIT);
 	}
 
+
 	for(PassList::const_iterator i=passes.begin(); i!=passes.end(); ++i)
 	{
-		Bind bind_depth_test(i->get_depth_test());
-		Bind bind_blend(i->get_blend());
-		Bind bind_lighting(i->get_lighting());
+		if(const DepthTest *dt = i->get_depth_test())
+			dt->bind();
+		else
+			DepthTest::unbind();
+
+		if(const Blend *b = i->get_blend())
+			b->bind();
+		else
+			Blend::unbind();
+
+		renderer.set_lighting(i->get_lighting());
+		renderer.set_clipping(i->get_clipping());
+
+		if(const Renderable *renderable = i->get_renderable())
+			renderer.render(*renderable, i->get_tag());
 
 		for(vector<Slot>::const_iterator j=renderables.begin(); j!=renderables.end(); ++j)
 			if(j->passes.empty() || j->passes.count(i->get_tag()))
@@ -158,6 +214,9 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
 
 	if(target[0])
 	{
+		DepthTest::unbind();
+		Blend::unbind();
+
 		if(samples)
 			target[0]->fbo.blit_from(target_ms->fbo, COLOR_BUFFER_BIT|DEPTH_BUFFER_BIT, false);
 
@@ -167,11 +226,9 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
 			if(i+1<postproc.size())
 				target[1-j]->fbo.bind();
 			else
-				Framebuffer::unbind();
-			postproc[i]->render(target[j]->color, target[j]->depth);
+				out_fbo->bind();
+			postproc[i]->render(renderer, target[j]->color, target[j]->depth);
 		}
-
-		Framebuffer::unbind();
 	}
 
 	if(!was_in_frame)
@@ -207,11 +264,13 @@ void Pipeline::create_targets(unsigned recreate)
 }
 
 
-Pipeline::Pass::Pass(const Tag &t):
+Pipeline::Pass::Pass(const Tag &t, const Renderable *r):
 	tag(t),
 	lighting(0),
 	depth_test(0),
-	blend(0)
+	blend(0),
+	clipping(0),
+	renderable(r)
 { }
 
 void Pipeline::Pass::set_lighting(const Lighting *l)
@@ -229,6 +288,11 @@ void Pipeline::Pass::set_blend(const Blend *b)
 	blend = b;
 }
 
+void Pipeline::Pass::set_clipping(const Clipping *c)
+{
+	clipping =c;
+}
+
 
 Pipeline::Slot::Slot(const Renderable *r):
 	renderable(r)