changed format to screen fbo for a more explicit and safe one

forwardmodule.h

@@ -18,7 +18,7 @@ class ForwardModule : public Module
 public:
     ForwardModule() :
         shaderSources(NULL),
-        renderTarget(FrameBuffer::getScreen())
+        renderTarget(FrameBuffer::screen)
     {}
 
     virtual void renderGL(Camera* myCamera, Scene* scene);
@@ -47,7 +47,7 @@ private:
     ShaderSource* shaderSources;
     std::vector<Shader*> shaders;
     std::vector<unsigned int> flags;
-    FrameBuffer* renderTarget;
+    const FrameBuffer* renderTarget;
 
     void lightPass(Camera* myCamera, Scene* scene, Light* light, unsigned int type);
 };

framebuffer.cpp

@@ -2,12 +2,11 @@
 #include "texture.h"
 #include "glassert.h"
 
-FrameBuffer::FrameBuffer(bool isNULL)
+const FrameBuffer* FrameBuffer::screen = new FrameBuffer(0);
+
+FrameBuffer::FrameBuffer()
 {
-    if(isNULL)
+    glAssert(glGenFramebuffers(1, &fbo));
-        fbo = 0;
-    else
-        glAssert(glGenFramebuffers(1, &fbo));
 }
 
 FrameBuffer::~FrameBuffer()
@@ -18,32 +17,34 @@ FrameBuffer::~FrameBuffer()
 
 void FrameBuffer::addTexture(Texture* tex, GLenum attachment)
 {
-    textures.push_back(tex);
+    if(fbo != 0)
-    bindFBO();
+    {
-    glAssert(glFramebufferTexture2D(GL_FRAMEBUFFER, attachment, tex->getTarget(), tex->getId(), 0));
+        textures.push_back(tex);
-    attachments.push_back(attachment);
+        bindFBO();
+        glAssert(glFramebufferTexture2D(GL_FRAMEBUFFER, attachment, tex->getTarget(), tex->getId(), 0));
+        attachments.push_back(attachment);
+    }
 }
 
 void FrameBuffer::initGL()
 {
-    bindFBO();
+    if(fbo != 0)
-    glAssert(glDrawBuffers(attachments.size(), attachments.data()));
+    {
+        bindFBO();
+        glAssert(glDrawBuffers(attachments.size(), attachments.data()));
+    }
 }
 
-void FrameBuffer::bindFBO()
+void FrameBuffer::bindFBO() const
 {
     glAssert(glBindFramebuffer(GL_FRAMEBUFFER, fbo));
 }
 
 Texture* FrameBuffer::getTexture(int texId)
 {
-    return textures[texId];
+    if(fbo != 0)
+        return textures[texId];
+    else
+        return NULL;
 }
 
-FrameBuffer* FrameBuffer::getScreen()
-{
-    static FrameBuffer* screen = NULL;
-    if(screen == NULL)
-        screen = new FrameBuffer(true);
-    return screen;
-}

framebuffer.h

@@ -8,19 +8,23 @@ class Texture;
 
 class FrameBuffer
 {
+private:
+    FrameBuffer(int id) : fbo(id) {}
+
 protected:
     GLuint fbo;
     std::vector<Texture*> textures;
     std::vector<GLuint> attachments;
+
 public:
-    FrameBuffer(bool isNULL = false);
+    FrameBuffer();
     ~FrameBuffer();
     void addTexture(Texture* tex, GLenum attachment);
     void initGL();
 
-    void bindFBO();
+    void bindFBO() const;
     Texture* getTexture(int texId);
-    static FrameBuffer* getScreen();
+    static const FrameBuffer* screen;
 };
 
 #endif // FRAMEBUFFER_H