mesa: Implement EnableVertexArrayAttrib
authorFredrik Höglund <fredrik@kde.org>
Mon, 2 Mar 2015 17:27:58 +0000 (18:27 +0100)
committerFredrik Höglund <fredrik@kde.org>
Fri, 8 May 2015 13:31:02 +0000 (15:31 +0200)
Reviewed-by: Laura Ekstrand <laura@jlekstrand.net>
src/mapi/glapi/gen/ARB_direct_state_access.xml
src/mesa/main/tests/dispatch_sanity.cpp
src/mesa/main/varray.c
src/mesa/main/varray.h

index 3592b1c963f211fb53b83a82689730dd4a807ac8..bffacc6dc132339c814e0f133e226753c02ca579 100644 (file)
       <param name="index" type="GLuint" />
    </function>
 
+   <function name="EnableVertexArrayAttrib" offset="assign">
+      <param name="vaobj" type="GLuint" />
+      <param name="index" type="GLuint" />
+   </function>
+
    <!-- Sampler object functions -->
 
    <function name="CreateSamplers" offset="assign">
index 7909d388fb2ba20da06a5eef68be1372e1081269..db3ea39e74e2c45675ccf59febe5b141ba225a2e 100644 (file)
@@ -1019,6 +1019,7 @@ const struct function gl_core_functions_possible[] = {
    { "glTextureBufferRange", 45, -1 },
    { "glCreateVertexArrays", 45, -1 },
    { "glDisableVertexArrayAttrib", 45, -1 },
+   { "glEnableVertexArrayAttrib", 45, -1 },
    { "glCreateSamplers", 45, -1 },
    { "glCreateProgramPipelines", 45, -1 },
    { "glCreateQueries", 45, -1 },
index b5370a819fae66632ca0b16cbdc5863a4c719207..06b1cbb1d39ba4efcf8fce360241f7d4a98952c0 100644 (file)
@@ -718,20 +718,18 @@ _mesa_VertexAttribLPointer(GLuint index, GLint size, GLenum type,
                 size, type, stride, GL_TRUE, GL_FALSE, GL_TRUE, ptr);
 }
 
-void GLAPIENTRY
-_mesa_EnableVertexAttribArray(GLuint index)
-{
-   struct gl_vertex_array_object *vao;
-   GET_CURRENT_CONTEXT(ctx);
 
+static void
+enable_vertex_array_attrib(struct gl_context *ctx,
+                           struct gl_vertex_array_object *vao,
+                           GLuint index,
+                           const char *func)
+{
    if (index >= ctx->Const.Program[MESA_SHADER_VERTEX].MaxAttribs) {
-      _mesa_error(ctx, GL_INVALID_VALUE,
-                  "glEnableVertexAttribArrayARB(index)");
+      _mesa_error(ctx, GL_INVALID_VALUE, "%s(index)", func);
       return;
    }
 
-   vao = ctx->Array.VAO;
-
    assert(VERT_ATTRIB_GENERIC(index) < ARRAY_SIZE(vao->VertexAttrib));
 
    if (!vao->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled) {
@@ -744,6 +742,36 @@ _mesa_EnableVertexAttribArray(GLuint index)
 }
 
 
+void GLAPIENTRY
+_mesa_EnableVertexAttribArray(GLuint index)
+{
+   GET_CURRENT_CONTEXT(ctx);
+   enable_vertex_array_attrib(ctx, ctx->Array.VAO, index,
+                              "glEnableVertexAttribArray");
+}
+
+
+void GLAPIENTRY
+_mesa_EnableVertexArrayAttrib(GLuint vaobj, GLuint index)
+{
+   GET_CURRENT_CONTEXT(ctx);
+   struct gl_vertex_array_object *vao;
+
+   /* The ARB_direct_state_access specification says:
+    *
+    *   "An INVALID_OPERATION error is generated by EnableVertexArrayAttrib
+    *    and DisableVertexArrayAttrib if <vaobj> is not
+    *    [compatibility profile: zero or] the name of an existing vertex
+    *    array object."
+    */
+   vao = _mesa_lookup_vao_err(ctx, vaobj, "glEnableVertexArrayAttrib");
+   if (!vao)
+      return;
+
+   enable_vertex_array_attrib(ctx, vao, index, "glEnableVertexArrayAttrib");
+}
+
+
 static void
 disable_vertex_array_attrib(struct gl_context *ctx,
                             struct gl_vertex_array_object *vao,
index f783f888553108c1a4f42e7efad207c29bba9806..a86dc6512e71f252613fce8ae7bb2351bd8d807d 100644 (file)
@@ -175,6 +175,10 @@ extern void GLAPIENTRY
 _mesa_EnableVertexAttribArray(GLuint index);
 
 
+extern void GLAPIENTRY
+_mesa_EnableVertexArrayAttrib(GLuint vaobj, GLuint index);
+
+
 extern void GLAPIENTRY
 _mesa_DisableVertexAttribArray(GLuint index);