Build sequence without target format if there's no postprocessors

[libs/gl.git] / source / builders / sequencebuilder.cpp

diff --git a/source/builders/sequencebuilder.cpp b/source/builders/sequencebuilder.cpp
index c80e2f09a07ad4c69164fdf63686017f692f2211..01208cf4cc32a6c5a4c5245a57d74b32c0740cda 100644 (file)
--- a/source/builders/sequencebuilder.cpp
+++ b/source/builders/sequencebuilder.cpp
@@ -146,27 +146,30 @@ Sequence *SequenceBuilder::build() const
 
 Sequence *SequenceBuilder::build(unsigned w, unsigned h) const
 {
 
 Sequence *SequenceBuilder::build(unsigned w, unsigned h) const
 {

-       RefPtr<Sequence> sequence = new Sequence(w, h, create_frame_format());
+       RefPtr<Sequence> sequence = create_sequence(w, h);

        build(*sequence);
        return sequence.release();
 }
 
 Sequence *SequenceBuilder::build(const View &view) const
 {
        build(*sequence);
        return sequence.release();
 }
 
 Sequence *SequenceBuilder::build(const View &view) const
 {

-       RefPtr<Sequence> sequence = new Sequence(view.get_width(), view.get_height(), create_frame_format());
+       RefPtr<Sequence> sequence = create_sequence(view.get_width(), view.get_height());

        build(*sequence);
        return sequence.release();
 }
 
 Sequence *SequenceBuilder::build(const Framebuffer &fbo) const
 {
        build(*sequence);
        return sequence.release();
 }
 
 Sequence *SequenceBuilder::build(const Framebuffer &fbo) const
 {

-       RefPtr<Sequence> sequence = new Sequence(fbo.get_width(), fbo.get_height(), create_frame_format());
+       RefPtr<Sequence> sequence = create_sequence(fbo.get_width(), fbo.get_height());

        build(*sequence);
        return sequence.release();
 }
 
        build(*sequence);
        return sequence.release();
 }
 

-FrameFormat SequenceBuilder::create_frame_format() const
+Sequence * SequenceBuilder::create_sequence(unsigned w, unsigned h) const

 {
 {

+       if(!w || !h || tmpl.get_postprocessors().empty())
+               return new Sequence;
+

        unsigned samples = min(tmpl.get_maximum_multisample(), Device::get_current().get_info().limits.max_samples);
        if(samples<tmpl.get_required_multisample())
                throw invalid_operation("SequenceBuilder::create_frame_format");
        unsigned samples = min(tmpl.get_maximum_multisample(), Device::get_current().get_info().limits.max_samples);
        if(samples<tmpl.get_required_multisample())
                throw invalid_operation("SequenceBuilder::create_frame_format");


@@ -174,7 +177,8 @@ FrameFormat SequenceBuilder::create_frame_format() const
        DataType color_type = (tmpl.get_hdr() ? HALF_FLOAT : UNSIGNED_BYTE);
        PixelFormat color_pf = make_pixelformat(RGBA, color_type);
 
        DataType color_type = (tmpl.get_hdr() ? HALF_FLOAT : UNSIGNED_BYTE);
        PixelFormat color_pf = make_pixelformat(RGBA, color_type);
 

-       return (COLOR_ATTACHMENT,color_pf, DEPTH_ATTACHMENT).set_samples(samples);
+       FrameFormat format = (COLOR_ATTACHMENT,color_pf, DEPTH_ATTACHMENT).set_samples(samples);
+       return new Sequence(w, h, format);

 }
 
 } // namespace GL
 }
 
 } // namespace GL