From 4aea0d6e48e14a7521f0edeb288f55ff4417e0d2 Mon Sep 17 00:00:00 2001 From: Neil Roberts Date: Wed, 10 Dec 2008 12:13:20 +0000 Subject: [PATCH] * clutter/cogl/gl/cogl.c: * clutter/cogl/gl/cogl-defines.h.in: * clutter/cogl/gl/cogl-context.h: * clutter/cogl/common/cogl-mesh.c: Rename the glBufferDataSub function to glBufferSubData. When calling glXGetProcAddress with the former Mesa returns a stub dispatch function which will segfault if you try to use it. With NVIDIA it returns NULL so _cogl_features_init decides the card doesn't have VBO support. --- common/cogl-mesh.c | 14 ++------------ gl/cogl-context.h | 2 +- gl/cogl-defines.h.in | 2 +- gl/cogl.c | 8 ++++---- 4 files changed, 8 insertions(+), 18 deletions(-) diff --git a/common/cogl-mesh.c b/common/cogl-mesh.c index 5b35463ea..88a0ba473 100644 --- a/common/cogl-mesh.c +++ b/common/cogl-mesh.c @@ -145,7 +145,7 @@ #define glGenBuffers ctx->pf_glGenBuffersARB #define glBindBuffer ctx->pf_glBindBufferARB #define glBufferData ctx->pf_glBufferDataARB -#define glBufferDataSub ctx->pf_glBufferDataSubARB +#define glBufferSubData ctx->pf_glBufferSubDataARB #define glDeleteBuffers ctx->pf_glDeleteBuffersARB #define glMapBuffer ctx->pf_glMapBufferARB #define glUnmapBuffer ctx->pf_glUnmapBufferARB @@ -153,14 +153,6 @@ #define GL_ARRAY_BUFFER GL_ARRAY_BUFFER_ARB #endif -#elif defined (HAVE_COGL_GLES) - -/* NB: GLES has had VBOs/GLSL since 1.1, so we don't need any defines in - * this case except for glBufferSubData which, just for the fun of it, has a - * different name: - */ -#define glBufferDataSub glBufferSubData - #elif defined (HAVE_COGL_GLES2) #define glVertexPointer cogl_wrap_glVertexPointer @@ -172,8 +164,6 @@ #define glEnableClientState cogl_wrap_glEnableClientState #define glDisableClientState cogl_wrap_glDisableClientState -#define glBufferDataSub glBufferSubData - #endif /* @@ -979,7 +969,7 @@ upload_multipack_vbo_via_buffer_sub_data (CoglMeshVBO *cogl_vbo) PAD_FOR_ALIGNMENT (offset, gl_type_size); - GE (glBufferDataSub (GL_ARRAY_BUFFER, + GE (glBufferSubData (GL_ARRAY_BUFFER, offset, attribute_size, attribute->u.pointer)); diff --git a/gl/cogl-context.h b/gl/cogl-context.h index df62a53e8..8a2fc877d 100644 --- a/gl/cogl-context.h +++ b/gl/cogl-context.h @@ -115,7 +115,7 @@ typedef struct COGL_PFNGLGENBUFFERSARBPROC pf_glGenBuffersARB; COGL_PFNGLBINDBUFFERARBPROC pf_glBindBufferARB; COGL_PFNGLBUFFERDATAARBPROC pf_glBufferDataARB; - COGL_PFNGLBUFFERDATASUBARBPROC pf_glBufferDataSubARB; + COGL_PFNGLBUFFERSUBDATAARBPROC pf_glBufferSubDataARB; COGL_PFNGLMAPBUFFERARBPROC pf_glMapBufferARB; COGL_PFNGLUNMAPBUFFERARBPROC pf_glUnmapBufferARB; COGL_PFNGLDELETEBUFFERSARBPROC pf_glDeleteBuffersARB; diff --git a/gl/cogl-defines.h.in b/gl/cogl-defines.h.in index 8658bed46..480aafb0e 100644 --- a/gl/cogl-defines.h.in +++ b/gl/cogl-defines.h.in @@ -872,7 +872,7 @@ typedef void GLenum usage); typedef void - (APIENTRYP COGL_PFNGLBUFFERDATASUBARBPROC) + (APIENTRYP COGL_PFNGLBUFFERSUBDATAARBPROC) (GLenum target, GLintptr offset, GLsizeiptr size, diff --git a/gl/cogl.c b/gl/cogl.c index 64bd6faf3..411cd8adc 100644 --- a/gl/cogl.c +++ b/gl/cogl.c @@ -1107,9 +1107,9 @@ _cogl_features_init () ctx->pf_glBufferDataARB = (COGL_PFNGLBUFFERDATAARBPROC) cogl_get_proc_address ("glBufferDataARB"); - ctx->pf_glBufferDataSubARB = - (COGL_PFNGLBUFFERDATASUBARBPROC) - cogl_get_proc_address ("glBufferDataSubARB"); + ctx->pf_glBufferSubDataARB = + (COGL_PFNGLBUFFERSUBDATAARBPROC) + cogl_get_proc_address ("glBufferSubDataARB"); ctx->pf_glDeleteBuffersARB = (COGL_PFNGLDELETEBUFFERSARBPROC) cogl_get_proc_address ("glDeleteBuffersARB"); @@ -1122,7 +1122,7 @@ _cogl_features_init () if (ctx->pf_glGenBuffersARB && ctx->pf_glBindBufferARB && ctx->pf_glBufferDataARB - && ctx->pf_glBufferDataSubARB + && ctx->pf_glBufferSubDataARB && ctx->pf_glDeleteBuffersARB && ctx->pf_glMapBufferARB && ctx->pf_glUnmapBufferARB)