Brian Paul <bri...@vmware.com> writes: > --- > src/mesa/main/light.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/src/mesa/main/light.c b/src/mesa/main/light.c > index 4021dbe..f215c93 100644 > --- a/src/mesa/main/light.c > +++ b/src/mesa/main/light.c > @@ -44,14 +44,14 @@ _mesa_ShadeModel( GLenum mode ) > if (MESA_VERBOSE & VERBOSE_API) > _mesa_debug(ctx, "glShadeModel %s\n", _mesa_lookup_enum_by_nr(mode)); > > + if (ctx->Light.ShadeModel == mode) > + return; > + > if (mode != GL_FLAT && mode != GL_SMOOTH) { > _mesa_error(ctx, GL_INVALID_ENUM, "glShadeModel"); > return; > } > > - if (ctx->Light.ShadeModel == mode) > - return; > - > FLUSH_VERTICES(ctx, _NEW_LIGHT); > ctx->Light.ShadeModel = mode;
Patches 1-3 are: Reviewed-by: Eric Anholt <e...@anholt.net> I'm pretty skeptical of adding the memcmp in 6/6, though.
signature.asc
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev