From: Eric Anholt <eric@anholt.net>
Date: Thu, 5 Mar 2009 18:57:43 +0000 (-0800)
Subject: i965: Add a note about why the _NEW_STENCIL is required in draw_buffers.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=98826950529041a022f736d183b52421eedead31;p=mesa.git

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

diff --git a/src/mesa/drivers/dri/intel/intel_buffers.c b/src/mesa/drivers/dri/intel/intel_buffers.c
index 732bae5b5ae..0929a2c223c 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);
    }