document some assumptions made about compressed texture tile size
authorBrian Paul <brian.paul@tungstengraphics.com>
Fri, 2 Jul 2004 16:39:59 +0000 (16:39 +0000)
committerBrian Paul <brian.paul@tungstengraphics.com>
Fri, 2 Jul 2004 16:39:59 +0000 (16:39 +0000)
src/mesa/main/teximage.c

index d11b6e46094a90df58c65c16f427dd3b78ad53db..b1f526c175d8336c17506c078539baa1d2e5d3e1 100644 (file)
@@ -2833,6 +2833,9 @@ compressed_texture_error_check(GLcontext *ctx, GLint dimensions,
 
 /**
  * Error checking for glCompressedTexSubImage[123]D().
+ * \warning  There are some bad assumptions here about the size of compressed
+ *           texture tiles (multiple of 4) used to test the validity of the
+ *           offset and size parameters.
  * \return error code or GL_NO_ERROR.
  */
 static GLenum
@@ -2890,6 +2893,9 @@ compressed_subtexture_error_check(GLcontext *ctx, GLint dimensions,
    if (level < 0 || level >= maxLevels)
       return GL_INVALID_VALUE;
 
+   /* XXX these tests are specific to the compressed format.
+    * this code should be generalized in some way.
+    */
    if ((xoffset & 3) != 0 || (yoffset & 3) != 0)
       return GL_INVALID_VALUE;