mesa: fix compressed_subtexture_error_check() return value

The function should return GLboolean, not GLenum.
If we detect invalid compressed pixel storage parameters, we should
return GL_TRUE, not GL_FALSE so that the function is no-op'd.
An update to the piglit s3tc-errors test will check this.

Reviewed-by: Matt Turner <mattst88@gmail.com>
This commit is contained in:
Brian Paul 2014-08-15 16:55:40 -06:00
parent cf8b680f40
commit 9d9879abed

View file

@ -4182,9 +4182,9 @@ out:
/**
* Error checking for glCompressedTexSubImage[123]D().
* \return error code or GL_NO_ERROR.
* \return GL_TRUE if error, GL_FALSE if no error
*/
static GLenum
static GLboolean
compressed_subtexture_error_check(struct gl_context *ctx, GLint dims,
GLenum target, GLint level,
GLint xoffset, GLint yoffset, GLint zoffset,
@ -4246,7 +4246,7 @@ compressed_subtexture_error_check(struct gl_context *ctx, GLint dims,
if (!_mesa_compressed_pixel_storage_error_check(ctx, dims,
&ctx->Unpack,
"glCompressedTexSubImage")) {
return GL_FALSE;
return GL_TRUE;
}
expectedSize = compressed_tex_size(width, height, depth, format);