mesa: move no-change glDepthFunc check earlier
authorBrian Paul <brianp@vmware.com>
Fri, 15 May 2015 18:12:04 +0000 (12:12 -0600)
committerBrian Paul <brianp@vmware.com>
Wed, 3 Jun 2015 17:35:46 +0000 (11:35 -0600)
If the incoming func matches the current state it must be a legal
value so we can do this before the switch statement.

Signed-off-by: Brian Paul <brianp@vmware.com>
src/mesa/main/depth.c

index 29851ecb8a40ec1bc739a3b190e9b1339ea91332..bb4591cf152e88aa79669b01e3525cf9fa3a281e 100644 (file)
@@ -65,6 +65,9 @@ _mesa_DepthFunc( GLenum func )
    if (MESA_VERBOSE & VERBOSE_API)
       _mesa_debug(ctx, "glDepthFunc %s\n", _mesa_lookup_enum_by_nr(func));
 
+   if (ctx->Depth.Func == func)
+      return;
+
    switch (func) {
    case GL_LESS:    /* (default) pass if incoming z < stored z */
    case GL_GEQUAL:
@@ -80,9 +83,6 @@ _mesa_DepthFunc( GLenum func )
       return;
    }
 
-   if (ctx->Depth.Func == func)
-      return;
-
    FLUSH_VERTICES(ctx, _NEW_DEPTH);
    ctx->Depth.Func = func;