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 - 2 files changed, 4 insertions(+), 5 deletions(-) (limited to 'src/mesa/main') 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]; /*@}*/ }; -- cgit v1.2.3