<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 12, 2016 at 1:42 AM, Alejandro Piñeiro <span dir="ltr"><<a href="mailto:apinheiro@igalia.com" target="_blank">apinheiro@igalia.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 12/03/16 00:16, Anuj Phogat wrote:<br>
> Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
<br>
Any reason to not just move the slice assert at line 243 as part of the<br>
checks of check_map_teximage?<br></blockquote><div>Didn't notice it. I'll take your suggestion and send out a separate patch.</div><div>Dropping this one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
> ---<br>
>  src/mesa/swrast/s_texture.c | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/mesa/swrast/s_texture.c b/src/mesa/swrast/s_texture.c<br>
> index 9ccd0e3..6ea7b6c 100644<br>
> --- a/src/mesa/swrast/s_texture.c<br>
> +++ b/src/mesa/swrast/s_texture.c<br>
> @@ -178,7 +178,7 @@ _swrast_free_texture_image_buffer(struct gl_context *ctx,<br>
>   */<br>
>  static void<br>
>  check_map_teximage(const struct gl_texture_image *texImage,<br>
> -                   GLuint slice, GLuint x, GLuint y, GLuint w, GLuint h)<br>
> +                   GLuint x, GLuint y, GLuint w, GLuint h)<br>
>  {<br>
><br>
>     if (texImage->TexObject->Target == GL_TEXTURE_1D)<br>
> @@ -216,7 +216,7 @@ _swrast_map_teximage(struct gl_context *ctx,<br>
>     GLint stride, texelSize;<br>
>     GLuint bw, bh;<br>
><br>
> -   check_map_teximage(texImage, slice, x, y, w, h);<br>
> +   check_map_teximage(texImage, x, y, w, h);<br>
><br>
>     if (!swImage->Buffer) {<br>
>        /* Either glTexImage was called with a NULL <pixels> argument or<br>
<br>
</div></div></blockquote></div><br></div></div>