st/mesa: remove redundant st_buffer_object::size field and error checks
authorBrian Paul <brianp@vmware.com>
Fri, 19 Jun 2009 18:19:15 +0000 (12:19 -0600)
committerBrian Paul <brianp@vmware.com>
Fri, 19 Jun 2009 18:19:15 +0000 (12:19 -0600)
Just use the gl_buffer_object::Size field.  Remove unnecessary size/offset
error checks.  Core Mesa will have already done these checks before these
functions are called.

src/mesa/state_tracker/st_cb_bufferobjects.c
src/mesa/state_tracker/st_cb_bufferobjects.h

index 7021d732089507f7c37fab3cec4ad743359aaddf..1ecf7bf7844fa8356e72764ed4f67d50cd4e51fa 100644 (file)
@@ -98,9 +98,6 @@ st_bufferobj_subdata(GLcontext *ctx,
 {
    struct st_buffer_object *st_obj = st_buffer_object(obj);
 
-   if (offset >= st_obj->size || size > (st_obj->size - offset))
-      return;
-
    st_cond_flush_pipe_buffer_write(st_context(ctx), st_obj->buffer,
                                   offset, size, data);
 }
@@ -118,9 +115,6 @@ st_bufferobj_get_subdata(GLcontext *ctx,
 {
    struct st_buffer_object *st_obj = st_buffer_object(obj);
 
-   if (offset >= st_obj->size || size > (st_obj->size - offset))
-      return;
-
    st_cond_flush_pipe_buffer_read(st_context(ctx), st_obj->buffer,
                                  offset, size, data);
 }
@@ -172,8 +166,6 @@ st_bufferobj_data(GLcontext *ctx,
       return;
    }
 
-   st_obj->size = size;
-
    if (data)
       st_no_flush_pipe_buffer_write(st_context(ctx), st_obj->buffer, 0,
                                    size, data);
index 0adda5d460445cce1953a6e7c342f21b5e233ee0..fda6d05dd3467118e9ba70f9058b5816e4060e29 100644 (file)
@@ -40,7 +40,6 @@ struct st_buffer_object
 {
    struct gl_buffer_object Base;
    struct pipe_buffer *buffer;  
-   GLsizeiptrARB size;
 };