From f2c5923f005a3e22ddc12a4bb700d838f470bb1d Mon Sep 17 00:00:00 2001 From: Dan Goodliffe Date: Tue, 15 Nov 2022 17:47:56 +0000 Subject: Shaders can be deleted once attached to a program --- gfx/gl/program.h | 2 +- gfx/gl/shader.cpp | 15 +++++++-------- gfx/gl/shader.h | 5 ++--- 3 files changed, 10 insertions(+), 12 deletions(-) (limited to 'gfx') diff --git a/gfx/gl/program.h b/gfx/gl/program.h index f14b0f3..eb10888 100644 --- a/gfx/gl/program.h +++ b/gfx/gl/program.h @@ -12,7 +12,7 @@ class Program { public: template Program(const S &... srcs) { - (glAttachShader(m_program, srcs), ...); + (glAttachShader(m_program, srcs.compile()), ...); linkAndValidate(); } virtual ~Program() = default; diff --git a/gfx/gl/shader.cpp b/gfx/gl/shader.cpp index 8b0f614..5f83b83 100644 --- a/gfx/gl/shader.cpp +++ b/gfx/gl/shader.cpp @@ -3,16 +3,15 @@ #include #include -Shader::operator GLuint() const +Shader::ShaderRef +Shader::compile() const { - if (!shader) { - shader.emplace(type); - glShaderSource(*shader, 1, &text, &len); - glCompileShader(*shader); + ShaderRef shader {type}; + glShaderSource(shader, 1, &text, &len); + glCompileShader(shader); - CheckShaderError(*shader, GL_COMPILE_STATUS, false, "Error compiling shader!"); - } - return *shader; + CheckShaderError(shader, GL_COMPILE_STATUS, false, "Error compiling shader!"); + return shader; } void diff --git a/gfx/gl/shader.h b/gfx/gl/shader.h index 1a8ddef..a20636d 100644 --- a/gfx/gl/shader.h +++ b/gfx/gl/shader.h @@ -7,15 +7,14 @@ class Shader { public: + using ShaderRef = glRef; constexpr Shader(const GLchar * text, GLint len, GLuint type) : text {text}, len {len}, type {type} { } - operator GLuint() const; + ShaderRef compile() const; static void CheckShaderError(GLuint shader, GLuint flag, bool isProgram, std::string_view errorMessage); private: - using ShaderRef = glRef; const GLchar * text; GLint len; GLuint type; - mutable std::optional shader {}; }; -- cgit v1.2.3