Mesa (master): i965: Add a note about why the _NEW_STENCIL is required in draw_buffers.

Eric Anholt anholt at kemper.freedesktop.org
Fri Mar 6 07:44:11 UTC 2009


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

Author: Eric Anholt <eric at anholt.net>
Date:   Thu Mar  5 10:57:43 2009 -0800

i965: Add a note about why the _NEW_STENCIL is required in draw_buffers.

---

 src/mesa/drivers/dri/intel/intel_buffers.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/dri/intel/intel_buffers.c b/src/mesa/drivers/dri/intel/intel_buffers.c
index 732bae5..0929a2c 100644
--- a/src/mesa/drivers/dri/intel/intel_buffers.c
+++ b/src/mesa/drivers/dri/intel/intel_buffers.c
@@ -277,6 +277,11 @@ intel_draw_buffer(GLcontext * ctx, struct gl_framebuffer *fb)
                          (ctx->Stencil._Enabled && fb->Visual.stencilBits > 0));
    }
    else {
+      /* Mesa's Stencil._Enabled field is updated when
+       * _NEW_BUFFERS | _NEW_STENCIL, but i965 code assumes that the value
+       * only changes with _NEW_STENCIL (which seems sensible).  So flag it
+       * here since this is the _NEW_BUFFERS path.
+       */
       ctx->NewState |= (_NEW_DEPTH | _NEW_STENCIL);
    }
 




More information about the mesa-commit mailing list