From 5679ef99b82b7ae8803b91f6dcde19e53c223c5e Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Sun, 21 Aug 2022 18:27:08 -0400 Subject: [PATCH] glapi: remove EXT and ARB suffixes from Draw functions This swaps the function names with aliased names that don't have those suffixes. Acked-by: Pierre-Eric Pelloux-Prayer Acked-By: Mike Blumenkrantz Reviewed-by: Pierre-Eric Pelloux-Prayer Part-of: --- src/glx/indirect_vertex_array.c | 6 ++-- src/glx/tests/indirect_api.cpp | 4 +-- src/mapi/glapi/gen/ARB_draw_instanced.xml | 8 ++--- src/mapi/glapi/gen/GL3x.xml | 4 +-- src/mapi/glapi/gen/gl_API.xml | 4 +-- src/mapi/glapi/gen/static_data.py | 6 ++-- src/mapi/glapi/tests/check_table.cpp | 2 +- src/mesa/main/dlist.c | 18 +++++----- src/mesa/main/draw.c | 12 +++---- src/mesa/main/glthread_draw.c | 56 +++++++++++++++---------------- src/mesa/vbo/vbo_save_api.c | 4 +-- 11 files changed, 62 insertions(+), 62 deletions(-) diff --git a/src/glx/indirect_vertex_array.c b/src/glx/indirect_vertex_array.c index 4d24d43..9a14afa 100644 --- a/src/glx/indirect_vertex_array.c +++ b/src/glx/indirect_vertex_array.c @@ -1096,9 +1096,9 @@ __indirect_glMultiDrawArrays(GLenum mode, const GLint *first, void -__indirect_glMultiDrawElementsEXT(GLenum mode, const GLsizei * count, - GLenum type, const GLvoid * const * indices, - GLsizei primcount) +__indirect_glMultiDrawElements(GLenum mode, const GLsizei * count, + GLenum type, const GLvoid * const * indices, + GLsizei primcount) { struct glx_context *gc = __glXGetCurrentContext(); const __GLXattribute *state = diff --git a/src/glx/tests/indirect_api.cpp b/src/glx/tests/indirect_api.cpp index cc24729..bbff944 100644 --- a/src/glx/tests/indirect_api.cpp +++ b/src/glx/tests/indirect_api.cpp @@ -590,7 +590,7 @@ void __indirect_glSecondaryColor3us(GLushort red, GLushort green, GLushort blue) void __indirect_glSecondaryColor3usv(const GLushort *v) { } void __indirect_glSecondaryColorPointer(GLint size, GLenum type, GLsizei stride, const void *pointer) { } void __indirect_glMultiDrawArrays(GLenum mode, const GLint *first, const GLsizei *count, GLsizei drawcount) { } -void __indirect_glMultiDrawElementsEXT(GLenum mode, const GLsizei *count, GLenum type, const void *const*indices, GLsizei primcount) { } +void __indirect_glMultiDrawElements(GLenum mode, const GLsizei *count, GLenum type, const void *const*indices, GLsizei primcount) { } void __indirect_glFogCoordPointer(GLenum type, GLsizei stride, const void *pointer) { } void __indirect_glFogCoordd(GLdouble coord) { } void __indirect_glFogCoorddv(const GLdouble *coord) { } @@ -1363,7 +1363,7 @@ TEST_F(IndirectAPI, EXT_secondary_color) TEST_F(IndirectAPI, EXT_multi_draw_arrays) { EXPECT_EQ((_glapi_proc) __indirect_glMultiDrawArrays, table[_glapi_get_proc_offset("glMultiDrawArraysEXT")]); - EXPECT_EQ((_glapi_proc) __indirect_glMultiDrawElementsEXT, table[_glapi_get_proc_offset("glMultiDrawElementsEXT")]); + EXPECT_EQ((_glapi_proc) __indirect_glMultiDrawElements, table[_glapi_get_proc_offset("glMultiDrawElementsEXT")]); } TEST_F(IndirectAPI, EXT_fog_coord) diff --git a/src/mapi/glapi/gen/ARB_draw_instanced.xml b/src/mapi/glapi/gen/ARB_draw_instanced.xml index 5a63129..dbfe012 100644 --- a/src/mapi/glapi/gen/ARB_draw_instanced.xml +++ b/src/mapi/glapi/gen/ARB_draw_instanced.xml @@ -8,14 +8,14 @@ - + - + @@ -28,14 +28,14 @@ - + - + diff --git a/src/mapi/glapi/gen/GL3x.xml b/src/mapi/glapi/gen/GL3x.xml index e6fd27b..e6c639b 100644 --- a/src/mapi/glapi/gen/GL3x.xml +++ b/src/mapi/glapi/gen/GL3x.xml @@ -542,14 +542,14 @@ - + - + diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml index 7eb25f8..374c059 100644 --- a/src/mapi/glapi/gen/gl_API.xml +++ b/src/mapi/glapi/gen/gl_API.xml @@ -4804,7 +4804,7 @@ - + @@ -9861,7 +9861,7 @@ - + diff --git a/src/mapi/glapi/gen/static_data.py b/src/mapi/glapi/gen/static_data.py index 0a147b2..7d5a1a0 100644 --- a/src/mapi/glapi/gen/static_data.py +++ b/src/mapi/glapi/gen/static_data.py @@ -692,8 +692,8 @@ offsets = { "GetInfoLogARB": 656, "GetObjectParameterfvARB": 657, "GetObjectParameterivARB": 658, - "DrawArraysInstancedARB": 659, - "DrawElementsInstancedARB": 660, + "DrawArraysInstanced": 659, + "DrawElementsInstanced": 660, "BindFramebuffer": 661, "BindRenderbuffer": 662, "BlitFramebuffer": 663, @@ -1206,7 +1206,7 @@ offsets = { "PushDebugGroup": 1170, "SecondaryColor3fEXT": 1171, "SecondaryColor3fvEXT": 1172, - "MultiDrawElementsEXT": 1173, + "MultiDrawElements": 1173, "FogCoordfEXT": 1174, "FogCoordfvEXT": 1175, "ResizeBuffersMESA": 1176, diff --git a/src/mapi/glapi/tests/check_table.cpp b/src/mapi/glapi/tests/check_table.cpp index 82da3f2..6dad1bb 100644 --- a/src/mapi/glapi/tests/check_table.cpp +++ b/src/mapi/glapi/tests/check_table.cpp @@ -1290,7 +1290,7 @@ const struct name_offset known_dispatch[] = { { "glSecondaryColor3usv", _O(SecondaryColor3usv) }, { "glSecondaryColorPointer", _O(SecondaryColorPointer) }, { "glMultiDrawArrays", _O(MultiDrawArrays) }, - { "glMultiDrawElementsEXT", _O(MultiDrawElementsEXT) }, + { "glMultiDrawElements", _O(MultiDrawElements) }, { "glFogCoordPointer", _O(FogCoordPointer) }, { "glFogCoordd", _O(FogCoordd) }, { "glFogCoorddv", _O(FogCoorddv) }, diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c index 0d92edf..148e3b6 100644 --- a/src/mesa/main/dlist.c +++ b/src/mesa/main/dlist.c @@ -1532,10 +1532,10 @@ save_BlendEquationSeparateiARB(GLuint buf, GLenum modeRGB, GLenum modeA) /* GL_ARB_draw_instanced. */ void GLAPIENTRY -save_DrawArraysInstancedARB(UNUSED GLenum mode, - UNUSED GLint first, - UNUSED GLsizei count, - UNUSED GLsizei primcount) +save_DrawArraysInstanced(UNUSED GLenum mode, + UNUSED GLint first, + UNUSED GLsizei count, + UNUSED GLsizei primcount) { GET_CURRENT_CONTEXT(ctx); _mesa_error(ctx, GL_INVALID_OPERATION, @@ -1543,11 +1543,11 @@ save_DrawArraysInstancedARB(UNUSED GLenum mode, } void GLAPIENTRY -save_DrawElementsInstancedARB(UNUSED GLenum mode, - UNUSED GLsizei count, - UNUSED GLenum type, - UNUSED const GLvoid *indices, - UNUSED GLsizei primcount) +save_DrawElementsInstanced(UNUSED GLenum mode, + UNUSED GLsizei count, + UNUSED GLenum type, + UNUSED const GLvoid *indices, + UNUSED GLsizei primcount) { GET_CURRENT_CONTEXT(ctx); _mesa_error(ctx, GL_INVALID_OPERATION, diff --git a/src/mesa/main/draw.c b/src/mesa/main/draw.c index 8843dce..87d88a81 100644 --- a/src/mesa/main/draw.c +++ b/src/mesa/main/draw.c @@ -1393,8 +1393,8 @@ _mesa_DrawArrays(GLenum mode, GLint start, GLsizei count) * display list mode). */ void GLAPIENTRY -_mesa_DrawArraysInstancedARB(GLenum mode, GLint start, GLsizei count, - GLsizei numInstances) +_mesa_DrawArraysInstanced(GLenum mode, GLint start, GLsizei count, + GLsizei numInstances) { GET_CURRENT_CONTEXT(ctx); FLUSH_FOR_DRAW(ctx); @@ -1835,8 +1835,8 @@ _mesa_DrawElementsBaseVertex(GLenum mode, GLsizei count, GLenum type, * Called by glDrawElementsInstanced() in immediate mode. */ void GLAPIENTRY -_mesa_DrawElementsInstancedARB(GLenum mode, GLsizei count, GLenum type, - const GLvoid * indices, GLsizei numInstances) +_mesa_DrawElementsInstanced(GLenum mode, GLsizei count, GLenum type, + const GLvoid * indices, GLsizei numInstances) { GET_CURRENT_CONTEXT(ctx); FLUSH_FOR_DRAW(ctx); @@ -2076,8 +2076,8 @@ _mesa_validated_multidrawelements(struct gl_context *ctx, GLenum mode, void GLAPIENTRY -_mesa_MultiDrawElementsEXT(GLenum mode, const GLsizei *count, GLenum type, - const GLvoid * const *indices, GLsizei primcount) +_mesa_MultiDrawElements(GLenum mode, const GLsizei *count, GLenum type, + const GLvoid * const *indices, GLsizei primcount) { GET_CURRENT_CONTEXT(ctx); FLUSH_FOR_DRAW(ctx); diff --git a/src/mesa/main/glthread_draw.c b/src/mesa/main/glthread_draw.c index a7bb30d..fc6cd3c 100644 --- a/src/mesa/main/glthread_draw.c +++ b/src/mesa/main/glthread_draw.c @@ -292,7 +292,7 @@ _mesa_unmarshal_DrawArrays(struct gl_context *ctx, } /* DrawArraysInstancedBaseInstance and NOT supporting user buffers. Ignore the name. */ -struct marshal_cmd_DrawArraysInstancedARB +struct marshal_cmd_DrawArraysInstanced { struct marshal_cmd_base cmd_base; GLenum mode; @@ -303,9 +303,9 @@ struct marshal_cmd_DrawArraysInstancedARB }; uint32_t -_mesa_unmarshal_DrawArraysInstancedARB(struct gl_context *ctx, - const struct marshal_cmd_DrawArraysInstancedARB *cmd, - const uint64_t *last) +_mesa_unmarshal_DrawArraysInstanced(struct gl_context *ctx, + const struct marshal_cmd_DrawArraysInstanced *cmd, + const uint64_t *last) { /* Ignore the function name. We use DISPATCH_CMD_DrawArrays{Instanced} * for all DrawArrays variants without user buffers, and @@ -339,9 +339,9 @@ draw_arrays_async(struct gl_context *ctx, GLenum mode, GLint first, cmd->first = first; cmd->count = count; } else { - int cmd_size = sizeof(struct marshal_cmd_DrawArraysInstancedARB); - struct marshal_cmd_DrawArraysInstancedARB *cmd = - _mesa_glthread_allocate_command(ctx, DISPATCH_CMD_DrawArraysInstancedARB, cmd_size); + int cmd_size = sizeof(struct marshal_cmd_DrawArraysInstanced); + struct marshal_cmd_DrawArraysInstanced *cmd = + _mesa_glthread_allocate_command(ctx, DISPATCH_CMD_DrawArraysInstanced, cmd_size); cmd->mode = mode; cmd->first = first; @@ -613,7 +613,7 @@ sync: } /* DrawElementsInstanced not supporting user buffers. */ -struct marshal_cmd_DrawElementsInstancedARB +struct marshal_cmd_DrawElementsInstanced { struct marshal_cmd_base cmd_base; GLenum16 mode; @@ -628,9 +628,9 @@ struct marshal_cmd_DrawElementsInstancedARB */ uint32_t -_mesa_unmarshal_DrawElementsInstancedARB(struct gl_context *ctx, - const struct marshal_cmd_DrawElementsInstancedARB *cmd, - const uint64_t *last) +_mesa_unmarshal_DrawElementsInstanced(struct gl_context *ctx, + const struct marshal_cmd_DrawElementsInstanced *cmd, + const uint64_t *last) { /* Ignore the function name. We use DISPATCH_CMD_DrawElementsInstanced- * BaseVertexBaseInstance for all DrawElements variants with user buffers, @@ -644,8 +644,8 @@ _mesa_unmarshal_DrawElementsInstancedARB(struct gl_context *ctx, const GLvoid *indices = cmd->indices; const GLsizei instance_count = cmd->instance_count; - CALL_DrawElementsInstancedARB(ctx->CurrentServerDispatch, - (mode, count, type, indices, instance_count)); + CALL_DrawElementsInstanced(ctx->CurrentServerDispatch, + (mode, count, type, indices, instance_count)); const unsigned cmd_size = align(sizeof(*cmd), 8) / 8; assert(cmd_size == cmd->cmd_base.cmd_size); return cmd_size; @@ -794,9 +794,9 @@ draw_elements_async(struct gl_context *ctx, GLenum mode, GLsizei count, } } else { if (basevertex == 0 && baseinstance == 0) { - int cmd_size = sizeof(struct marshal_cmd_DrawElementsInstancedARB); - struct marshal_cmd_DrawElementsInstancedARB *cmd = - _mesa_glthread_allocate_command(ctx, DISPATCH_CMD_DrawElementsInstancedARB, cmd_size); + int cmd_size = sizeof(struct marshal_cmd_DrawElementsInstanced); + struct marshal_cmd_DrawElementsInstanced *cmd = + _mesa_glthread_allocate_command(ctx, DISPATCH_CMD_DrawElementsInstanced, cmd_size); cmd->mode = MIN2(mode, 0xffff); cmd->type = MIN2(type, 0xffff); @@ -1086,8 +1086,8 @@ _mesa_unmarshal_MultiDrawElementsBaseVertex(struct gl_context *ctx, (mode, count, type, indices, draw_count, basevertex)); } else { - CALL_MultiDrawElementsEXT(ctx->CurrentServerDispatch, - (mode, count, type, indices, draw_count)); + CALL_MultiDrawElements(ctx->CurrentServerDispatch, + (mode, count, type, indices, draw_count)); } /* Restore states. */ @@ -1294,8 +1294,8 @@ sync: (mode, count, type, indices, draw_count, basevertex)); } else { - CALL_MultiDrawElementsEXT(ctx->CurrentServerDispatch, - (mode, count, type, indices, draw_count)); + CALL_MultiDrawElements(ctx->CurrentServerDispatch, + (mode, count, type, indices, draw_count)); } } @@ -1306,8 +1306,8 @@ _mesa_marshal_DrawArrays(GLenum mode, GLint first, GLsizei count) } void GLAPIENTRY -_mesa_marshal_DrawArraysInstancedARB(GLenum mode, GLint first, GLsizei count, - GLsizei instance_count) +_mesa_marshal_DrawArraysInstanced(GLenum mode, GLint first, GLsizei count, + GLsizei instance_count) { draw_arrays(mode, first, count, instance_count, 0, false); } @@ -1336,8 +1336,8 @@ _mesa_marshal_DrawRangeElements(GLenum mode, GLuint start, GLuint end, } void GLAPIENTRY -_mesa_marshal_DrawElementsInstancedARB(GLenum mode, GLsizei count, GLenum type, - const GLvoid *indices, GLsizei instance_count) +_mesa_marshal_DrawElementsInstanced(GLenum mode, GLsizei count, GLenum type, + const GLvoid *indices, GLsizei instance_count) { draw_elements(mode, count, type, indices, instance_count, 0, 0, false, 0, 0, false); } @@ -1383,9 +1383,9 @@ _mesa_marshal_DrawElementsInstancedBaseVertexBaseInstance(GLenum mode, GLsizei c } void GLAPIENTRY -_mesa_marshal_MultiDrawElementsEXT(GLenum mode, const GLsizei *count, - GLenum type, const GLvoid *const *indices, - GLsizei draw_count) +_mesa_marshal_MultiDrawElements(GLenum mode, const GLsizei *count, + GLenum type, const GLvoid *const *indices, + GLsizei draw_count) { _mesa_marshal_MultiDrawElementsBaseVertex(mode, count, type, indices, draw_count, NULL); @@ -1413,7 +1413,7 @@ _mesa_unmarshal_DrawElementsInstancedBaseInstance(struct gl_context *ctx, const } uint32_t -_mesa_unmarshal_MultiDrawElementsEXT(struct gl_context *ctx, const struct marshal_cmd_MultiDrawElementsEXT *cmd, const uint64_t *last) +_mesa_unmarshal_MultiDrawElements(struct gl_context *ctx, const struct marshal_cmd_MultiDrawElements *cmd, const uint64_t *last) { unreachable("never used - MultiDrawElementsBaseVertex is used instead"); return 0; diff --git a/src/mesa/vbo/vbo_save_api.c b/src/mesa/vbo/vbo_save_api.c index 6379bcb..8283792 100644 --- a/src/mesa/vbo/vbo_save_api.c +++ b/src/mesa/vbo/vbo_save_api.c @@ -1881,8 +1881,8 @@ save_DrawRangeElementsBaseVertex(GLenum mode, GLuint start, GLuint end, } void GLAPIENTRY -save_MultiDrawElementsEXT(GLenum mode, const GLsizei *count, GLenum type, - const GLvoid * const *indices, GLsizei primcount) +save_MultiDrawElements(GLenum mode, const GLsizei *count, GLenum type, + const GLvoid * const *indices, GLsizei primcount) { GET_CURRENT_CONTEXT(ctx); struct _glapi_table *dispatch = ctx->CurrentServerDispatch; -- 2.7.4