Remove dynamic allocation from VertexFormat

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

diff --git a/source/pipeline.cpp b/source/pipeline.cpp
index 466aa23c69b6b3107457817f8efea8f33aa829ac..dc1eba1d117cdcca5d6b190c53b3545ed31fd86c 100644 (file)
--- a/source/pipeline.cpp
+++ b/source/pipeline.cpp
@@ -37,14 +37,38 @@ Pipeline::~Pipeline()
 
 void Pipeline::set_hdr(bool h)
 {
+       if(h==hdr)
+               return;
+
+       bool old_hdr= hdr;
        hdr = h;
-       create_targets(true);
+       try
+       {
+               create_targets(2);
+       }
+       catch(...)
+       {
+               hdr = old_hdr;
+               throw;
+       }
 }
 
 void Pipeline::set_multisample(unsigned s)
 {
+       if(s==samples)
+               return;
+
+       unsigned old_samples = samples;
        samples = s;
-       create_targets(false);
+       try
+       {
+               create_targets(1);
+       }
+       catch(...)
+       {
+               samples = old_samples;
+               throw;
+       }
 }
 
 void Pipeline::set_camera(const Camera *c)
@@ -54,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();
 }
 
@@ -93,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(false);
+       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();
 }
@@ -109,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();
 }
@@ -131,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()))
@@ -152,6 +214,9 @@ void Pipeline::render(Renderer &renderer, const Tag &tag) const
 
        if(target[0])
        {
+               BindRestore unbind_depth_test(static_cast<DepthTest *>(0));
+               BindRestore unbind_blend(static_cast<Blend *>(0));
+
                if(samples)
                        target[0]->fbo.blit_from(target_ms->fbo, COLOR_BUFFER_BIT|DEPTH_BUFFER_BIT, false);
 
@@ -161,45 +226,51 @@ 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)
                finish_frame();
 }
 
-void Pipeline::create_targets(bool recreate)
+void Pipeline::create_targets(unsigned recreate)
 {
-       if(recreate)
+       if(recreate>=2)
        {
                delete target[0];
                delete target[1];
-               delete target_ms;
                target[0] = 0;
                target[1] = 0;
+       }
+       if(recreate>=1)
+       {
+               delete target_ms;
                target_ms = 0;
        }
 
        PixelFormat fmt = (hdr ? RGB16F : RGB);
-       if(!target[0])
-               target[0] = new RenderTarget(width, height, fmt);
-       if(!target[1] && postproc.size()>1)
-               target[1] = new RenderTarget(width, height, fmt);
+       if(!postproc.empty() || samples)
+       {
+               if(!target[0])
+                       target[0] = new RenderTarget(width, height, fmt);
+               if(!target[1] && postproc.size()>1)
+                       target[1] = new RenderTarget(width, height, fmt);
+       }
 
        if(!target_ms && samples)
                target_ms = new MultisampleTarget(width, height, samples, fmt);
 }
 
 
-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)
@@ -217,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)
@@ -236,6 +312,8 @@ Pipeline::RenderTarget::RenderTarget(unsigned w, unsigned h, PixelFormat f)
        depth.set_wrap(CLAMP_TO_EDGE);
        depth.storage(DEPTH_COMPONENT, w, h);
        fbo.attach(DEPTH_ATTACHMENT, depth, 0);
+
+       fbo.require_complete();
 }
 
 
@@ -246,6 +324,8 @@ Pipeline::MultisampleTarget::MultisampleTarget(unsigned w, unsigned h, unsigned
 
        depth.storage_multisample(s, DEPTH_COMPONENT, w, h);
        fbo.attach(DEPTH_ATTACHMENT, depth);
+
+       fbo.require_complete();
 }
 
 } // namespace GL