On 07/19/2012 07:46 AM, Julien Cristau wrote:
From: Julien Cristau <julien.cris...@logilab.fr>

Doing
        size = reply.length * 4;
        [...]
        extra = 4 - (size & 3);

is useless, size & 3 will always be 0.

Signed-off-by: Julien Cristau <julien.cris...@logilab.fr>

Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>

---
  src/mapi/glapi/gen/glX_proto_send.py |    5 -----
  1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/src/mapi/glapi/gen/glX_proto_send.py 
b/src/mapi/glapi/gen/glX_proto_send.py
index bec0222..9068a61 100644
--- a/src/mapi/glapi/gen/glX_proto_send.py
+++ b/src/mapi/glapi/gen/glX_proto_send.py
@@ -246,12 +246,7 @@ __glXReadPixelReply( Display *dpy, struct glx_context * 
gc, unsigned max_dim,
              __glXSetError(gc, GL_OUT_OF_MEMORY);
          }
          else {
-            const GLint extra = 4 - (size & 3);
-
              _XRead(dpy, buf, size);
-            if ( extra < 4 ) {
-                _XEatData(dpy, extra);
-            }

              __glEmptyImage(gc, 3, width, height, depth, format, type,
                             buf, dest);



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

Reply via email to