From 619baeae3ca5b9f993d1c6ddd9c4c0541e5a4f11 Mon Sep 17 00:00:00 2001 From: Mathias Fröhlich Date: Wed, 29 Feb 2012 18:19:33 +0100 Subject: mesa: Remove gl_light_attrib::_Flags. This variable is only used locally in _mesa_update_lighting. Reviewed-by: Alex Deucher Reviewed-by: Brian Paul Signed-off-by: Mathias Froehlich --- src/mesa/main/light.c | 8 ++++---- src/mesa/main/mtypes.h | 1 - src/mesa/x86/gen_matypes.c | 1 - 3 files changed, 4 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c index 552efa7f0a6..7ebf9bf51cf 100644 --- a/src/mesa/main/light.c +++ b/src/mesa/main/light.c @@ -1003,23 +1003,23 @@ _mesa_validate_all_lighting_tables( struct gl_context *ctx ) void _mesa_update_lighting( struct gl_context *ctx ) { + GLbitfield flags = 0; struct gl_light *light; ctx->Light._NeedEyeCoords = GL_FALSE; - ctx->Light._Flags = 0; if (!ctx->Light.Enabled) return; foreach(light, &ctx->Light.EnabledList) { - ctx->Light._Flags |= light->_Flags; + flags |= light->_Flags; } ctx->Light._NeedVertices = - ((ctx->Light._Flags & (LIGHT_POSITIONAL|LIGHT_SPOT)) || + ((flags & (LIGHT_POSITIONAL|LIGHT_SPOT)) || ctx->Light.Model.ColorControl == GL_SEPARATE_SPECULAR_COLOR || ctx->Light.Model.LocalViewer); - ctx->Light._NeedEyeCoords = ((ctx->Light._Flags & LIGHT_POSITIONAL) || + ctx->Light._NeedEyeCoords = ((flags & LIGHT_POSITIONAL) || ctx->Light.Model.LocalViewer); /* XXX: This test is overkill & needs to be fixed both for software and diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index b5753df1a64..2f5f85f4ea7 100644 --- a/src/mesa/main/mtypes.h +++ b/src/mesa/main/mtypes.h @@ -966,7 +966,6 @@ struct gl_light_attrib /*@{*/ GLboolean _NeedEyeCoords; GLboolean _NeedVertices; /**< Use fast shader? */ - GLbitfield _Flags; /**< LIGHT_* flags, see above */ GLfloat _BaseColor[2][3]; /*@}*/ }; diff --git a/src/mesa/x86/gen_matypes.c b/src/mesa/x86/gen_matypes.c index 7af82b4aea6..4fe99e799ff 100644 --- a/src/mesa/x86/gen_matypes.c +++ b/src/mesa/x86/gen_matypes.c @@ -103,7 +103,6 @@ int main( int argc, char **argv ) OFFSET( "CTX_LIGHT_COLOR_MAT_ENABLED ", struct gl_context, Light.ColorMaterialEnabled ); OFFSET( "CTX_LIGHT_ENABLED_LIST ", struct gl_context, Light.EnabledList ); OFFSET( "CTX_LIGHT_NEED_VERTS ", struct gl_context, Light._NeedVertices ); - OFFSET( "CTX_LIGHT_FLAGS ", struct gl_context, Light._Flags ); OFFSET( "CTX_LIGHT_BASE_COLOR ", struct gl_context, Light._BaseColor ); -- cgit v1.2.3