summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIan Romanick <[email protected]>2011-09-12 11:49:55 -0500
committerIan Romanick <[email protected]>2011-09-19 10:04:47 -0700
commit13757f7080426e99511456ea6c44af63b8e7e861 (patch)
treeec365dc6b97c2e76d5840fb41dccae9bc7ba1b96
parent3ebbfc8372d410801c799b3eb7821075dae73b17 (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.c34
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 */