October 2011 - Wine-Devel - list.winehq.org

Re: Patch resend etiquette

by Austin English

On Sat, Oct 22, 2011 at 10:48, Erich Hoover <ehoover(a)mines.edu> wrote: > How long should one wait to resend patches marked as "New" on the > patch list? It seems that once they've gone "far enough" down on the > list that there's no chance they'll be looked at. Thanks in advance > for your advice. As with a lot of things in Wine, there is no hard rule, but on irc it's been mentioned a few times to wait until the patch falls off of the list. -- -Austin

Oct. 22, 2011

Re: msctf: increase buffer to have room for full key

by Austin English

On Fri, Oct 21, 2011 at 20:26, Aric Stewart <aric(a)codeweavers.com> wrote: > > found by Dan Kegel with Valgrind. > Fixes bug 25826 And Clang :). -- -Austin

Oct. 22, 2011

Patch resend etiquette

by Erich Hoover

How long should one wait to resend patches marked as "New" on the patch list? It seems that once they've gone "far enough" down on the list that there's no chance they'll be looked at. Thanks in advance for your advice. Erich Hoover ehoover(a)mines.edu

Oct. 22, 2011

re: winmm: Avoid casting return value of Heap(Re)Alloc calls.

by Dan Kegel

Doesn't build here: ccache gcc -c -I. -I. -I../../include -I../../include -D__WINESRC__ -D_WINMM_ -D_REENTRANT -fPIC -Wall -pipe -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wwrite-strings -fno-omit-frame-pointer -Wpointer-arith -Wlogical-op -I/usr/include/freetype2 -g -O0 -Werror -Wno-unused-but-set-variable -Wno-unused-but-set-parameter -o lolvldrv.o lolvldrv.c lolvldrv.c: In function 'MMDRV_InitPerType': lolvldrv.c:406:13: error: pointer of type 'void *' used in arithmetic [-Werror=pointer-arith] lolvldrv.c:411:13: error: pointer of type 'void *' used in arithmetic [-Werror=pointer-arith] cc1: all warnings being treated as errors make[1]: Leaving directory `/home/bob/wineslave.dir/sandbox/slave/runtests-default/build/dlls/winmm' make[1]: *** [lolvldrv.o] Error 1 make: *** [dlls/winmm] Error 2 http://buildbot.kegel.com/builders/runtests-default/builds/874

Oct. 22, 2011

Re: [PATCH 1/5] wined3d: Move the cursor in wined3d_device_set_cursor_position().

by Henri Verbeet

On 21 October 2011 17:12, Stefan Dösinger <stefandoesinger(a)gmx.at> wrote: > Does this address Alexandre's need for a software cursor, or is this > unrelated? > It's related, but software cursors are still somewhat broken.

Oct. 21, 2011

Re: [PATCH 1/5] wined3d: Move the cursor in wined3d_device_set_cursor_position().

by Stefan Dösinger

Hi, Does this address Alexandre's need for a software cursor, or is this unrelated? In the tests I recommend to test a bigger cursor in addition to the 32x32 cursor to see what happens in situations where Windows falls back to a software cursor. Stefan On Thursday 20 October 2011 12:42:55 Henri Verbeet wrote: > --- > dlls/wined3d/device.c | 8 +++++++- > 1 files changed, 7 insertions(+), 1 deletions(-) > > diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c > index 67e50b5..6f43715 100644 > --- a/dlls/wined3d/device.c > +++ b/dlls/wined3d/device.c > @@ -5251,10 +5251,16 @@ void CDECL > wined3d_device_set_cursor_position(struct wined3d_device *device, > device->xScreenSpace = x_screen_space; > device->yScreenSpace = y_screen_space; > > - /* switch to the software cursor if position diverges from the > hardware one */ if (device->hardwareCursor) > { > POINT pt; > + > + GetCursorPos( &pt ); > + if (x_screen_space == pt.x && y_screen_space == pt.y) > + return; > + SetCursorPos( x_screen_space, y_screen_space ); > + > + /* Switch to the software cursor if position diverges from the > hardware one. */ GetCursorPos( &pt ); > if (x_screen_space != pt.x || y_screen_space != pt.y) > {

Oct. 21, 2011

Re: [PATCH 2/2] vbscript: Added remaining week day constants implementations

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15047 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 21, 2011

Re: [PATCH 1/2] vbscript: Added vbSunday constant implementation

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15046 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 21, 2011

Re: dpnet/tests: Avoid casting return value of HeapAlloc calls

by Michael Stefaniuc

Frédéric Delanoy wrote: > On Fri, Oct 21, 2011 at 02:13, Marvin <testbot(a)testbot.winehq.org> wrote: >> Hi, >> >> While running your changed tests on Windows, I think I found new failures. >> Being a bot and all I'm not very good at pattern recognition, so I might be >> wrong, but could you please double-check? >> Full results can be found at >> http://testbot.winehq.org/JobDetails.pl?Key=15042 >> >> Your paranoid android. >> >> >> === WNT4WSSP6 (32 bit peer) === >> peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 >> peer: unhandled exception c0000005 at 004013BF >> >> === W2KPROSP4 (32 bit peer) === >> peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 >> peer: unhandled exception c0000005 at 004013BF > > This also happens when you send a comment-only patch to testbot. Flaky test? Nope. There is no dpnet dll on those boxes. winetest.exe skips those but the WTB doesn't. http://test.winehq.org/data/bbbee31c38cde112cfcc7417872416bf34955367/nt4_wt… "dpnet missing" bye michael

Oct. 21, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Michael Stefaniuc

Vitaliy Margolen wrote: > On 10/20/2011 06:18 PM, Dan Kegel wrote: >> On Thu, Oct 20, 2011 at 5:15 PM, Vitaliy Margolen >> <wine-devel(a)kievinfo.com> wrote: >>>>> device.c:117: Test failed: GetDeviceData() should have failed: >>>>> 80040207 >>>>> device.c:120: Test failed: GetDeviceData() should have failed: >>>>> 80040207 >>>>> make: *** [device.ok] Error 8 >>> >>> Could be because of definition problems. Here's what I get compiling >>> wine64: >>> ../../../../wine.git/dlls/dinput/tests/device.c:117:5: warning: >>> comparison >>> is always false due to limited range of data type >>> ../../../../wine.git/dlls/dinput/tests/device.c:120:5: warning: >>> comparison >>> is always false due to limited range of data type >>> >>> HRESULT defined as "typedef LONG HRESULT;". But error code defined as: >>> #define DIERR_NOTBUFFERED 0x80040207L >>> >>> Not sure what to do here. Test passed on all win64 bit platforms >>> according >>> to test bot. So unless we want to redefine all LONG constants in PSDK >>> (and >>> remove that last L) I'd say ignore this. >> >> How can we ignore it? Once we check it in, the 64 bit build will >> start failing its tests :-( > > So you saying we should modify all long error constants in PSDK? I'm not > sure what else can be done. Yes, that's the right way. I got that as a janitorial task from Alexandre during the last WineConf. One commit is in already http://source.winehq.org/git/wine.git/?a=commit;h=0324fc57469f402102b677c4e… While the search and replace is trivial it uncovers interesting challenges and opportunities in the Wine code... That's why I'm not done yet with this task. bye michael

Oct. 21, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Vitaliy Margolen

On 10/20/2011 06:26 PM, Vitaliy Margolen wrote: > On 10/20/2011 06:18 PM, Dan Kegel wrote: >> On Thu, Oct 20, 2011 at 5:15 PM, Vitaliy Margolen >> <wine-devel(a)kievinfo.com> wrote: >>>>> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >>>>> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >>>>> make: *** [device.ok] Error 8 >>> >>> Could be because of definition problems. Here's what I get compiling wine64: >>> ../../../../wine.git/dlls/dinput/tests/device.c:117:5: warning: comparison >>> is always false due to limited range of data type >>> ../../../../wine.git/dlls/dinput/tests/device.c:120:5: warning: comparison >>> is always false due to limited range of data type >>> >>> HRESULT defined as "typedef LONG HRESULT;". But error code defined as: >>> #define DIERR_NOTBUFFERED 0x80040207L >>> >>> Not sure what to do here. Test passed on all win64 bit platforms according >>> to test bot. So unless we want to redefine all LONG constants in PSDK (and >>> remove that last L) I'd say ignore this. >> >> How can we ignore it? Once we check it in, the 64 bit build will >> start failing its tests :-( > > So you saying we should modify all long error constants in PSDK? I'm not > sure what else can be done. > Ok, casting error code to HRESULT seemed to work. Vitaliy.

Oct. 20, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Vitaliy Margolen

On 10/20/2011 06:18 PM, Dan Kegel wrote: > On Thu, Oct 20, 2011 at 5:15 PM, Vitaliy Margolen > <wine-devel(a)kievinfo.com> wrote: >>>> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >>>> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >>>> make: *** [device.ok] Error 8 >> >> Could be because of definition problems. Here's what I get compiling wine64: >> ../../../../wine.git/dlls/dinput/tests/device.c:117:5: warning: comparison >> is always false due to limited range of data type >> ../../../../wine.git/dlls/dinput/tests/device.c:120:5: warning: comparison >> is always false due to limited range of data type >> >> HRESULT defined as "typedef LONG HRESULT;". But error code defined as: >> #define DIERR_NOTBUFFERED 0x80040207L >> >> Not sure what to do here. Test passed on all win64 bit platforms according >> to test bot. So unless we want to redefine all LONG constants in PSDK (and >> remove that last L) I'd say ignore this. > > How can we ignore it? Once we check it in, the 64 bit build will > start failing its tests :-( So you saying we should modify all long error constants in PSDK? I'm not sure what else can be done. Vitaliy.

Oct. 20, 2011

Re: dpnet/tests: Avoid casting return value of HeapAlloc calls

by Frédéric Delanoy

On Fri, Oct 21, 2011 at 02:13, Marvin <testbot(a)testbot.winehq.org> wrote: > Hi, > > While running your changed tests on Windows, I think I found new failures. > Being a bot and all I'm not very good at pattern recognition, so I might be > wrong, but could you please double-check? > Full results can be found at > http://testbot.winehq.org/JobDetails.pl?Key=15042 > > Your paranoid android. > > > === WNT4WSSP6 (32 bit peer) === > peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 > peer: unhandled exception c0000005 at 004013BF > > === W2KPROSP4 (32 bit peer) === > peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 > peer: unhandled exception c0000005 at 004013BF This also happens when you send a comment-only patch to testbot. Flaky test?

Oct. 20, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Dan Kegel

On Thu, Oct 20, 2011 at 5:15 PM, Vitaliy Margolen <wine-devel(a)kievinfo.com> wrote: >>> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >>> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >>> make: *** [device.ok] Error 8 > > Could be because of definition problems. Here's what I get compiling wine64: > ../../../../wine.git/dlls/dinput/tests/device.c:117:5: warning: comparison > is always false due to limited range of data type > ../../../../wine.git/dlls/dinput/tests/device.c:120:5: warning: comparison > is always false due to limited range of data type > > HRESULT defined as "typedef LONG HRESULT;". But error code defined as: > #define DIERR_NOTBUFFERED 0x80040207L > > Not sure what to do here. Test passed on all win64 bit platforms according > to test bot. So unless we want to redefine all LONG constants in PSDK (and > remove that last L) I'd say ignore this. How can we ignore it? Once we check it in, the 64 bit build will start failing its tests :-( - Dan

Oct. 20, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Vitaliy Margolen

Could be because of definition problems. Here's what I get compiling wine64: ../../../../wine.git/dlls/dinput/tests/device.c:117:5: warning: comparison is always false due to limited range of data type ../../../../wine.git/dlls/dinput/tests/device.c:120:5: warning: comparison is always false due to limited range of data type HRESULT defined as "typedef LONG HRESULT;". But error code defined as: #define DIERR_NOTBUFFERED 0x80040207L Not sure what to do here. Test passed on all win64 bit platforms according to test bot. So unless we want to redefine all LONG constants in PSDK (and remove that last L) I'd say ignore this. Vitaliy. On 10/20/2011 09:46 AM, Dan Kegel wrote: > Hi Vitaliy, > could you check on win64? I had a test failure there with your patch. > - Dan > > On Thu, Oct 20, 2011 at 7:49 AM,<buildbot(a)kegel.com> wrote: >> This is an experimental automated build and test service. >> Please feel free to ignore this email while we work the kinks out. >> >> For more info about this message, see http://wiki.winehq.org/BuildBot >> >> The Buildbot has detected a failed build on builder runtests-default-x86_64 while building Wine. >> Full details are available at: http://buildbot.kegel.com/builders/runtests-default-x86_64/builds/769 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) >> BUILD FAILED: failed shell_3 >> >> Errors: >> alarum: failed command was ../../../wine dinput_test.exe.so device.c >> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:117: Test failed: GetDeviceData() should have failed: 80040207 >> device.c:120: Test failed: GetDeviceData() should have failed: 80040207 >> make: *** [device.ok] Error 8 >> >> > >

Oct. 20, 2011

Re: dpnet/tests: Avoid casting return value of HeapAlloc calls

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15042 Your paranoid android. === WNT4WSSP6 (32 bit peer) === peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 peer: unhandled exception c0000005 at 004013BF === W2KPROSP4 (32 bit peer) === peer.c:52: Test failed: CoCreateInstance failed with 0x80040154 peer: unhandled exception c0000005 at 004013BF

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Evil Jay

On 10/20/2011 02:34 PM, L. Rahyen wrote: > If such a system will be used, I will volunteer too. By the way, I > like this idea because I think it can prevent all or almost all spam > (most spammers never post "intelligent posts"). It would definitely be an improvement. I moderate in some other tech forums and only *rarely* have I seen a bot programmed to post a few generic ("me too" and such) posts before it started spewing the spam.

Oct. 20, 2011

Re: 80200: wininet: add a stub for ShowClientAuthCerts

by Dan Kegel

Fails to build here? On Thu, Oct 20, 2011 at 3:52 PM, <buildbot(a)kegel.com> wrote: > This is an experimental automated build and test service. > Please feel free to ignore this email while we work the kinks out. > > For more info about this message, see http://wiki.winehq.org/BuildBot > > The Buildbot has detected a failed build on builder runtests-default while building Wine. > Full details are available at: http://buildbot.kegel.com/builders/runtests-default/builds/804 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) > BUILD FAILED: failed shell_2 > > Errors: > internet.c: In function 'ShowClientAuthCerts': > internet.c:4391:159: error: format '%p' expects a matching 'void *' argument [-Werror=format] > make[1]: *** [internet.o] Error 1 > make: *** [dlls/wininet] Error 2 > make: *** Waiting for unfinished jobs.... > >

Oct. 20, 2011

Re: 80199: shdocvw: add a stub for InstallReg_RunDLL

by Dan Kegel

Fails to build here? On Thu, Oct 20, 2011 at 3:51 PM, <buildbot(a)kegel.com> wrote: > This is an experimental automated build and test service. > Please feel free to ignore this email while we work the kinks out. > > For more info about this message, see http://wiki.winehq.org/BuildBot > > The Buildbot has detected a failed build on builder runtests-default while building Wine. > Full details are available at: http://buildbot.kegel.com/builders/runtests-default/builds/803 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) > BUILD FAILED: failed shell_2 > > Errors: > shdocvw_main.c: In function 'InstallReg_RunDLL': > shdocvw_main.c:547:159: error: format '%p' expects a matching 'void *' argument [-Werror=format] > shdocvw_main.c:547:159: error: format '%p' expects a matching 'void *' argument [-Werror=format] > make[1]: *** [shdocvw_main.o] Error 1 > make: *** [dlls/shdocvw] Error 2 > make: *** Waiting for unfinished jobs.... > >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by L. Rahyen

On 2011-10-20 (October, Thursday) 19:17:56 Jerome Leclanche wrote: > On Thu, Oct 20, 2011 at 8:07 PM, Josh Juran <josh(a)iswifter.net> wrote: > > Quarantine new users' posts until approved by a human moderator. After a > > few intelligent posts, grant the user unmoderated posting privilege. > > The Boost mailing list does this. > I think that's a great system, but you'll need volunteers :) I can > volunteer for that matter, but I won't exactly spend a lot of time on > it. If such a system will be used, I will volunteer too. By the way, I like this idea because I think it can prevent all or almost all spam (most spammers never post "intelligent posts").

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Ken Sharp

Here here On 20/10/11 8:17 PM, Jerome Leclanche wrote: > I think that's a great system, but you'll need volunteers :) I can > volunteer for that matter, but I won't exactly spend a lot of time on > it. > > JL > > On Thu, Oct 20, 2011 at 8:07 PM, Josh Juran<josh(a)iswifter.net> wrote: >> On Oct 20, 2011, at 11:58 AM, Jerome Leclanche wrote: >> >>> I don't subscribe to websites that have annoying captchas either way; >>> and many people are like that. Bot authors have proved they can beat >>> any system; and captchas (recaptcha especially) has proved itself as >>> one of the most effective and unintrusive free captcha systems. >> >> Quarantine new users' posts until approved by a human moderator. After a few intelligent posts, grant the user unmoderated posting privilege. The Boost mailing list does this. >> >> Josh >> >> >> >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Jerome Leclanche

I think that's a great system, but you'll need volunteers :) I can volunteer for that matter, but I won't exactly spend a lot of time on it. JL On Thu, Oct 20, 2011 at 8:07 PM, Josh Juran <josh(a)iswifter.net> wrote: > On Oct 20, 2011, at 11:58 AM, Jerome Leclanche wrote: > >> I don't subscribe to websites that have annoying captchas either way; >> and many people are like that. Bot authors have proved they can beat >> any system; and captchas (recaptcha especially) has proved itself as >> one of the most effective and unintrusive free captcha systems. > > Quarantine new users' posts until approved by a human moderator. After a few intelligent posts, grant the user unmoderated posting privilege. The Boost mailing list does this. > > Josh > > >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Josh Juran

On Oct 20, 2011, at 11:58 AM, Jerome Leclanche wrote: > I don't subscribe to websites that have annoying captchas either way; > and many people are like that. Bot authors have proved they can beat > any system; and captchas (recaptcha especially) has proved itself as > one of the most effective and unintrusive free captcha systems. Quarantine new users' posts until approved by a human moderator. After a few intelligent posts, grant the user unmoderated posting privilege. The Boost mailing list does this. Josh

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Jerome Leclanche

I don't subscribe to websites that have annoying captchas either way; and many people are like that. Bot authors have proved they can beat any system; and captchas (recaptcha especially) has proved itself as one of the most effective and unintrusive free captcha systems. JL On Thu, Oct 20, 2011 at 7:54 PM, Ken Sharp <kennybobs(a)o2.co.uk> wrote: > Isn't the system for creating new accounts only? If it's just to post > messages then anything will be annoying. > > On 20/10/11 7:48 PM, Jerome Leclanche wrote: >> >> Unless you want to implement a captcha system, any system is going to >> annoy the users. Bots can be programmed to answer those fields pretty >> easily; if a spammer wants to get around it, they will :) >> >> JL >> >> On Thu, Oct 20, 2011 at 7:24 PM, Ken Sharp<kennybobs(a)o2.co.uk> wrote: >>> >>> >>> On 20/10/11 4:13 PM, Bruno Jesus wrote: >>>> >>>> On Thu, Oct 20, 2011 at 11:48, Jeremy Newman<jnewman(a)codeweavers.com> >>>> wrote: >>>>> >>>>> I will take suggestions on other questions that regular users can >>>>> answer >>>>> easily, but would stump (for a bit anyway) a bot, or bot author. >>>> >>>> Why not the good and old random math questions inside an image? For >>>> example 2 + 4 = ... >>>> >>>> >>> >>> A bot should be able to figure that out too, unless the answer is also a >>> picture. >>> >>> Maybe A+B = ? The answer being AB. >>> >>> Don't know if any of this is possible with the software, however. >>> >>> >>> >>> >> >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Ken Sharp

Isn't the system for creating new accounts only? If it's just to post messages then anything will be annoying. On 20/10/11 7:48 PM, Jerome Leclanche wrote: > Unless you want to implement a captcha system, any system is going to > annoy the users. Bots can be programmed to answer those fields pretty > easily; if a spammer wants to get around it, they will :) > > JL > > On Thu, Oct 20, 2011 at 7:24 PM, Ken Sharp<kennybobs(a)o2.co.uk> wrote: >> >> >> On 20/10/11 4:13 PM, Bruno Jesus wrote: >>> >>> On Thu, Oct 20, 2011 at 11:48, Jeremy Newman<jnewman(a)codeweavers.com> >>> wrote: >>>> >>>> I will take suggestions on other questions that regular users can answer >>>> easily, but would stump (for a bit anyway) a bot, or bot author. >>> >>> Why not the good and old random math questions inside an image? For >>> example 2 + 4 = ... >>> >>> >> >> A bot should be able to figure that out too, unless the answer is also a >> picture. >> >> Maybe A+B = ? The answer being AB. >> >> Don't know if any of this is possible with the software, however. >> >> >> >> >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Jerome Leclanche

Unless you want to implement a captcha system, any system is going to annoy the users. Bots can be programmed to answer those fields pretty easily; if a spammer wants to get around it, they will :) JL On Thu, Oct 20, 2011 at 7:24 PM, Ken Sharp <kennybobs(a)o2.co.uk> wrote: > > > On 20/10/11 4:13 PM, Bruno Jesus wrote: >> >> On Thu, Oct 20, 2011 at 11:48, Jeremy Newman<jnewman(a)codeweavers.com> >> wrote: >>> >>> I will take suggestions on other questions that regular users can answer >>> easily, but would stump (for a bit anyway) a bot, or bot author. >> >> Why not the good and old random math questions inside an image? For >> example 2 + 4 = ... >> >> > > A bot should be able to figure that out too, unless the answer is also a > picture. > > Maybe A+B = ? The answer being AB. > > Don't know if any of this is possible with the software, however. > > > >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Ken Sharp

On 20/10/11 4:13 PM, Bruno Jesus wrote: > On Thu, Oct 20, 2011 at 11:48, Jeremy Newman<jnewman(a)codeweavers.com> wrote: >> I will take suggestions on other questions that regular users can answer >> easily, but would stump (for a bit anyway) a bot, or bot author. > Why not the good and old random math questions inside an image? For > example 2 + 4 = ... > > A bot should be able to figure that out too, unless the answer is also a picture. Maybe A+B = ? The answer being AB. Don't know if any of this is possible with the software, however.

Oct. 20, 2011

Re: 80186: [PATCH 2/2] dinput: Restore checks for DirectX 3 buffer size.

by Dan Kegel

Hi Vitaliy, could you check on win64? I had a test failure there with your patch. - Dan On Thu, Oct 20, 2011 at 7:49 AM, <buildbot(a)kegel.com> wrote: > This is an experimental automated build and test service. > Please feel free to ignore this email while we work the kinks out. > > For more info about this message, see http://wiki.winehq.org/BuildBot > > The Buildbot has detected a failed build on builder runtests-default-x86_64 while building Wine. > Full details are available at: http://buildbot.kegel.com/builders/runtests-default-x86_64/builds/769 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) > BUILD FAILED: failed shell_3 > > Errors: > alarum: failed command was ../../../wine dinput_test.exe.so device.c > device.c:117: Test failed: GetDeviceData() should have failed: 80040207 > device.c:120: Test failed: GetDeviceData() should have failed: 80040207 > device.c:117: Test failed: GetDeviceData() should have failed: 80040207 > device.c:120: Test failed: GetDeviceData() should have failed: 80040207 > device.c:117: Test failed: GetDeviceData() should have failed: 80040207 > device.c:120: Test failed: GetDeviceData() should have failed: 80040207 > device.c:117: Test failed: GetDeviceData() should have failed: 80040207 > device.c:120: Test failed: GetDeviceData() should have failed: 80040207 > make: *** [device.ok] Error 8 > >

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Bruno Jesus

On Thu, Oct 20, 2011 at 11:48, Jeremy Newman <jnewman(a)codeweavers.com> wrote: > I will take suggestions on other questions that regular users can answer > easily, but would stump (for a bit anyway) a bot, or bot author. Why not the good and old random math questions inside an image? For example 2 + 4 = ...

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Jeremy Newman

On 10/19/2011 08:03 PM, Vitaliy Margolen wrote: >> The new question is: >> What operating system does Wine run applications from? >> >> Should be obvious for a human. I will take suggestions on other questions that regular users can answer easily, but would stump (for a bit anyway) a bot, or bot author. Our textual question thing, does have the ability to randomly pick from a list of questions. Limiting it to just one makes it easier for a bot author to work around. > Any ETA on the new version of PHPBB? Soon-ish. Upgrading from v2 to v3 is fairly staight forward with the build in conversion tools. I just need find some time to convert our theme files to work on v3. -N

Oct. 20, 2011

Re: attrib: Use FormatMessageW() for better internationalization support.

by Alexandre Julliard

Francois Gouget <fgouget(a)free.fr> writes: > @@ -72,10 +72,12 @@ static int ATTRIB_wprintf(const WCHAR *format, ...) { > } > > va_start(parms, format); > - len = vsnprintfW(output_bufW, MAX_WRITECONSOLE_SIZE/sizeof(WCHAR), format, parms); > + SetLastError(NO_ERROR); > + len = FormatMessageW(FORMAT_MESSAGE_FROM_STRING, format, 0, 0, output_bufW, > + MAX_WRITECONSOLE_SIZE/sizeof(*output_bufW), &parms); You have to use __ms_va_list with FormatMessage for it to work properly on 64-bit. -- Alexandre Julliard julliard(a)winehq.org

Oct. 20, 2011

Re: vbscript: Added Global_isNull implementation

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15023 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 20, 2011

Re: Anyone wants to ban spammer on forum

by Vitaliy Margolen

On 10/19/2011 09:49 AM, Jeremy Newman wrote: > On 10/19/2011 10:21 AM, Tijl Coosemans wrote: >> The past few days there's been spam messages every few minutes and as >> soon as you ban an account an other one appears. Obviously the bots have >> figured out the answer to the captcha. > > Alrighty then, time for me to change the question... Done! > > The new question is: > What operating system does Wine run applications from? > > Should be obvious for a human. Hope that won't upset users of other well known OS <g>. Any ETA on the new version of PHPBB? Vitaliy.

Oct. 19, 2011

Re: Regression testing breakthrough

by Hin-Tak Leung

I have two suggestions - - git clone has a "--depth" option which does a shallow clone (i.e. with some history removed). - you can use "git-archive" to export arbitrary commits out as a tar ball dynamically; there is no need to have store a tar ball permanently. That said, I am doubtful about the usefulness of this in a linux environment, because the system libraries (in the general sense - libjpeg, etc) varies between different linux distros. --- On Wed, 19/10/11, Joel Holdsworth <joel(a)airwebreathe.org.uk> wrote: Alternatively, have you considered doing a .tar.gz of every build snapshot, and placing that on a server somewhere? e.g. a folder full of 36def4af0ca85a1d0e66b5207056775bcb3b09ff.tar.gz files? Then one could write a simple wine regression bisect tool that implements similar semantics to git bisect, but would essentially wrap wget. Then in your server you could have an index file which is a list of the sha commit ids. This would save the user having to clone a 26Gb repository when most of the commits will be irrelevant. Extra bonus points for doing a better job of compressing the small deltas between binaries*, rather than compressing full wine builds. Joel * Are binaries deterministic like this? or do they tend to be completely scrambled? On 18 October 2011 at 09:45 Damjan Jovanovic <damjan.jov(a)gmail.com> wrote: > Hi > > Since the beginning, I've had issues with regression testing. Despite the > fact it's very useful, it takes forever, it's easy to make a mistake > (especially during "reverse regression testing"), users find it too long and > technical, and only a small minority of regressions are ever bisected. And > several patches need backporting to allow older versions of Wine to compile > and run on today's make, gcc, and libraries - this is the case even for the > 1.0.x releases from less than 3 years ago! > > The problem is of course compilation. "configure" takes at least 40 seconds, > without any way to speed it up on multi-core CPUs. "make" takes > 5 minutes, > and it's only taking longer as Wine gets bigger. Compilation is > fundamentally complex and technical to users. > > But what if we had precompiled binaries, and regression testing consisted of > just running different versions of Wine? > > Wine binaries take up about 122 MB and take over 5 minutes to compile. > There's now 35770 commits between 36def4af0ca85a1d0e66b5207056775bcb3b09ff > (Release 1.0) and "origin". That's about 4.4 terrabytes of storage and over > 4 months of compilation, if each of those versions had to be compiled and > installed into its own prefix, way beyond what most users are willing or > able to store or do. Most patches however end up affecting only a few binary > files in the end, and compiling successive versions allows "make" to be very > quick. > > So I've written a tool that compiles Wine and adds each commit's binaries > into a Git repository. It knows how to compile old versions of Wine > (currently as far back as 1.0). It knows that commits affecting only > ANNOUNCE, .gitignore, and files in dll/ or programs/ ending with .c and such > don't need to go through the endlessly slow "configure", only "make". It is > stateless: if interrupted, it can resume from the last successful commit. It > works around bugs in GNU make (you won't believe how many there are...). > > This tool compiled all 35000 or so commits from Wine 1.0 to around 4th > October 2011 in only 7 days, generating a Git repository of Wine binaries > that's only 26 gigabytes in size. Regression testing with binaries is a > pleasure: it takes only a few seconds :-) on each bisection. I bisected a 16 > step regression in just 20 minutes, and most of that time was spent running > the application and dealing with 2 X-server crashes. > > I haven't figured out how to make the binaries available to users. Few users > can clone a 26 gigabyte repository, and even fewer places can serve that > much to multiple users. Maybe Git can compress it further? The other idea I > had is that users should be able to regression test through a GUI tool. > Maybe the GUI tool can just download and run the +/- 122 MB binary snapshots > for specific commits, instead of having the entire binary repository > locally? > > Any other ideas? Would you like to see this tool? Can I send an attachment > with it? > > Thank you > Damjan Jovanovic

Oct. 19, 2011

Re: Bug 28791: ensure that dashes value is not 0 for PS_USERSTYLE linestyles

by Damian Dixon

Just removing the zeros will shift the pattern to the left potentially producing an effect that is not expected. I will take a look at what happens on Windows to see what needs to be replicated. On 19 October 2011 20:05, Alexandre Julliard <julliard(a)winehq.org> wrote: > damian dixon <damian.dixon(a)gmail.com> writes: > > > diff --git a/dlls/winex11.drv/pen.c b/dlls/winex11.drv/pen.c > > index b677515..71b1bea 100644 > > --- a/dlls/winex11.drv/pen.c > > +++ b/dlls/winex11.drv/pen.c > > @@ -108,7 +108,7 @@ HPEN X11DRV_SelectPen( PHYSDEV dev, HPEN hpen ) > > case PS_USERSTYLE: > > physDev->pen.dash_len = min(elp->elpNumEntries, > MAX_DASHLEN); > > for(i = 0; i < physDev->pen.dash_len ; i++) > > - physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], > 255); > > + physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], 255) > ? min(elp->elpStyleEntry[i], 255) : 1; > > Removing 0 entries (and merging the adjacent entries) would probably be > more correct. It could also use some test cases. > > -- > Alexandre Julliard > julliard(a)winehq.org >

Oct. 19, 2011

Re: Bug 28791: ensure that dashes value is not 0 for PS_USERSTYLE linestyles

by Alexandre Julliard

damian dixon <damian.dixon(a)gmail.com> writes: > diff --git a/dlls/winex11.drv/pen.c b/dlls/winex11.drv/pen.c > index b677515..71b1bea 100644 > --- a/dlls/winex11.drv/pen.c > +++ b/dlls/winex11.drv/pen.c > @@ -108,7 +108,7 @@ HPEN X11DRV_SelectPen( PHYSDEV dev, HPEN hpen ) > case PS_USERSTYLE: > physDev->pen.dash_len = min(elp->elpNumEntries, MAX_DASHLEN); > for(i = 0; i < physDev->pen.dash_len ; i++) > - physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], 255); > + physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], 255) ? min(elp->elpStyleEntry[i], 255) : 1; Removing 0 entries (and merging the adjacent entries) would probably be more correct. It could also use some test cases. -- Alexandre Julliard julliard(a)winehq.org

Oct. 19, 2011

Re: [PATCH 2/2] vbscript: Added Global_IsEmpty implementation

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15011 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 19, 2011

Re: Fix Bug 23124 with an "ln -s"

by Austin English

On Tue, Oct 18, 2011 at 17:25, Ken Sharp <kennybobs(a)o2.co.uk> wrote: > http://bugs.winehq.org/show_bug.cgi?id=23124 > > Could someone tell me if the attached patch would actually work? > > It does compile and works correctly with LANG=en_PH.utf-8 but I'm don't know > that: > > 1. Using an "ln -s" is acceptable, nor if it will work given some file > systems may be unable to use soft links. This will break the mingw/cygwin builds, as well as compiling on fat32 (already broken, but let's not make it worse..). You'll need to do this using autoconf/Makefile magic, $(LN_S), which will use `cp -p` on systems that don't support `ln -s`. -- -Austin

Oct. 19, 2011

Re: vbscript: Added Global_IsEmpty implementation

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15010 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 19, 2011

Re: vbscript: Added Global_IsEmpty implementation

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=15010 Your paranoid android. === WNT4WSSP6 (32 bit) === No test summary line found === W2KPROSP4 (32 bit) === No test summary line found === WXPPROSP3 (32 bit) === No test summary line found === W2K3R2SESP2 (32 bit) === No test summary line found === WVISTAADM (32 bit) === No test summary line found === W2K8SE (32 bit) === No test summary line found === W7PRO (32 bit) === No test summary line found === W7PROX64 (32 bit) === No test summary line found === TEST64_W7SP1 (32 bit) === No test summary line found === W7PROX64 (64 bit) === No test summary line found === TEST64_W7SP1 (64 bit) === No test summary line found

Oct. 19, 2011

Re: 80141: vbscript: Added Global_IsEmpty implementation

by Jacek Caban

On 10/19/11 7:11 PM, Dan Kegel wrote: > Hi Jacek, > build failed here... did you forget #include<assert.h> ? > Yes, it was added by another patch in my tree, thanks. Jacek

Oct. 19, 2011

Re: 80141: vbscript: Added Global_IsEmpty implementation

by Dan Kegel

Hi Jacek, build failed here... did you forget #include <assert.h> ? On Wed, Oct 19, 2011 at 9:56 AM, <buildbot(a)kegel.com> wrote: > This is an experimental automated build and test service. > Please feel free to ignore this email while we work the kinks out. > > For more info about this message, see http://wiki.winehq.org/BuildBot > > The Buildbot has detected a failed build on builder runtests-default while building Wine. > Full details are available at: http://buildbot.kegel.com/builders/runtests-default/builds/715 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) > BUILD FAILED: failed shell_2 > > Errors: > global.c: In function 'Global_IsEmpty': > global.c:281:5: error: implicit declaration of function 'assert' [-Werror=implicit-function-declaration] > make[1]: *** [global.o] Error 1 > make: *** [dlls/vbscript] Error 2 > make: *** Waiting for unfinished jobs.... > >

Oct. 19, 2011

Re: Anyone wants to ban spammer on forum

by Ken Sharp

On 19/10/11 16:49, Jeremy Newman wrote: > On 10/19/2011 10:21 AM, Tijl Coosemans wrote: >> The past few days there's been spam messages every few minutes and as >> soon as you ban an account an other one appears. Obviously the bots have >> figured out the answer to the captcha. > > Alrighty then, time for me to change the question... Done! > > The new question is: > What operating system does Wine run applications from? > > Should be obvious for a human. Including the human who wrote the bot? > > -N > > >

Oct. 19, 2011

Re: Anyone wants to ban spammer on forum

by Paul Romanyszyn

On 10/19/2011 11:21 AM, Tijl Coosemans wrote: > On Wednesday 19 October 2011 14:08:14 L. Rahyen wrote: >> On 2011-10-19 (October, Wednesday) 02:37:41 Vitaliy Margolen wrote: >>> On 10/18/2011 07:37 PM, Vitaliy Margolen wrote: >>>> So any admins actually watching and want to bad roberbdib3a on forum? >>>> >>>> Also why aren't every moderator has these rights to block spammers, >>>> since we have only one forum. >>> >>> So no takers? I'm guessing we need more forum admins then, so more >>> timezones can be covered. >> >> I agree. I and other morerators have deleted spam messages, but the >> spammer kept using the same account to post more. Inability to ban >> spammer's account is frustrating. I would happely volonteer to be >> forum admin so I can ban spammers. > > The past few days there's been spam messages every few minutes and as > soon as you ban an account an other one appears. Obviously the bots have > figured out the answer to the captcha. > > > > Any way to determine IP address and do something like failtoban and lock the IP our for a few hours. This should be a firewall function not forum software. Paul R.

Oct. 19, 2011

Re: Anyone wants to ban spammer on forum

by Jeremy Newman

On 10/19/2011 10:21 AM, Tijl Coosemans wrote: > The past few days there's been spam messages every few minutes and as > soon as you ban an account an other one appears. Obviously the bots have > figured out the answer to the captcha. Alrighty then, time for me to change the question... Done! The new question is: What operating system does Wine run applications from? Should be obvious for a human. -N

Oct. 19, 2011

Re: Anyone wants to ban spammer on forum

by Tijl Coosemans

On Wednesday 19 October 2011 14:08:14 L. Rahyen wrote: > On 2011-10-19 (October, Wednesday) 02:37:41 Vitaliy Margolen wrote: >> On 10/18/2011 07:37 PM, Vitaliy Margolen wrote: >>> So any admins actually watching and want to bad roberbdib3a on forum? >>> >>> Also why aren't every moderator has these rights to block spammers, >>> since we have only one forum. >> >> So no takers? I'm guessing we need more forum admins then, so more >> timezones can be covered. > > I agree. I and other morerators have deleted spam messages, but the > spammer kept using the same account to post more. Inability to ban > spammer's account is frustrating. I would happely volonteer to be > forum admin so I can ban spammers. The past few days there's been spam messages every few minutes and as soon as you ban an account an other one appears. Obviously the bots have figured out the answer to the captcha.

Oct. 19, 2011

Re: Regression testing breakthrough

by Marcus Meissner

On Wed, Oct 19, 2011 at 04:18:50PM +0200, Frédéric Delanoy wrote: > On Wed, Oct 19, 2011 at 15:50, Marcus Meissner <meissner(a)suse.de> wrote: > > On Wed, Oct 19, 2011 at 02:42:29PM +0100, Ken Sharp wrote: > >> > >> > >> On 19/10/11 13:43, Frédéric Delanoy wrote: > >> >On Wed, Oct 19, 2011 at 14:08, Joel Holdsworth<joel(a)airwebreathe.org.uk> wrote: > >> >>Alternatively, have you considered doing a .tar.gz of every build snapshot, > >> >>and placing that on a server somewhere? > >> >> > >> >>e.g. a folder full of 36def4af0ca85a1d0e66b5207056775bcb3b09ff.tar.gz files? > >> > > >> >tar.xz would compress better > >> > >> tar.lzma? > > > > Having tars of all builds would be way larger I guess. GIT compresses and > > shares objects that are the same. > > > > Ciao, Marcus > > You're talking about using a git tree just to store binaries for each > committed patch, I suppose? > But then you would have to download the whole repository (which can be > quite big) to get compression benefits, right? True, yes. Ciao, Marcus

Oct. 19, 2011

Re: 80135: [PATCH 05/10] gdi32: Implement SetPixel.

by Huw Davies

On Wed, Oct 19, 2011 at 07:04:02AM -0700, Dan Kegel wrote: > Sounds like your temporary todo_wine didn't quite do the trick...? Ah, I see. I was getting a failure on 111 and not 104, presumably due to different XServers. Anyway they all pass again after the entire series is applied. Huw.

Oct. 19, 2011

Re: Regression testing breakthrough

by Frédéric Delanoy

On Wed, Oct 19, 2011 at 15:50, Marcus Meissner <meissner(a)suse.de> wrote: > On Wed, Oct 19, 2011 at 02:42:29PM +0100, Ken Sharp wrote: >> >> >> On 19/10/11 13:43, Frédéric Delanoy wrote: >> >On Wed, Oct 19, 2011 at 14:08, Joel Holdsworth<joel(a)airwebreathe.org.uk> wrote: >> >>Alternatively, have you considered doing a .tar.gz of every build snapshot, >> >>and placing that on a server somewhere? >> >> >> >>e.g. a folder full of 36def4af0ca85a1d0e66b5207056775bcb3b09ff.tar.gz files? >> > >> >tar.xz would compress better >> >> tar.lzma? > > Having tars of all builds would be way larger I guess. GIT compresses and > shares objects that are the same. > > Ciao, Marcus You're talking about using a git tree just to store binaries for each committed patch, I suppose? But then you would have to download the whole repository (which can be quite big) to get compression benefits, right?

Oct. 19, 2011

Re: 80135: [PATCH 05/10] gdi32: Implement SetPixel.

by Dan Kegel

On Wed, Oct 19, 2011 at 6:59 AM, Huw Davies <huw(a)codeweavers.com> wrote: > On Wed, Oct 19, 2011 at 06:46:49AM -0700, Dan Kegel wrote: >> Patches 5-7 failed on WINEDEBUG=warn+heap here: >> palette.c:104: Test failed: getColor=00280000 >> Might be flaky; patch 8 passed. patches 9 and 10 aren't done testing yet. > > It's because I wanted to move SetPixel and GetPixel in separate > patches. I added a temporary todo_wine in patch 5 (SetPixel) to > prevent this test failure, which then got removed in patch 8 > (GetPixel). Sounds like your temporary todo_wine didn't quite do the trick...?

Oct. 19, 2011