Re: [Mesa-dev] [PATCH v2 1/3] glx: Simplify __glXReadPixelReply

2012-07-20 Thread Matt Turner
On Fri, Jul 20, 2012 at 2:09 AM, Julien Cristau wrote: Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH v2 1/3] glx: Simplify __glXReadPixelReply

2012-07-20 Thread Alex Deucher
On Fri, Jul 20, 2012 at 5:09 AM, Julien Cristau wrote: > From: Julien Cristau > > Doing > size = reply.length * 4; > [...] > extra = 4 - (size & 3); > > is useless, size & 3 will always be 0. > > Signed-off-by: Julien Cristau Reviewed-by: Alex Deucher > --- > v2: also

[Mesa-dev] [PATCH v2 1/3] glx: Simplify __glXReadPixelReply

2012-07-20 Thread Julien Cristau
From: Julien Cristau Doing size = reply.length * 4; [...] extra = 4 - (size & 3); is useless, size & 3 will always be 0. Signed-off-by: Julien Cristau --- v2: also change the copy in src/glx/apple src/glx/apple/glxreply.c |5 - src/mapi/glapi/gen/g