<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 30, 2015 at 3:03 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Jan 30, 2015 at 2:27 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> This fixes a bug on BDW when our meta-based stencil blit path assert-fails<br>
> due to an invalid internal format even though we do support the<br>
> ARB_stencil_texturing extension.<br>
> ---<br>
>  src/mesa/main/teximage.c | 10 ++++++++++<br>
>  1 file changed, 10 insertions(+)<br>
><br>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c<br>
> index 03264dd..0e37613 100644<br>
> --- a/src/mesa/main/teximage.c<br>
> +++ b/src/mesa/main/teximage.c<br>
> @@ -221,6 +221,16 @@ _mesa_base_tex_format( struct gl_context *ctx, GLint internalFormat )<br>
>        }<br>
>     }<br>
><br>
> +   if (ctx->Extensions.ARB_stencil_texturing) {<br>
> +      switch (internalFormat) {<br>
> +         case GL_STENCIL_INDEX:<br>
<br>
</span>Don't indent case.<br>
<span class=""><br>
> +         case GL_STENCIL_INDEX8:<br>
> +            return GL_STENCIL_INDEX;<br>
> +         default:<br>
> +            ; /* fallthrough */<br>
<br>
</span>Just make this a break statement.<br></blockquote><div><br></div><div>Both of the above are consistent with the rest of the function.  How much do we care about style vs. consistency?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I don't really know this code, but it seems fine otherwise.<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div><br></div></div>