Prefer RG pixelformats over LUMINANCE

[libs/gl.git] / source / pipeline.cpp

diff --git a/source/pipeline.cpp b/source/pipeline.cpp
index 1833001ff1288fb00257450fe694bd9e44ae62a1..e3da053070b0cb0dcadc84e62f6b91cf4befab5a 100644 (file)
--- a/source/pipeline.cpp
+++ b/source/pipeline.cpp
@@ -78,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();
 }
 
@@ -117,6 +117,12 @@ 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);
@@ -134,6 +140,9 @@ void Pipeline::add_postprocessor(PostProcessor &pp)
 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();
 }
@@ -141,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();
 }
@@ -164,7 +176,8 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
                setup_frame();
 
        const Framebuffer *out_fbo = Framebuffer::current();
-       // XXX Make sure the correct FBO is restored if an exception is thrown
+       // This is a no-op but will ensure the FBO binding gets restored
+       BindRestore restore_fbo(out_fbo);
 
        if(target[0])
        {
@@ -180,6 +193,9 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
                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()))
                                renderer.render(*j->renderable, i->get_tag());
@@ -201,8 +217,6 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
                                out_fbo->bind();
                        postproc[i]->render(target[j]->color, target[j]->depth);
                }
-
-               out_fbo->bind();
        }
 
        if(!was_in_frame)
@@ -238,12 +252,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),
-       clipping(0)
+       clipping(0),
+       renderable(r)
 { }
 
 void Pipeline::Pass::set_lighting(const Lighting *l)