September 2013 - Wine-Devel

Re: rsaenh: Add support for retrieving the security descriptor on key containers.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2276 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 4/4] mshtml: Added external MIME handling tests.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2275 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: add ualapi directory (try 2)
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2273 Your paranoid android. === build (build) === Missing build status line The build timed out

1 0

Re: ws2_32/tests: Mark WinNT result as broken
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2271 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 5/5] d3d8/tests: Test invalid volume lock boxes.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2270 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 4/5] d3d8/tests: Add tests for block-based volume formats.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2269 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 1/5] d3d9/tests: Read the caps from the device.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2268 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: shlwapi/tests: Fix stream test failures under some win2000 versions.
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2266 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: (resend)[3/4] imm32: use thread data from target HWND
by Marvin Sept. 22, 2013

Sept. 22, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2263 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: (resend)[3/4] imm32: use thread data from target HWND
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2262 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 2/2] d3dx9/tests: Add matrix column register count clamp test.
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2259 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [4/4] imm32: Prevent assocating cross thread default contexts
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2257 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [3/4] imm32: use thread data from target HWND
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2256 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: wininet: Don't assume that end of chunk means end of stream. (try 2)
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2255 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 5/5] d3d9/tests: Test invalid volume lock boxes.
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2254 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [PATCH 4/5] d3d9/tests: Add tests for block-based volume formats (try 2).
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2253 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [4/4] kernel32: Remove a 0-length read optimization from ReadFile. Resend.
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2252 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: [4/4] kernel32: Remove a 0-length read optimization from ReadFile. Resend.
by Marvin Sept. 21, 2013

Sept. 21, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2251 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: winex11: Don't trace a garbage value or read past end of caller's array in X11DRV_wglChoosePixelFormatARB().
by Roderick Colenbrander Sept. 20, 2013

Sept. 20, 2013

That change looks suspicious. If switching to i+1 fixes a problem, the real problem is that the if-statement in question gets entered in the first place. Some bounds checking on the outer for-loop (the one which loops through nCfgs and nMaxFormats), is probably wrong then. I had issues with this part in the past, so there maybe a small typo (it may just be off by 1). Roderick On Thu, Sep 19, 2013 at 6:37 PM, Ken Thomases <ken(a)codeweavers.com> wrote: > --- > dlls/winex11.drv/opengl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > >

2 1

Re: D3D command stream patches for testing
by Forest Sept. 20, 2013

Sept. 20, 2013

>In the past months I have been working on a command stream / worker >thread for wined3d. It moves most OpenGL calls into a separate thread >to improve performance (bug 11674) and fix some bugs that are >otherwise hard to fix (24684). Thank you, Stefan! I applied your patches to the Ubuntu PPA that I maintain for Guild Wars 2 players: https://launchpad.net/~foresto/+archive/winepatched/ I'm not seeing a spectacular difference with the CSMT patches, but I am seeing a difference. I ran some quick tests in Guild Wars 2 while standing still outside the Lion's Arch bank, looking down at the forge with a moderate number of players milling about and fireworks being launched. Pierre Franco said (earlier in this discussion) that he gets better results when running with the -dx9single command line switch, so I tested with and without it. Note that I play with the WINEDEBUG=-all environment variable. Here are my results: CSMT="disabled", running with -dx9single: 15-16 fps CSMT="enabled" running without -dx9single: 19-21 fps CSMT="enabled", running with -dx9single: 22-23 fps So, almost 50% improvement for me, which is not what I remember when running natively under Windows, but is still a very welcome improvement. I wonder if the 80%+ gains reported in other games are not to be seen in Guild Wars 2, or if they're heavily dependent on CPU, GPU, graphics driver, and screen resolution. For the record, I'm using an Intel i5-3570K (stock speeds), GeForce GTX 660Ti (factory overclocked), nVidia driver 313.30, and a 1920x1200 display. I'm running Xubuntu 64-bit. My in-game options are: Resolution: Full Screen - 1920x1200 Refresh Rate: Default Frame Limiter: Unlimited Interface Size: Normal Animation: High Anti-Aliasing: FXAA Environment: High LOD Distance: High Reflections: Terrain & Sky Textures: Medium Render Sampling: Native Shadows: Medium Shaders: Medium Post-Processing: High Character Model Limit: High Character Model Quality: High + Best Texture Filtering + Depth Blur + Effect LOD + High-Res Character Textures - Vertical Sync

3 4

Re: [PATCH 2/6] wined3d: Add support for converted volumes.
by Henri Verbeet Sept. 20, 2013

Sept. 20, 2013

On 20 September 2013 13:04, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > +static BOOL wined3d_volume_can_evict(const struct wined3d_volume *volume) > +{ > + if (volume->resource.pool != WINED3D_POOL_MANAGED) > + return FALSE; > + if (volume->download_count < 10) > + return FALSE; That doesn't look quite right.

1 0

Re: [PATCH 6/6] wined3d: Use GL_APPLE_client_storage for volumes if available.
by Henri Verbeet Sept. 20, 2013

Sept. 20, 2013

On 20 September 2013 13:04, Stefan Dösinger <stefan(a)codeweavers.com> wrote: > - volume->flags &= ~(WINED3D_VFLAG_ALLOCATED | WINED3D_VFLAG_SRGB_ALLOCATED); > + volume->flags &= ~(WINED3D_VFLAG_ALLOCATED | WINED3D_VFLAG_SRGB_ALLOCATED > + | WINED3D_VFLAG_CLIENT_STORAGE); > Indentation looks a bit off here.

1 0

Re: (resend)[3/4] imm32: use thread data from target HWND
by Alexandre Julliard Sept. 20, 2013

Sept. 20, 2013

Aric Stewart <aric(a)codeweavers.com> writes: > @@ -507,7 +524,7 @@ HIMC WINAPI ImmAssociateContext(HWND hWnd, HIMC hIMC) > LeaveCriticalSection(&threaddata_cs); > defaultContext = ImmCreateContext(); > ((InputContextData*)defaultContext)->threadDefault = TRUE; > - thread_data = IMM_GetThreadData(0); > + thread_data = IMM_GetThreadDataForWindow(hWnd); > thread_data->defaultContext = defaultContext; > } You have multiple race conditions here. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [1/2] oledb32: Correct DataLinks interface for win64
by Alexandre Julliard Sept. 20, 2013

Sept. 20, 2013

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > @@ -26,6 +26,20 @@ import "oledb.idl"; > #define vi_progid(str) > #endif > > +#ifdef _WIN64 > +cpp_quote("#ifdef _WIN64") > + typedef LONGLONG COMPATIBLE_LONG; > +cpp_quote("#else") > +cpp_quote(" typedef LONG COMPATIBLE_LONG;") > +cpp_quote("#endif") > +#else > +cpp_quote("#ifdef _WIN64") > +cpp_quote(" typedef LONGLONG COMPATIBLE_LONG;") > +cpp_quote("#else") > + typedef LONG COMPATIBLE_LONG; > +cpp_quote("#endif") > +#endif You can't do that, _WIN64 is not meaningful in idl. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 2/2] winemac: Implement support for global hot keys.
by Vincent Povirk Sept. 20, 2013

Sept. 20, 2013

I don't think this is going to work correctly when replacing hotkeys (by passing an already-registered hwnd and id to RegisterHotKey with a different key combination). Since the user driver's RegisterHotKey function will be called with the new id/vkey/flags before UnregisterHotKey is called for the existing hotkey's id/vkey/flags, the user driver has to distinguish them based on vkey/flags. I think you can still get away with a mapping with hwnd/id as keys, as long as you check for matching vkey/flags when you unregister a hotkey. (Of course, I'm not convinced the user driver should ever post WM_HOTKEY, since the input should go through low-level keyboard hooks first, which is why I did not include the id as an argument to these entry points. If we decided to pass the id through, there would technically be no need for RegisterHotKey to call the user driver's UnregisterHotKey in this case.)

1 0