Improved Binding of textures

examples/debug/main.cpp

@@ -86,10 +86,8 @@ int main(int argc, char** argv)
 
 		oglu::ClearScreen(GL_COLOR_BUFFER_BIT, oglu::Color(0.29f, 0.13f, 0.23f));
 
-		oglu::ActiveTexture(0);
+		crate->BindAs(0);
-		crate->Bind();
+		opengl->BindAs(1);
-		oglu::ActiveTexture(1);
-		opengl->Bind();
 
 		shader->Use();
 		shader->SetUniform("texture1", 0);

include/shader.hpp

@@ -14,6 +14,9 @@
 namespace oglu
 {
 	class Color;
+	class AbstractTexture;
+
+	typedef std::shared_ptr<AbstractTexture> Texture;
 
 	/**
 	 * @brief An object representing an OpenGL Shader Program.

include/texture.hpp

@@ -59,6 +59,16 @@ namespace oglu
 		 */
 		void Bind();
 
+		/**
+		 * @brief Sets active texture and binds this texture.
+		 * 
+		 * In order to use multiple textures in the same shader the active texture unit needs to be specified.
+		 * This function first sets the active texture unit and then binds the texture.
+		 * 
+		 * @param[in] index Index of the texture unit (Note: This index is actually an offset to @p GL_TEXTURE0)
+		 */
+		void BindAs(GLbyte index);
+
 		/**
 		 * @brief Unbind this texture.
 		 */

src/shader.cpp

@@ -40,6 +40,7 @@ namespace oglu
 			glGetShaderInfoLog(vertexShader, 512, NULL, infoLog);
 			std::string err = ("Failed to compile shader " + std::string(vertexShaderFile) + "\n" + infoLog);
 			delete source;
+			glDeleteShader(vertexShader);
 			throw std::runtime_error(err);
 		}
 		delete source;
@@ -57,6 +58,8 @@ namespace oglu
 			glGetShaderInfoLog(fragmentShader, 512, NULL, infoLog);
 			std::string err = ("Failed to compile shader " + std::string(fragmentShaderFile) + "\n" + infoLog);
 			delete source;
+			glDeleteShader(vertexShader);
+			glDeleteShader(fragmentShader);
 			throw std::runtime_error(err);
 		}
 		delete source;
@@ -72,6 +75,9 @@ namespace oglu
 		{
 			glGetProgramInfoLog(program, 512, NULL, infoLog);
 			std::string err = ("Failed to link program.\n" + std::string(infoLog));
+			glDeleteShader(vertexShader);
+			glDeleteShader(fragmentShader);
+			glDeleteProgram(program);
 			throw std::runtime_error(err);
 		}
 

src/texture.cpp

@@ -68,6 +68,12 @@ namespace oglu
 		glBindTexture(GL_TEXTURE_2D, texture);
 	}
 
+	void AbstractTexture::BindAs(GLbyte index)
+	{
+		glActiveTexture(GL_TEXTURE0 + index);
+		glBindTexture(GL_TEXTURE_2D, texture);
+	}
+
 	void AbstractTexture::Unbind()
 	{
 		glBindTexture(GL_TEXTURE_2D, 0);