mesa: rename gl_vertex_array_object::_VertexAttrib -> _VertexArray
authorBrian Paul <brianp@vmware.com>
Mon, 5 Feb 2018 16:33:58 +0000 (09:33 -0700)
committerBrian Paul <brianp@vmware.com>
Tue, 6 Feb 2018 22:36:47 +0000 (15:36 -0700)
Since the type is gl_vertex_array.  Update comment to explain that
these arrays are only used by the VBO module.

Also rename some local variables in _mesa_update_vao_derived_arrays().

Reviewed-by: Mathias Fröhlich <mathias.froehlich@web.de>
src/mesa/main/arrayobj.c
src/mesa/main/attrib.c
src/mesa/main/mtypes.h
src/mesa/vbo/vbo_exec_array.c

index 360d097ec1d8ce8b49df6aafbb093440f1fe0147..a6fa33c82c8033d390c52839e0862e07361be911 100644 (file)
@@ -283,8 +283,8 @@ unbind_array_object_vbos(struct gl_context *ctx, struct gl_vertex_array_object *
    for (i = 0; i < ARRAY_SIZE(obj->BufferBinding); i++)
       _mesa_reference_buffer_object(ctx, &obj->BufferBinding[i].BufferObj, NULL);
 
-   for (i = 0; i < ARRAY_SIZE(obj->_VertexAttrib); i++)
-      _mesa_reference_buffer_object(ctx, &obj->_VertexAttrib[i].BufferObj, NULL);
+   for (i = 0; i < ARRAY_SIZE(obj->_VertexArray); i++)
+      _mesa_reference_buffer_object(ctx, &obj->_VertexArray[i].BufferObj, NULL);
 }
 
 
@@ -453,14 +453,13 @@ _mesa_update_vao_derived_arrays(struct gl_context *ctx,
 
    while (arrays) {
       const int attrib = u_bit_scan(&arrays);
-      struct gl_vertex_array *client_array = &vao->_VertexAttrib[attrib];
-      const struct gl_array_attributes *attrib_array =
+      struct gl_vertex_array *array = &vao->_VertexArray[attrib];
+      const struct gl_array_attributes *attribs =
          &vao->VertexAttrib[attrib];
       const struct gl_vertex_buffer_binding *buffer_binding =
-         &vao->BufferBinding[attrib_array->BufferBindingIndex];
+         &vao->BufferBinding[attribs->BufferBindingIndex];
 
-      _mesa_update_vertex_array(ctx, client_array, attrib_array,
-                                buffer_binding);
+      _mesa_update_vertex_array(ctx, array, attribs, buffer_binding);
    }
 }
 
index a9e4a11d7d773813138d6621170805d5227a36a0..8ac5db062fcecb7bc65b65282ca1ed43d3da8c94 100644 (file)
@@ -1503,7 +1503,7 @@ copy_array_object(struct gl_context *ctx,
    /* skip RefCount */
 
    for (i = 0; i < ARRAY_SIZE(src->VertexAttrib); i++) {
-      _mesa_copy_vertex_array(ctx, &dest->_VertexAttrib[i], &src->_VertexAttrib[i]);
+      _mesa_copy_vertex_array(ctx, &dest->_VertexArray[i], &src->_VertexArray[i]);
       _mesa_copy_vertex_attrib_array(ctx, &dest->VertexAttrib[i], &src->VertexAttrib[i]);
       _mesa_copy_vertex_buffer_binding(ctx, &dest->BufferBinding[i], &src->BufferBinding[i]);
    }
index b6d606386e563e3f4341c70541a1b1b3cbb9e53f..4d6b771411459085d4c30dedddbbe422be993d36 100644 (file)
@@ -1603,9 +1603,9 @@ struct gl_vertex_array_object
     * Derived vertex attribute arrays
     *
     * This is a legacy data structure created from gl_array_attributes and
-    * gl_vertex_buffer_binding, for compatibility with existing driver code.
+    * gl_vertex_buffer_binding, only used by the VBO module at this time.
     */
-   struct gl_vertex_array _VertexAttrib[VERT_ATTRIB_MAX];
+   struct gl_vertex_array _VertexArray[VERT_ATTRIB_MAX];
 
    /** Vertex attribute arrays */
    struct gl_array_attributes VertexAttrib[VERT_ATTRIB_MAX];
index b30e044a8797677225427ee492b1425a55d7f144..9ce0aa38dc29223d6fd493db577db9130470cd02 100644 (file)
@@ -332,7 +332,7 @@ recalculate_input_bindings(struct gl_context *ctx)
    struct vbo_context *vbo = vbo_context(ctx);
    struct vbo_exec_context *exec = &vbo->exec;
    const struct gl_vertex_array_object *vao = ctx->Array.VAO;
-   const struct gl_vertex_array *vertexAttrib = vao->_VertexAttrib;
+   const struct gl_vertex_array *vertexAttrib = vao->_VertexArray;
    const struct gl_vertex_array **inputs = &exec->array.inputs[0];
 
    /* May shuffle the position and generic0 bits around */