Mesa (master): mesa: don't flush and don' t flag _NEW_STENCIL in ClearStencil, ActiveStencilFace

Marek Olšák mareko at kemper.freedesktop.org
Wed Apr 24 01:31:32 UTC 2013


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

Author: Marek Olšák <maraeo at gmail.com>
Date:   Mon Apr 15 02:54:58 2013 +0200

mesa: don't flush and don't flag _NEW_STENCIL in ClearStencil, ActiveStencilFace

The functions don't affect driver state. There is no code that would rely
on vertices being flushed prior to changing the states, and no code that
would check for _NEW_STENCIL before using the states.

Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/main/stencil.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/src/mesa/main/stencil.c b/src/mesa/main/stencil.c
index 3308417..d76842f 100644
--- a/src/mesa/main/stencil.c
+++ b/src/mesa/main/stencil.c
@@ -109,10 +109,6 @@ _mesa_ClearStencil( GLint s )
 {
    GET_CURRENT_CONTEXT(ctx);
 
-   if (ctx->Stencil.Clear == (GLuint) s)
-      return;
-
-   FLUSH_VERTICES(ctx, _NEW_STENCIL);
    ctx->Stencil.Clear = (GLuint) s;
 }
 
@@ -390,7 +386,6 @@ _mesa_ActiveStencilFaceEXT(GLenum face)
    }
 
    if (face == GL_FRONT || face == GL_BACK) {
-      FLUSH_VERTICES(ctx, _NEW_STENCIL);
       ctx->Stencil.ActiveFace = (face == GL_FRONT) ? 0 : 2;
    }
    else {




More information about the mesa-commit mailing list