From: Mikko Rasa <tdb@tdb.fi>
Date: Sun, 10 Oct 2010 09:49:41 +0000 (+0000)
Subject: Don't bind RenderBuffer in constructor
X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=commitdiff_plain;h=c4cacaf8f9225e006e6bb35f81eb6c2f78602bdc

Don't bind RenderBuffer in constructor
Use correct parameter for glDrawBuffer
---

diff --git a/source/framebuffer.cpp b/source/framebuffer.cpp
index 8dbd2d0e..77ac21be 100644
--- a/source/framebuffer.cpp
+++ b/source/framebuffer.cpp
@@ -52,7 +52,7 @@ void Framebuffer::update_attachment(unsigned mask) const
 {
 	if(current()==this)
 	{
-		bool has_color = false;
+		GLenum color_buf = GL_NONE;
 		bool has_depth = false;
 		for(unsigned i=0; i<attachments.size(); ++i)
 		{
@@ -68,12 +68,12 @@ void Framebuffer::update_attachment(unsigned mask) const
 			}
 
 			if(attch.attachment>=COLOR_ATTACHMENT0 && attch.attachment<=COLOR_ATTACHMENT3)
-				has_color = true;
+				color_buf = attch.attachment;
 			if(attch.attachment==DEPTH_ATTACHMENT)
 				has_depth = true;
 		}
 
-		glDrawBuffer(has_color ? GL_FRONT : GL_NONE);
+		glDrawBuffer(color_buf);
 		glDepthMask(has_depth);
 	}
 	else
diff --git a/source/renderbuffer.cpp b/source/renderbuffer.cpp
index 95cf56b8..d28237be 100644
--- a/source/renderbuffer.cpp
+++ b/source/renderbuffer.cpp
@@ -17,7 +17,6 @@ Renderbuffer::Renderbuffer()
 	static RequireExtension _ext("GL_EXT_framebuffer_object");
 
 	glGenRenderbuffersEXT(1, &id);
-	bind();
 }
 
 Renderbuffer::~Renderbuffer()