Changeset 5273 in orxonox.OLD for trunk/src/lib/graphics

Timestamp:

Oct 1, 2005, 10:39:27 PM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: now shaders do not execute (segfault) on non-shader hardware

File:

• trunk/src/lib/graphics/shader.cc (modified) (9 diffs)

trunk/src/lib/graphics/shader.cc

@@ -19 +19 @@
 
 #include "stdlibincl.h"
+#include "compiler.h"
 #include <stdio.h>
 #include "debug.h"
@@ -44 +45 @@
 
    if (GLEW_ARB_shader_objects && GLEW_ARB_shading_language_100)
-     this->shaderProgram = glCreateProgramObjectARB();
-
-   if (vertexShaderFile != NULL)
-     this->loadShaderProgramm(SHADER_VERTEX, vertexShaderFile);
-   if (fragmentShaderFile != NULL)
-     this->loadShaderProgramm(SHADER_FRAGMENT, fragmentShaderFile);
-  try  {
-   glLinkProgramARB(this->shaderProgram); }
-  catch(GLenum errorCode)  {
-   this->printError(this->shaderProgram); }
+     {
+       this->shaderProgram = glCreateProgramObjectARB();
+
+       if (vertexShaderFile != NULL)
+         this->loadShaderProgramm(SHADER_VERTEX, vertexShaderFile);
+       if (fragmentShaderFile != NULL)
+         this->loadShaderProgramm(SHADER_FRAGMENT, fragmentShaderFile);
+       try  {
+         glLinkProgramARB(this->shaderProgram); }
+       catch(GLenum errorCode)  {
+         this->printError(this->shaderProgram); }
+     }
+   else
+     {
+       PRINTF(2)("Shaders are not supported on your hardware\n");
+     }
 }
 
@@ -66 +73 @@
   this->deleteProgram(SHADER_FRAGMENT);
 
-  glDeleteObjectARB(this->fragmentShader);
-  glDeleteObjectARB(this->vertexShader);
-  glDeleteObjectARB(this->shaderProgram);
+  if (this->fragmentShader != 0)
+    glDeleteObjectARB(this->fragmentShader);
+  if (this->vertexShader != 0)
+    glDeleteObjectARB(this->vertexShader);
+  if (this->shaderProgram != 0)
+    glDeleteObjectARB(this->shaderProgram);
 }
 
@@ -98 +108 @@
     shader = this->fragmentShader = glCreateShaderObjectARB(GL_FRAGMENT_SHADER_ARB);
   }
-  glShaderSourceARB(shader, 1, (const GLcharARB**)&program, NULL);
-  try {
-    glCompileShaderARB(shader);
-  }
-  catch (...)
-  {
-    this->printError(shader);
-  }
-  glAttachObjectARB(this->shaderProgram, shader);
-  delete[] program;
-}
-
+
+  if (shader != 0)
+    {
+      glShaderSourceARB(shader, 1, (const GLcharARB**)&program, NULL);
+      try {
+        glCompileShaderARB(shader);
+      }
+      catch (...)
+        {
+          this->printError(shader);
+        }
+      glAttachObjectARB(this->shaderProgram, shader);
+      delete[] program;
+    }
+}
 
 char* Shader::fileRead(const char* fileName)
@@ -139 +152 @@
 void Shader::activateShader()
 {
-  glUseProgramObjectARB(this->shaderProgram);
+  if (likely (this->shaderProgram != 0))
+    glUseProgramObjectARB(this->shaderProgram);
 }
 
@@ -150 +164 @@
 void Shader::deleteProgram(SHADER_TYPE type)
 {
-  if (type == SHADER_VERTEX)
+  if (type == SHADER_VERTEX && this->vertexShader != 0)
   {
     delete[] this->vertexShaderFile;
@@ -157 +171 @@
     this->vertexShader = 0;
   }
-  else if (type == SHADER_FRAGMENT)
+  else if (type == SHADER_FRAGMENT && this->fragmentShader != 0)
   {
     delete[] this->fragmentShaderFile;
@@ -171 +185 @@
 void Shader::printError(GLenum program)
 {
+  if (program == 0)
+    return;
+
   int infologLength = 0;
   int charsWritten  = 0;
@@ -206 +223 @@
         PRINT(3)("%d: %s\n", i, this->fragmentShaderSource->getEntry(i));*/
   }
-
-
-}
-
+}
+