st/mesa: no longer special-case buffer object 0 in st_buffer_object() cast wrapper
authorBrian Paul <brianp@vmware.com>
Fri, 19 Jun 2009 18:12:54 +0000 (12:12 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 19 Jun 2009 18:13:15 +0000 (12:13 -0600)
Since commit 6629a35559ff7e3b993966f697f7c7f68e5a38d9 "mesa: create/destroy
buffer objects via driver functions" this is no longer needed, and actually
was causing a crash during context tear-down.

src/mesa/state_tracker/st_cb_bufferobjects.h

index dcbb5a52336a61a04d838812acf4a5714d46ac27..0adda5d460445cce1953a6e7c342f21b5e233ee0 100644 (file)
@@ -44,18 +44,11 @@ struct st_buffer_object
 };
 
 
-/* Are the obj->Name tests necessary?  Unfortunately yes, mesa
- * allocates a couple of gl_buffer_object structs statically, and the
- * Name == 0 test is the only way to identify them and avoid casting
- * them erroneously to our structs.
- */
+/** cast wrapper */
 static INLINE struct st_buffer_object *
 st_buffer_object(struct gl_buffer_object *obj)
 {
-   if (obj->Name)
-      return (struct st_buffer_object *) obj;
-   else
-      return NULL;
+   return (struct st_buffer_object *) obj;
 }