[Mesa-dev] [PATCH 3/3] mesa: move no-change glDepthFunc check earlier

Brian Paul brianp at vmware.com
Mon May 25 07:41:46 PDT 2015


If the incoming func matches the current state it must be a legal
value so we can do this before the switch statement.
---
 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;
 
-- 
1.9.1



More information about the mesa-dev mailing list