mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-05-02 20:58:04 +02:00
mesa: check for no state change in Enable/DisableVertexAttribArray()
Avoid setting dirty state flags when enabling or disabling a vertex attribute arrays when there's no change. Reviewed-by: José Fonseca <jfonseca@vmware.com>
This commit is contained in:
parent
e14b357367
commit
45453d8f69
1 changed files with 22 additions and 10 deletions
|
|
@ -483,6 +483,7 @@ _mesa_VertexAttribIPointer(GLuint index, GLint size, GLenum type,
|
|||
void GLAPIENTRY
|
||||
_mesa_EnableVertexAttribArrayARB(GLuint index)
|
||||
{
|
||||
struct gl_array_object *arrayObj;
|
||||
GET_CURRENT_CONTEXT(ctx);
|
||||
ASSERT_OUTSIDE_BEGIN_END(ctx);
|
||||
|
||||
|
|
@ -492,18 +493,24 @@ _mesa_EnableVertexAttribArrayARB(GLuint index)
|
|||
return;
|
||||
}
|
||||
|
||||
ASSERT(VERT_ATTRIB_GENERIC(index) < Elements(ctx->Array.ArrayObj->VertexAttrib));
|
||||
arrayObj = ctx->Array.ArrayObj;
|
||||
|
||||
FLUSH_VERTICES(ctx, _NEW_ARRAY);
|
||||
ctx->Array.ArrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled = GL_TRUE;
|
||||
ctx->Array.ArrayObj->_Enabled |= VERT_BIT_GENERIC(index);
|
||||
ctx->Array.NewState |= VERT_BIT_GENERIC(index);
|
||||
ASSERT(VERT_ATTRIB_GENERIC(index) < Elements(arrayObj->VertexAttrib));
|
||||
|
||||
if (!arrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled) {
|
||||
/* was disabled, now being enabled */
|
||||
FLUSH_VERTICES(ctx, _NEW_ARRAY);
|
||||
arrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled = GL_TRUE;
|
||||
arrayObj->_Enabled |= VERT_BIT_GENERIC(index);
|
||||
ctx->Array.NewState |= VERT_BIT_GENERIC(index);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
void GLAPIENTRY
|
||||
_mesa_DisableVertexAttribArrayARB(GLuint index)
|
||||
{
|
||||
struct gl_array_object *arrayObj;
|
||||
GET_CURRENT_CONTEXT(ctx);
|
||||
ASSERT_OUTSIDE_BEGIN_END(ctx);
|
||||
|
||||
|
|
@ -513,12 +520,17 @@ _mesa_DisableVertexAttribArrayARB(GLuint index)
|
|||
return;
|
||||
}
|
||||
|
||||
ASSERT(VERT_ATTRIB_GENERIC(index) < Elements(ctx->Array.ArrayObj->VertexAttrib));
|
||||
arrayObj = ctx->Array.ArrayObj;
|
||||
|
||||
FLUSH_VERTICES(ctx, _NEW_ARRAY);
|
||||
ctx->Array.ArrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled = GL_FALSE;
|
||||
ctx->Array.ArrayObj->_Enabled &= ~VERT_BIT_GENERIC(index);
|
||||
ctx->Array.NewState |= VERT_BIT_GENERIC(index);
|
||||
ASSERT(VERT_ATTRIB_GENERIC(index) < Elements(arrayObj->VertexAttrib));
|
||||
|
||||
if (arrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled) {
|
||||
/* was enabled, now being disabled */
|
||||
FLUSH_VERTICES(ctx, _NEW_ARRAY);
|
||||
arrayObj->VertexAttrib[VERT_ATTRIB_GENERIC(index)].Enabled = GL_FALSE;
|
||||
arrayObj->_Enabled &= ~VERT_BIT_GENERIC(index);
|
||||
ctx->Array.NewState |= VERT_BIT_GENERIC(index);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue