From 61ed88b1ddf8aea6f74518bcae5c13d9bf4ae822 Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Tue, 21 Jul 2015 18:44:07 -0600 Subject: [PATCH] mesa: move check for no-op glFrontFace call earlier If the new mode matches the current mode, there can be no error. Reviewed-by: Anuj Phogat Reviewed-by: Ian Romanick --- src/mesa/main/polygon.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/main/polygon.c b/src/mesa/main/polygon.c index c1c31660e12..60af88f9857 100644 --- a/src/mesa/main/polygon.c +++ b/src/mesa/main/polygon.c @@ -93,14 +93,14 @@ _mesa_FrontFace( GLenum mode ) if (MESA_VERBOSE&VERBOSE_API) _mesa_debug(ctx, "glFrontFace %s\n", _mesa_enum_to_string(mode)); + if (ctx->Polygon.FrontFace == mode) + return; + if (mode!=GL_CW && mode!=GL_CCW) { _mesa_error( ctx, GL_INVALID_ENUM, "glFrontFace" ); return; } - if (ctx->Polygon.FrontFace == mode) - return; - FLUSH_VERTICES(ctx, _NEW_POLYGON); ctx->Polygon.FrontFace = mode; -- 2.30.2