On 11/04/2012 03:43 PM, Brian Paul wrote:
---
  src/mesa/main/fbobject.c |    5 +++--
  1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index fc5681c..d54f6a9 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@ -2059,7 +2059,8 @@ framebuffer_texture(struct gl_context *ctx, const char 
*caller, GLenum target,
        }
        else if ((texObj->Target == GL_TEXTURE_1D_ARRAY_EXT) ||
                 (texObj->Target == GL_TEXTURE_2D_ARRAY_EXT)) {
-         if (zoffset < 0 || zoffset >= ctx->Const.MaxArrayTextureLayers) {
+         if (zoffset < 0 ||
+             zoffset >= (GLint) ctx->Const.MaxArrayTextureLayers) {
              _mesa_error(ctx, GL_INVALID_VALUE,
                          "glFramebufferTexture%sEXT(layer)", caller);
              return;
@@ -3142,7 +3143,7 @@ invalidate_framebuffer_storage(GLenum target, GLsizei 
numAttachments,
           case GL_COLOR_ATTACHMENT13:
           case GL_COLOR_ATTACHMENT14:
           case GL_COLOR_ATTACHMENT15: {
-            const int k = attachments[i] - GL_COLOR_ATTACHMENT0;
+            unsigned k = attachments[i] - GL_COLOR_ATTACHMENT0;

Why drop const?  No other assignments to k were added...

              if (k >= ctx->Const.MaxColorAttachments) {
                 _mesa_error(ctx, GL_INVALID_OPERATION,
                             "%s(attachment >= max. color attachments)", name);


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to