vbo: fix node_attrsz[] usage in vbo_bind_vertex_list()
authorBrian Paul <brianp@vmware.com>
Wed, 1 Feb 2012 15:42:48 +0000 (08:42 -0700)
committerBrian Paul <brianp@vmware.com>
Wed, 1 Feb 2012 20:45:09 +0000 (13:45 -0700)
The node_attrsz[] array is initially copied from the node->attrsz[]
array but some values get rewritten.  Thereafter, we need to use the
node_attrsz[] values.

Fixes a bug when replaying a display list that uses generic vertex
array[16] (at least).

NOTE: This is a candidate for the 8.0 branch.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/vbo/vbo_save_draw.c

index 9f0290561e8e9c99025624348cb921bff69ffb8e..b903757c027f5c5200e8ff9b2849172fc3938b70 100644 (file)
@@ -191,7 +191,7 @@ static void vbo_bind_vertex_list(struct gl_context *ctx,
          save->inputs[attr] = &arrays[attr];
 
         arrays[attr].Ptr = (const GLubyte *) NULL + buffer_offset;
-        arrays[attr].Size = node->attrsz[src];
+        arrays[attr].Size = node_attrsz[src];
         arrays[attr].StrideB = node->vertex_size * sizeof(GLfloat);
         arrays[attr].Stride = node->vertex_size * sizeof(GLfloat);
         arrays[attr].Type = GL_FLOAT;
@@ -205,7 +205,7 @@ static void vbo_bind_vertex_list(struct gl_context *ctx,
         
         assert(arrays[attr].BufferObj->Name);
 
-        buffer_offset += node->attrsz[src] * sizeof(GLfloat);
+        buffer_offset += node_attrsz[src] * sizeof(GLfloat);
          varying_inputs |= VERT_BIT(attr);
          ctx->NewState |= _NEW_ARRAY;
       }