mesa: GL_EXT_secondary_color is not optional
Every driver left in Mesa enables this extension all the time. There's no reason to let it be optional. Signed-off-by: Ian Romanick <ian.d.romanick@intel.com> Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Brian Paul <brianp@vmware.com>
This commit is contained in:

committed by
Kenneth Graunke

parent
b5305a303b
commit
c139708087
@@ -73,7 +73,6 @@ intelInitExtensions(struct gl_context *ctx)
|
||||
ctx->Extensions.EXT_pixel_buffer_object = true;
|
||||
ctx->Extensions.EXT_point_parameters = true;
|
||||
ctx->Extensions.EXT_provoking_vertex = true;
|
||||
ctx->Extensions.EXT_secondary_color = true;
|
||||
ctx->Extensions.EXT_separate_shader_objects = true;
|
||||
ctx->Extensions.EXT_texture_env_dot3 = true;
|
||||
ctx->Extensions.EXT_texture_filter_anisotropic = true;
|
||||
|
@@ -99,7 +99,6 @@ intelInitExtensions(struct gl_context *ctx)
|
||||
ctx->Extensions.EXT_pixel_buffer_object = true;
|
||||
ctx->Extensions.EXT_point_parameters = true;
|
||||
ctx->Extensions.EXT_provoking_vertex = true;
|
||||
ctx->Extensions.EXT_secondary_color = true;
|
||||
ctx->Extensions.EXT_separate_shader_objects = true;
|
||||
ctx->Extensions.EXT_texture_array = true;
|
||||
ctx->Extensions.EXT_texture_env_dot3 = true;
|
||||
|
@@ -201,7 +201,6 @@ nouveau_context_init(struct gl_context *ctx, struct nouveau_screen *screen,
|
||||
ctx->Extensions.EXT_fog_coord = true;
|
||||
ctx->Extensions.EXT_framebuffer_blit = true;
|
||||
ctx->Extensions.EXT_packed_depth_stencil = true;
|
||||
ctx->Extensions.EXT_secondary_color = true;
|
||||
ctx->Extensions.EXT_texture_filter_anisotropic = true;
|
||||
ctx->Extensions.NV_blend_square = true;
|
||||
ctx->Extensions.NV_texture_env_combine4 = true;
|
||||
|
@@ -387,7 +387,6 @@ GLboolean r200CreateContext( gl_api api,
|
||||
ctx->Extensions.EXT_blend_minmax = true;
|
||||
ctx->Extensions.EXT_fog_coord = true;
|
||||
ctx->Extensions.EXT_packed_depth_stencil = true;
|
||||
ctx->Extensions.EXT_secondary_color = true;
|
||||
ctx->Extensions.EXT_texture_env_dot3 = true;
|
||||
ctx->Extensions.EXT_texture_filter_anisotropic = true;
|
||||
ctx->Extensions.EXT_texture_mirror_clamp = true;
|
||||
|
@@ -346,7 +346,6 @@ r100CreateContext( gl_api api,
|
||||
ctx->Extensions.ARB_texture_env_dot3 = true;
|
||||
ctx->Extensions.EXT_fog_coord = true;
|
||||
ctx->Extensions.EXT_packed_depth_stencil = true;
|
||||
ctx->Extensions.EXT_secondary_color = true;
|
||||
ctx->Extensions.EXT_texture_env_dot3 = true;
|
||||
ctx->Extensions.EXT_texture_filter_anisotropic = true;
|
||||
ctx->Extensions.EXT_texture_mirror_clamp = true;
|
||||
|
@@ -755,7 +755,7 @@ _mesa_set_enable(struct gl_context *ctx, GLenum cap, GLboolean state)
|
||||
case GL_COLOR_SUM_EXT:
|
||||
if (ctx->API != API_OPENGL_COMPAT)
|
||||
goto invalid_enum_error;
|
||||
CHECK_EXTENSION2(EXT_secondary_color, ARB_vertex_program, cap);
|
||||
CHECK_EXTENSION(ARB_vertex_program, cap);
|
||||
if (ctx->Fog.ColorSumEnabled == state)
|
||||
return;
|
||||
FLUSH_VERTICES(ctx, _NEW_FOG);
|
||||
@@ -1425,7 +1425,6 @@ _mesa_IsEnabled( GLenum cap )
|
||||
case GL_SECONDARY_COLOR_ARRAY_EXT:
|
||||
if (ctx->API != API_OPENGL_COMPAT)
|
||||
goto invalid_enum_error;
|
||||
CHECK_EXTENSION(EXT_secondary_color);
|
||||
return (ctx->Array.ArrayObj->VertexAttrib[VERT_ATTRIB_COLOR1].Enabled != 0);
|
||||
case GL_POINT_SIZE_ARRAY_OES:
|
||||
if (ctx->API != API_OPENGLES)
|
||||
@@ -1441,7 +1440,7 @@ _mesa_IsEnabled( GLenum cap )
|
||||
case GL_COLOR_SUM_EXT:
|
||||
if (ctx->API != API_OPENGL_COMPAT)
|
||||
goto invalid_enum_error;
|
||||
CHECK_EXTENSION2(EXT_secondary_color, ARB_vertex_program);
|
||||
CHECK_EXTENSION(ARB_vertex_program);
|
||||
return ctx->Fog.ColorSumEnabled;
|
||||
|
||||
/* GL_ARB_multisample */
|
||||
|
@@ -198,7 +198,7 @@ static const struct extension extension_table[] = {
|
||||
{ "GL_EXT_polygon_offset", o(dummy_true), GLL, 1995 },
|
||||
{ "GL_EXT_provoking_vertex", o(EXT_provoking_vertex), GL, 2009 },
|
||||
{ "GL_EXT_rescale_normal", o(dummy_true), GLL, 1997 },
|
||||
{ "GL_EXT_secondary_color", o(EXT_secondary_color), GLL, 1999 },
|
||||
{ "GL_EXT_secondary_color", o(dummy_true), GLL, 1999 },
|
||||
{ "GL_EXT_separate_shader_objects", o(EXT_separate_shader_objects), GLL, 2008 },
|
||||
{ "GL_EXT_separate_specular_color", o(dummy_true), GLL, 1997 },
|
||||
{ "GL_EXT_shadow_funcs", o(EXT_shadow_funcs), GLL, 2002 },
|
||||
@@ -429,7 +429,6 @@ _mesa_enable_sw_extensions(struct gl_context *ctx)
|
||||
ctx->Extensions.EXT_point_parameters = GL_TRUE;
|
||||
ctx->Extensions.EXT_provoking_vertex = GL_TRUE;
|
||||
ctx->Extensions.EXT_shadow_funcs = GL_TRUE;
|
||||
ctx->Extensions.EXT_secondary_color = GL_TRUE;
|
||||
ctx->Extensions.EXT_stencil_two_side = GL_TRUE;
|
||||
ctx->Extensions.EXT_texture_array = GL_TRUE;
|
||||
ctx->Extensions.EXT_texture_compression_latc = GL_TRUE;
|
||||
|
@@ -272,12 +272,6 @@ static const int extra_flush_current[] = {
|
||||
EXTRA_END
|
||||
};
|
||||
|
||||
static const int extra_EXT_secondary_color_flush_current[] = {
|
||||
EXT(EXT_secondary_color),
|
||||
EXTRA_FLUSH_CURRENT,
|
||||
EXTRA_END
|
||||
};
|
||||
|
||||
static const int extra_EXT_fog_coord_flush_current[] = {
|
||||
EXT(EXT_fog_coord),
|
||||
EXTRA_FLUSH_CURRENT,
|
||||
@@ -345,8 +339,6 @@ static const int extra_MESA_texture_array_es3[] = {
|
||||
|
||||
EXTRA_EXT(ARB_texture_cube_map);
|
||||
EXTRA_EXT(MESA_texture_array);
|
||||
EXTRA_EXT2(EXT_secondary_color, ARB_vertex_program);
|
||||
EXTRA_EXT(EXT_secondary_color);
|
||||
EXTRA_EXT(EXT_fog_coord);
|
||||
EXTRA_EXT(NV_fog_distance);
|
||||
EXTRA_EXT(EXT_texture_filter_anisotropic);
|
||||
|
@@ -544,12 +544,12 @@ descriptor=[
|
||||
[ "TRANSPOSE_TEXTURE_MATRIX_ARB", "CONTEXT_MATRIX_T(TextureMatrixStack), NO_EXTRA" ],
|
||||
|
||||
# GL_EXT_secondary_color
|
||||
[ "COLOR_SUM", "CONTEXT_BOOL(Fog.ColorSumEnabled), extra_EXT_secondary_color_ARB_vertex_program" ],
|
||||
[ "CURRENT_SECONDARY_COLOR", "CONTEXT_FIELD(Current.Attrib[VERT_ATTRIB_COLOR1][0], TYPE_FLOATN_4), extra_EXT_secondary_color_flush_current" ],
|
||||
[ "SECONDARY_COLOR_ARRAY", "ARRAY_BOOL(VertexAttrib[VERT_ATTRIB_COLOR1].Enabled), extra_EXT_secondary_color" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_TYPE", "ARRAY_ENUM(VertexAttrib[VERT_ATTRIB_COLOR1].Type), extra_EXT_secondary_color" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_STRIDE", "ARRAY_INT(VertexAttrib[VERT_ATTRIB_COLOR1].Stride), extra_EXT_secondary_color" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_SIZE", "ARRAY_INT(VertexAttrib[VERT_ATTRIB_COLOR1].Size), extra_EXT_secondary_color" ],
|
||||
[ "COLOR_SUM", "CONTEXT_BOOL(Fog.ColorSumEnabled), extra_ARB_vertex_program" ],
|
||||
[ "CURRENT_SECONDARY_COLOR", "CONTEXT_FIELD(Current.Attrib[VERT_ATTRIB_COLOR1][0], TYPE_FLOATN_4), extra_flush_current" ],
|
||||
[ "SECONDARY_COLOR_ARRAY", "ARRAY_BOOL(VertexAttrib[VERT_ATTRIB_COLOR1].Enabled), NO_EXTRA" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_TYPE", "ARRAY_ENUM(VertexAttrib[VERT_ATTRIB_COLOR1].Type), NO_EXTRA" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_STRIDE", "ARRAY_INT(VertexAttrib[VERT_ATTRIB_COLOR1].Stride), NO_EXTRA" ],
|
||||
[ "SECONDARY_COLOR_ARRAY_SIZE", "ARRAY_INT(VertexAttrib[VERT_ATTRIB_COLOR1].Size), NO_EXTRA" ],
|
||||
|
||||
# GL_EXT_fog_coord
|
||||
[ "CURRENT_FOG_COORDINATE", "CONTEXT_FLOAT(Current.Attrib[VERT_ATTRIB_FOG][0]), extra_EXT_fog_coord_flush_current" ],
|
||||
|
@@ -3067,7 +3067,6 @@ struct gl_extensions
|
||||
GLboolean EXT_point_parameters;
|
||||
GLboolean EXT_provoking_vertex;
|
||||
GLboolean EXT_shadow_funcs;
|
||||
GLboolean EXT_secondary_color;
|
||||
GLboolean EXT_separate_shader_objects;
|
||||
GLboolean EXT_stencil_two_side;
|
||||
GLboolean EXT_texture3D;
|
||||
|
@@ -206,8 +206,7 @@ compute_version(struct gl_context *ctx)
|
||||
ctx->Extensions.EXT_blend_func_separate &&
|
||||
ctx->Extensions.EXT_blend_minmax &&
|
||||
ctx->Extensions.EXT_fog_coord &&
|
||||
ctx->Extensions.EXT_point_parameters &&
|
||||
ctx->Extensions.EXT_secondary_color);
|
||||
ctx->Extensions.EXT_point_parameters);
|
||||
const GLboolean ver_1_5 = (ver_1_4 &&
|
||||
ctx->Extensions.ARB_occlusion_query &&
|
||||
ctx->Extensions.EXT_shadow_funcs);
|
||||
|
@@ -1145,11 +1145,6 @@ vtxAttribItem: POSITION
|
||||
}
|
||||
| COLOR optColorType
|
||||
{
|
||||
if (!state->ctx->Extensions.EXT_secondary_color) {
|
||||
yyerror(& @2, state, "GL_EXT_secondary_color not supported");
|
||||
YYERROR;
|
||||
}
|
||||
|
||||
$$ = VERT_ATTRIB_COLOR0 + $2;
|
||||
}
|
||||
| FOGCOORD
|
||||
|
@@ -557,7 +557,6 @@ void st_init_extensions(struct st_context *st)
|
||||
ctx->Extensions.EXT_pixel_buffer_object = GL_TRUE;
|
||||
ctx->Extensions.EXT_point_parameters = GL_TRUE;
|
||||
ctx->Extensions.EXT_provoking_vertex = GL_TRUE;
|
||||
ctx->Extensions.EXT_secondary_color = GL_TRUE;
|
||||
ctx->Extensions.EXT_separate_shader_objects = GL_TRUE;
|
||||
ctx->Extensions.EXT_texture_env_dot3 = GL_TRUE;
|
||||
ctx->Extensions.EXT_vertex_array_bgra = GL_TRUE;
|
||||
|
Reference in New Issue
Block a user