Fixed|GL|OS X: Don’t fake EXT_compiled_vertex_array

We should respect what the driver says is supported.
skyjake · Dec 12, 2013 · ed37dfd · danij-deng · Dec 12, 2013 · ed37dfd
1 parent f0bf647
commit ed37dfd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/doomsday/client/include/gl/sys_opengl.h b/doomsday/client/include/gl/sys_opengl.h
@@ -47,7 +47,6 @@
 #if defined(UNIX) && defined(MACOSX)
 #  define GL_GLEXT_PROTOTYPES
 #  if defined(MACOSX_NATIVESDK) || defined(MACOS_10_7)
-#    define GL_EXT_compiled_vertex_array 1
 #    include <OpenGL/gl.h>
 #    include <OpenGL/glext.h>
 #    include <OpenGL/OpenGL.h>

diff --git a/doomsday/client/src/render/rend_model.cpp b/doomsday/client/src/render/rend_model.cpp
@@ -400,11 +400,15 @@ static void configureArrays(void *vertices, void *colors, int numCoords = 0,
         }
     }
 
+#if GL_EXT_compiled_vertex_array
     if(GL_state.features.elementArrays && lock > 0)
     {
         // 'lock' is the number of vertices to lock.
         glLockArraysEXT(0, lock);
+
+        /// @todo Never unlocked? Maybe this is dead code? -jk
     }
+#endif
 
     DENG_ASSERT(!Sys_GLCheckError());
 }