summaryrefslogtreecommitdiff
path: root/src/mesa/main/bufferobj.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/mesa/main/bufferobj.c')
-rw-r--r--src/mesa/main/bufferobj.c43
1 files changed, 7 insertions, 36 deletions
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 8b6ede65d5..0e417a2615 100644
--- a/src/mesa/main/bufferobj.c
+++ b/src/mesa/main/bufferobj.c
@@ -145,7 +145,7 @@ _mesa_new_buffer_object( GLcontext *ctx, GLuint name, GLenum target )
* Delete a buffer object.
*
* This function is intended to be called via
- * \c dd_function_table::DeleteBufferObject.
+ * \c dd_function_table::DeleteBuffer.
*/
void
_mesa_delete_buffer_object( GLcontext *ctx, struct gl_buffer_object *bufObj )
@@ -452,43 +452,14 @@ _mesa_DeleteBuffersARB(GLsizei n, const GLuint *ids)
_mesa_HashLookup(ctx->Shared->BufferObjects, ids[i]);
if (bufObj) {
/* unbind any vertex pointers bound to this buffer */
- GLuint j;
-
- ASSERT(bufObj->Name != 0);
-
- if (ctx->Array.Vertex.BufferObj == bufObj)
- ctx->Array.Vertex.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.Normal.BufferObj == bufObj)
- ctx->Array.Normal.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.Color.BufferObj == bufObj)
- ctx->Array.Color.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.SecondaryColor.BufferObj == bufObj)
- ctx->Array.SecondaryColor.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.FogCoord.BufferObj == bufObj)
- ctx->Array.FogCoord.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.Index.BufferObj == bufObj)
- ctx->Array.Index.BufferObj = ctx->Array.NullBufferObj;
- if (ctx->Array.EdgeFlag.BufferObj == bufObj)
- ctx->Array.EdgeFlag.BufferObj = ctx->Array.NullBufferObj;
- for (j = 0; j < MAX_TEXTURE_UNITS; j++) {
- if (ctx->Array.TexCoord[j].BufferObj == bufObj)
- ctx->Array.TexCoord[j].BufferObj = ctx->Array.NullBufferObj;
- }
- for (j = 0; j < VERT_ATTRIB_MAX; j++) {
- if (ctx->Array.VertexAttrib[j].BufferObj == bufObj)
- ctx->Array.VertexAttrib[j].BufferObj = ctx->Array.NullBufferObj;
- }
-
- /* if deleting bound buffers, rebind to zero */
- if (ctx->Array.ArrayBufferObj == bufObj) {
- _mesa_BindBufferARB( GL_ARRAY_BUFFER_ARB, 0 );
- }
- if (ctx->Array.ElementArrayBufferObj == bufObj) {
- _mesa_BindBufferARB( GL_ELEMENT_ARRAY_BUFFER_ARB, 0 );
- }
-
+ ASSERT(bufObj->Name == ids[i]);
+ /* decrement refcount and delete if <= 0 */
bufObj->RefCount--;
if (bufObj->RefCount <= 0) {
+ /* buffer should not be bound anymore! */
+ ASSERT(ctx->Array.ArrayBufferObj != bufObj);
+ ASSERT(ctx->Array.ElementArrayBufferObj != bufObj);
+ ASSERT(ctx->Array.Vertex.BufferObj != bufObj);
_mesa_remove_buffer_object(ctx, bufObj);
ASSERT(ctx->Driver.DeleteBuffer);
(*ctx->Driver.DeleteBuffer)(ctx, bufObj);