[PATCH] Fix glReadPixels call from read_from_framebuffer

Jeff Zaroyko jeffzaroyko at gmail.com
Sun Dec 21 00:03:55 CST 2008


On Sun, Dec 21, 2008 at 4:40 PM, Nick Burns <adger44 at hotmail.com> wrote:
>
> This is my last gfx fix for SHOGOThe readpixels call was putting data into the wrong place in the pbo    (fixed with pixelstore)And the y-flip code was flipping the wrong data as well    (set the bottom row to the bottom row and not the height'th row)
> The code used to handle fullscreen 2d blits (or blts without any colorkey masking)However sub-blits had issues (in the pbo path)   1 - readpixels read into the wrong part of the pbo   2 - the y-flip code would move around the uninited data (from the readpixels) and it read from the wrong place   3 - After 1 and 2 the pbo is corrupt and the blt code later had no chance...
> This patch fixes 1 and 2 -- letting the blt code shineThis can be seen in the SHOGO menu (now not corrupt!)
> Changelog     Fix glReadPixels call from read_from_framebuffer    Fix the call to readpixels so that 2d blts going thru the pbo path end up in the right place and get flipped correctly
>
>  - Nick
>

Hi

+    GLint rowLen = 0;
+    GLint skipPix = 0;
+    GLint skipRow = 0;

You're declaring variables in a place which is not the start of a
block, this isn't legal in ANSI C/C90 as gcc points out:
surface.c: In function 'read_from_framebuffer':
surface.c:786: warning: ISO C90 forbids mixed declarations and code

Also it looks like your email address is not correct in this patch?

From: Nick Burns <nwburns at nwburns2.mshome.net>
git repo-config user.email "me at example.com"

-Jeff



More information about the wine-devel mailing list