X-Git-Url: http://git.tdb.fi/?a=blobdiff_plain;f=source%2Fshader.cpp;h=5ff2eb3609bd33a6fe1b30ae00eecd145f209a1f;hb=b617c5d7b5283ad260a77f01e42e6170cabbc03d;hp=45d56a16070c9fb9f50d892b3b6f17188877f8c5;hpb=5318aa4fd553be4ce0bc428e73592b787842cdea;p=libs%2Fgl.git

diff --git a/source/shader.cpp b/source/shader.cpp
index 45d56a16..5ff2eb36 100644
--- a/source/shader.cpp
+++ b/source/shader.cpp
@@ -6,6 +6,7 @@ Distributed under the LGPL
 */
 
 #include "arb_shader_objects.h"
+#include "except.h"
 #include "extension.h"
 #include "shader.h"
 
@@ -14,29 +15,29 @@ using namespace std;
 namespace Msp {
 namespace GL {
 
-Shader::Shader(ShaderType t):
-	compiled(false)
+Shader::Shader(ShaderType t)
 {
-	if(t==FRAGMENT_SHADER)
-		require_extension("GL_ARB_fragment_program");
-	else if(t==VERTEX_SHADER)
-		require_extension("GL_ARB_vertex_program");
+	init(t);
+}
 
-	id=glCreateShaderObjectARB(t);
+Shader::Shader(ShaderType t, const string &src)
+{
+	init(t);
+
+	source(src);
+	compile();
 }
 
-Shader::Shader(ShaderType t, const string &src):
-	compiled(false)
+void Shader::init(ShaderType t)
 {
+	compiled = false;
+
 	if(t==FRAGMENT_SHADER)
-		require_extension("GL_ARB_fragment_program");
+		static RequireExtension _ext("GL_ARB_fragment_shader");
 	else if(t==VERTEX_SHADER)
-		require_extension("GL_ARB_vertex_program");
-
-	id=glCreateShaderObjectARB(t);
+		static RequireExtension _ext("GL_ARB_vertex_shader");
 
-	source(src);
-	compile();
+	id = glCreateShaderObjectARB(t);
 }
 
 Shader::~Shader()
@@ -44,7 +45,7 @@ Shader::~Shader()
 	glDeleteObjectARB(id);
 }
 
-void Shader::source(sizei count, const char **str, const int *len)
+void Shader::source(unsigned count, const char **str, const int *len)
 {
 	glShaderSourceARB(id, count, str, len);
 }
@@ -59,11 +60,11 @@ void Shader::source(const char *str, int len)
 	source(1, &str, &len);
 }
 
-bool Shader::compile()
+void Shader::compile()
 {
 	glCompileShaderARB(id);
-	compiled=get_param(GL_COMPILE_STATUS);
-	return compiled;
+	if(!(compiled = get_param(GL_COMPILE_STATUS)))
+		throw CompileError(get_info_log());
 }
 
 int Shader::get_param(GLenum param) const
@@ -75,9 +76,9 @@ int Shader::get_param(GLenum param) const
 
 string Shader::get_info_log() const
 {
-	sizei len=get_param(GL_INFO_LOG_LENGTH);
+	GLsizei len = get_param(GL_INFO_LOG_LENGTH);
 	char log[len+1];
-	glGetInfoLogARB(id, len+1, reinterpret_cast<GLsizei *>(&len), log);
+	glGetInfoLogARB(id, len+1, &len, log);
 	return string(log, len);
 }