[Mesa-dev] [PATCH 3/3] i965: Use ctx->Stencil._WriteEnabled in DEPTH_STENCIL_STATE.

[Paul Berry]

On 2 April 2013 11:11, Kenneth Graunke <kenneth at whitecape.org> wrote:

> This is the same computation as the _WriteEnabled flag, so we may as
> well use it.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
>

This series is:

Reviewed-by: Paul Berry <stereotype441 at gmail.com>


> ---
>  src/mesa/drivers/dri/i965/gen6_depthstencil.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen6_depthstencil.c
> b/src/mesa/drivers/dri/i965/gen6_depthstencil.c
> index 4ea517f..940d91f 100644
> --- a/src/mesa/drivers/dri/i965/gen6_depthstencil.c
> +++ b/src/mesa/drivers/dri/i965/gen6_depthstencil.c
> @@ -74,11 +74,7 @@ gen6_upload_depth_stencil_state(struct brw_context *brw)
>          ds->ds1.bf_stencil_test_mask = ctx->Stencil.ValueMask[back];
>        }
>
> -      /* Not really sure about this:
> -       */
> -      if (ctx->Stencil.WriteMask[0] ||
> -         (ctx->Stencil._TestTwoSide && ctx->Stencil.WriteMask[back]))
> -        ds->ds0.stencil_write_enable = 1;
> +      ds->ds0.stencil_write_enable = ctx->Stencil._WriteEnabled;
>     }
>
>     /* _NEW_DEPTH */
> --
> 1.8.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20130404/29baaa4c/attachment-0001.html>