Mesa (master): mesa: fix MSVC compilation errors

Brian Paul brianp at kemper.freedesktop.org
Thu Jan 8 21:37:17 UTC 2015


Module: Mesa
Branch: master
Commit: e2bf5b183b104d7c3b7b458e655bd16cd4f2b702
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=e2bf5b183b104d7c3b7b458e655bd16cd4f2b702

Author: Brian Paul <brianp at vmware.com>
Date:   Thu Jan  8 14:08:58 2015 -0700

mesa: fix MSVC compilation errors

Move assertions after declarations and don't use void pointer arithmetic.

Reviewed-by: Jose Fonseca <jfonseca at vmware.com>

---

 src/mesa/main/texgetimage.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c
index 58d9cc5..96b019d 100644
--- a/src/mesa/main/texgetimage.c
+++ b/src/mesa/main/texgetimage.c
@@ -811,13 +811,12 @@ getteximage_error_check(struct gl_context *ctx,
                         GLenum format, GLenum type, GLsizei clientMemSize,
                         GLvoid *pixels, bool dsa)
 {
-   assert(texImage);
-
    const GLint maxLevels = _mesa_max_texture_levels(ctx, target);
    const GLuint dimensions = (target == GL_TEXTURE_3D) ? 3 : 2;
    GLenum baseFormat;
    const char *suffix = dsa ? "ture" : "";
 
+   assert(texImage);
    assert(maxLevels != 0);
    if (level < 0 || level >= maxLevels) {
       _mesa_error(ctx, GL_INVALID_VALUE,
@@ -1133,7 +1132,7 @@ _mesa_GetTextureImage(GLuint texture, GLint level, GLenum format,
          image_stride = _mesa_image_image_stride(&ctx->Pack, texImage->Width,
                                                  texImage->Height, format,
                                                  type);
-         pixels += image_stride;
+         pixels = (GLubyte *) pixels + image_stride;
          bufSize -= image_stride;
       }
    }
@@ -1158,11 +1157,12 @@ getcompressedteximage_error_check(struct gl_context *ctx,
                                   GLint level, GLsizei clientMemSize,
                                   GLvoid *img, bool dsa)
 {
-   assert(texImage);
    const GLint maxLevels = _mesa_max_texture_levels(ctx, target);
    GLuint compressedSize, dimensions;
    const char *suffix = dsa ? "ture" : "";
 
+   assert(texImage);
+
    if (!legal_getteximage_target(ctx, target, dsa)) {
       _mesa_error(ctx, GL_INVALID_ENUM,
                   "glGetCompressedTex%sImage(target=%s)", suffix,
@@ -1343,7 +1343,7 @@ _mesa_GetCompressedTextureImage(GLuint texture, GLint level,
                                                 texImage->Width,
                                                 texImage->Height, 1);
 
-         pixels += image_stride;
+         pixels = (GLubyte *) pixels + image_stride;
          bufSize -= image_stride;
       }
    }




More information about the mesa-commit mailing list