Mesa (master): mesa: move no-change glDepthFunc check earlier

Brian Paul brianp at kemper.freedesktop.org
Wed Jun 3 17:38:47 UTC 2015


Module: Mesa
Branch: master
Commit: 56b2b3d385170ab33934ec71fd9d0a6e0e1af9a8
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=56b2b3d385170ab33934ec71fd9d0a6e0e1af9a8

Author: Brian Paul <brianp at vmware.com>
Date:   Fri May 15 12:12:04 2015 -0600

mesa: move no-change glDepthFunc check earlier

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 at vmware.com>

---

 src/mesa/main/depth.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mesa/main/depth.c b/src/mesa/main/depth.c
index 29851ec..bb4591c 100644
--- a/src/mesa/main/depth.c
+++ b/src/mesa/main/depth.c
@@ -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;
 




More information about the mesa-commit mailing list