Re: [PATCH] advapi32: close scm_handle on error exit (Coverity)

2013-01-07 Thread 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=23785 Your paranoid android

Re: [PATCH] ddraw: Don't discard buffers that haven't been used in draws (try 2)

2013-01-07 Thread 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=23786 Your paranoid android

Re: [PATCH] dplayx: check return of CoCreateInstance (Coverity)

2013-01-07 Thread 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=23787 Your paranoid android

Re: [PATCH] dplayx: check return of CoCreateInstance (Coverity)

2013-01-07 Thread 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=23787 Your paranoid android

Re: [PATCH 2/7] d3dx9: Always set the table = NULL in D3DXGetShaderConstantTableEx(). (resend)

2013-01-07 Thread 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=23782 Your paranoid android

Re: [PATCH 3/7] d3dx9: Handle invalid byte code in D3DXGetShaderConstantTableEx(). (resend)

2013-01-07 Thread 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=23783 Your paranoid android

Re: [PATCH 7/7] d3dcompiler43/tests: Don't use IUnknown_Release(). (resend)

2013-01-07 Thread 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=23784 Your paranoid android

Re: windowscodecs: Don't copy extra data from the end of a row.

2013-01-07 Thread 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=23777 Your paranoid android

Re: [PATCH 3/6] loader: On Mac, embed Info.plist in (__TEXT, __info_plist) section. [try 2]

2013-01-07 Thread Alexandre Julliard
Ken Thomases writes: > @@ -731,11 +731,13 @@ case $host_os in > AC_SUBST(IOKITLIB,"-framework IOKit -framework CoreFoundation") > AC_SUBST(APPLICATIONSERVICESLIB,"-framework ApplicationServices") > AC_SUBST(CORESERVICESLIB,"-framework CoreServices") > +AC_SUBST(WINE_INFO_PLIST,

Re: [PATCH 3/6] loader: On Mac, embed Info.plist in (__TEXT, __info_plist) section.

2013-01-07 Thread Vincent Povirk
On Mon, Jan 7, 2013 at 11:24 AM, Charles Davis wrote: > One area that might still be a problem, though, is if we associate *.exe > files with Wine itself: then when a user double-clicks on a Win32 executable, > LS will go straight to executing Wine. You shouldn't be doing that anyway. Any assoc

Re: wininet: Fixed handling NULL buffer in INTERNET_OPTION_PROXY queries

2013-01-07 Thread 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=23776 Your paranoid android

Re: [PATCH 3/6] loader: On Mac, embed Info.plist in (__TEXT, __info_plist) section.

2013-01-07 Thread Charles Davis
On Jan 7, 2013, at 9:15 AM, Ken Thomases wrote: > On Jan 7, 2013, at 1:23 AM, Per Johansson wrote: > >> On Mon, Jan 7, 2013 at 4:48 AM, Charles Davis wrote: >>> >>> With this, you won't be able to launch Wine from the Finder or with open(1) >>> prior to 10.6. >> >> I don't see how you'd do u

Re: [PATCH 3/6] loader: On Mac, embed Info.plist in (__TEXT, __info_plist) section.

2013-01-07 Thread Ken Thomases
On Jan 7, 2013, at 1:23 AM, Per Johansson wrote: > On Mon, Jan 7, 2013 at 4:48 AM, Charles Davis wrote: >> >> With this, you won't be able to launch Wine from the Finder or with open(1) >> prior to 10.6. > > I don't see how you'd do use open or the Finder to launch wine, could > you explain mo

Re: [PATCH 2/3] mshtml: Added IHTMLStyle::put_pixelLeft implementation

2013-01-07 Thread 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=23774 Your paranoid android

Re: [PATCH 1/3] mshtml: Added IHTMLStyle::get_pixelLeft implementation

2013-01-07 Thread 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=23773 Your paranoid android

Re: [1/3] xmllite: Implement PI parsing

2013-01-07 Thread 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=23770 Your paranoid android

Re: [2/3] xmllite: Support streams starting with comments, simplify tests

2013-01-07 Thread 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=23771 Your paranoid android

Re: [3/3] xmllite: Support name reporting for comments and PIs

2013-01-07 Thread 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=23772 Your paranoid android

Re: [1/3] xmllite: Implement PI parsing

2013-01-07 Thread Nikolay Sivov
On 1/7/2013 13:30, Marvin 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.

Re: [PATCH] kernel32: improve condition variable testcase

2013-01-07 Thread 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=23768 Your paranoid android

Re: [PATCH] kernel32: implement condition variables

2013-01-07 Thread 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=23769 Your paranoid android