diff options
author | Ian Romanick <[email protected]> | 2011-09-12 11:49:55 -0500 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2011-09-19 10:04:47 -0700 |
commit | 13757f7080426e99511456ea6c44af63b8e7e861 (patch) | |
tree | ec365dc6b97c2e76d5840fb41dccae9bc7ba1b96 | |
parent | 3ebbfc8372d410801c799b3eb7821075dae73b17 (diff) |
mesa: Remove redundant compressed paletted texture error checks
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Brian Paul <[email protected]>
Tested-by: Jin Yang <[email protected]>
-rw-r--r-- | src/mesa/main/texpal.c | 34 |
1 files changed, 5 insertions, 29 deletions
diff --git a/src/mesa/main/texpal.c b/src/mesa/main/texpal.c index 0684638f19e..ed2261b01cb 100644 --- a/src/mesa/main/texpal.c +++ b/src/mesa/main/texpal.c @@ -127,31 +127,6 @@ _mesa_cpal_compressed_size(int level, GLenum internalFormat, return expect_size; } -static const struct cpal_format_info * -cpal_get_info(GLint level, GLenum internalFormat, - GLsizei width, GLsizei height, GLsizei imageSize) -{ - const struct cpal_format_info *info; - - info = &formats[internalFormat - GL_PALETTE4_RGB8_OES]; - ASSERT(info->cpal_format == internalFormat); - - if (level > 0) { - _mesa_error(_mesa_get_current_context(), GL_INVALID_VALUE, - "glCompressedTexImage2D(level=%d)", level); - return NULL; - } - - expect_size = _mesa_cpal_compressed_size(level, internalFormat, - width, height); - if (expect_size > imageSize) { - _mesa_error(_mesa_get_current_context(), GL_INVALID_VALUE, - "glCompressedTexImage2D(imageSize=%d)", imageSize); - return NULL; - } - return info; -} - void _mesa_cpal_compressed_format_type(GLenum internalFormat, GLenum *format, GLenum *type) @@ -184,12 +159,13 @@ _mesa_cpal_compressed_teximage2d(GLenum target, GLint level, GLint saved_align, align; GET_CURRENT_CONTEXT(ctx); - info = cpal_get_info(level, internalFormat, width, height, imageSize); - if (!info) - return; + /* By this point, the internalFormat should have been validated. + */ + assert(internalFormat >= GL_PALETTE4_RGB8_OES + && internalFormat <= GL_PALETTE8_RGB5_A1_OES); info = &formats[internalFormat - GL_PALETTE4_RGB8_OES]; - ASSERT(info->cpal_format == internalFormat); + num_levels = -level + 1; /* first image follows the palette */ |