July 2010 - Wine-devel - list.winehq.org

Re: [PATCH 1/4] ddraw: Use a less offensive handle table implementation for materials.
by Alexandre Julliard 28 Jul '10

28 Jul '10

Henri Verbeet <hverbeet(a)codeweavers.com> writes: > IDirect3DDeviceImpl_CreateHandle() is particular is just awful. > --- > dlls/ddraw/ddraw_private.h | 30 ++++++++++++- > dlls/ddraw/device.c | 63 ++++++++++++------------- > dlls/ddraw/direct3d.c | 10 ++++ > dlls/ddraw/executebuffer.c | 26 ++++------ > dlls/ddraw/main.c | 109 ++++++++++++++++++++++++++++++++++++++++++++ > dlls/ddraw/material.c | 13 ++--- > dlls/ddraw/viewport.c | 34 ++++++-------- > 7 files changed, 211 insertions(+), 74 deletions(-) Not sure it's really less offensive: ../../../tools/runtest -q -P wine -M ddraw.dll -T ../../.. -p ddraw_test.exe.so visual.c && touch visual.ok visual.c:1770: Tests skipped: device has no P8 texture support, skipping test visual.c:1952: Test failed: IDirect3DMaterial_GetHandle failed: 80070057 visual.c:1989: Test failed: Got color 00000000, expected 00ff0000 visual.c:1997: Test failed: Got color 00000000, expected 000000ff wine: Unhandled page fault on write access to 0x00000083 at address 0x686daffd (thread 0024), starting debugger... Unhandled exception: page fault on write access to 0x00000083 in 32-bit code (0x686daffd). Register dump: CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b EIP:686daffd ESP:0032fa60 EBP:0032fae8 EFLAGS:00010206( R- -- I - -P- ) EAX:00000059 EBX:6870b914 ECX:00149058 EDX:0000001b ESI:000004b0 EDI:00000097 Stack dump: 0x0032fa60: 00000000 6870c5e8 686fbe22 686f95a0 0x0032fa70: 00000097 0000001b 00020808 00000000 0x0032fa80: 00000000 3f800000 00000000 00149058 0x0032fa90: 0000001b 686f954c 6870c70c 686f95a0 0x0032faa0: 686f95d0 6870c5e8 686fbe22 00000000 0x0032fab0: 001329e8 00000000 00000000 00000000 Backtrace: =>0 0x686daffd IDirect3DDeviceImpl_7_Release+0x25d(iface=0x149058) [/home/julliard/wine/wine/dlls/ddraw/device.c:422] in ddraw (0x0032fb28) 1 0x686cfd60 Thunk_IDirect3DDeviceImpl_1_Release+0x5f(iface=0x149064) [/home/julliard/wine/wine/dlls/ddraw/device.c:474] in ddraw (0x0032fd38) 2 0x686a29fd func_visual+0x155c() [/home/julliard/wine/wine/dlls/ddraw/tests/visual.c:1099] in ddraw_test (0x0032fd88) 3 0x686a2dee run_test+0x15d(name=<is not available>) [/home/julliard/wine/wine/dlls/ddraw/tests/../../../include/wine/test.h:556] in ddraw_test (0x0032fe48) 4 0x686a2fe7 main+0x156(argc=<couldn't compute location>, argv=<couldn't compute location>) [/home/julliard/wine/wine/dlls/ddraw/tests/../../../include/wine/test.h:624] in ddraw_test (0x0032fe90) 5 0x686a3b6c __wine_spec_exe_entry+0x7b(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in ddraw_test (0x0032fea8) 6 0x7b85521c call_process_entry+0xb() in kernel32 (0x0032fee8) 7 0x7b85766b start_process+0x5a(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/kernel32/process.c:996] in kernel32 (0x0032fef8) 8 0x7bc72370 call_thread_func+0xb() in ntdll (0x0032ffc8) 9 0x7bc72540 call_thread_entry_point+0x6f(entry=0x7b857610, arg=0x7ffdf000) [/home/julliard/wine/wine/dlls/ntdll/signal_i386.c:2473] in ntdll (0x0032ffe8) 10 0x7bc4d8ca start_process+0x29(kernel_start=0x7b857610) [/home/julliard/wine/wine/dlls/ntdll/loader.c:2612] in ntdll (0x00000000) 0x686daffd IDirect3DDeviceImpl_7_Release+0x25d [/home/julliard/wine/wine/dlls/ddraw/device.c:422] in ddraw: movl $0x0,0x68(%edx) 422 m->Handle = 0; -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH] comdlg32/tests: Use GetOpenFileNameA instead of GetOpenFileNameW
by testbot＠testbot.winehq.org 28 Jul '10

28 Jul '10

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=3990 Your paranoid android. === W7PROX64 (32 bit filedlg) === No test summary line found

1 0

Re: [3/5] urlmon: Implemented canonicalization for ports in URIs
by Alexandre Julliard 28 Jul '10

28 Jul '10

Thomas Mullaly <thomas.mullaly(a)gmail.com> writes: > --- > dlls/urlmon/uri.c | 88 > +++++++++++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 86 insertions(+), 2 deletions(-) It doesn't work here: ../../../tools/runtest -q -P wine -M urlmon.dll -T ../../.. -p urlmon_test.exe.so uri.c && touch uri.ok uri.c:2788: Test failed: Error: Expected a length of 0 but got -2 on uri_tests[13].str_props[6]. make: *** [uri.ok] Error 1 -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 5/11] quartz: Add MediaSeekingPassThru to video renderer
by Alexandre Julliard 28 Jul '10

28 Jul '10

Maarten Lankhorst <m.b.lankhorst(a)gmail.com> writes: > From 42723d1e254d1fa5b6a64a56d895e05c035780bc Mon Sep 17 00:00:00 2001 > From: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> > Date: Sat, 22 May 2010 15:58:15 +0200 > Subject: [PATCH 13/32] quartz: Add MediaSeekingPassThru to video renderer It doesn't work here: ../../../tools/runtest -q -P wine -M quartz.dll -T ../../.. -p quartz_test.exe.so filtergraph.c && touch filtergraph.ok filtergraph.c:276: Test failed: SetPositions failed: 80040209 filtergraph.c:278: Test failed: SetPositions failed: 80040209 make: *** [filtergraph.ok] Error 2 -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: comctl32/treeview: Fix a crash in TVS_SINGELEXPAND helper (Coverity)
by testbot＠testbot.winehq.org 28 Jul '10

28 Jul '10

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=3980 Your paranoid android. === WNT4WSSP6 (32 bit treeview) === treeview.c:896: Test failed: got wrong state 60

1 0

Re: [1/2] d3dx9_36: Handle parent_data parameter of D3DXInclude.Open() function.
by Henri Verbeet 28 Jul '10

28 Jul '10

On 27 July 2010 23:32, Matteo Bruni <matteo.mystral(a)gmail.com> wrote: > + if(parent_include == NULL) > + ERR("Parent include file missing\n"); If that's an internal error, shouldn't the call fail? Similar for the wpp_open_mem() changes.

1 0

Re: [2/2] d3dx9: Implement D3DXAssembleShaderFromFile.
by Henri Verbeet 28 Jul '10

28 Jul '10

On 27 July 2010 23:32, Matteo Bruni <matteo.mystral(a)gmail.com> wrote: > + if(parent_data != NULL) > + parent_name = *((char **)parent_data - 1); That casts away const. The ID3DXInclude interface itself looks somewhat broken to me, but I don't think you need to modify the parent name here. Please use "const char *" and "const void *" or "LPCSTR" and "LPCVOID".

1 0

Re: ddraw: Added a todo_wine test for SetCooperativeLevel
by Stefan Dösinger 28 Jul '10

28 Jul '10

Am 28.07.2010 um 00:39 schrieb Johan Gill: > > <new_ddraw_test.txt> The test seems OK to me. Not sure why patchbot is complaining, but it is running in virtual machines, so I guess we can blame its ddraw setup.

1 0

Re: ddraw: Added a todo_wine test for SetCooperativeLevel
by testbot＠testbot.winehq.org 27 Jul '10

27 Jul '10

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=3977 Your paranoid android. === W98SE (32 bit ddrawmodes) === No test summary line found === W7PROX64 (64 bit ddrawmodes) === Timeout

1 0

Re: ddraw: Ensure fullscreen size of fullscreen windows
by Stefan Dösinger 27 Jul '10

27 Jul '10

Am 27.07.2010 um 00:19 schrieb Johan Gill: > This patch fixes the long-standing regression bug 17215. > > Enjoy, and provide feedback if you don't ;) > <17215.txt> Usually wined3d does the window setup. I recommend to take a look at WineD3D's AcquireFocusWindow and SetDisplayMode The problem is somewhat more complicated though - there's swapchain_setup_fullscreen_window and swapchain_restore_fullscreen_window, but wined3ddevice::Setdisplaymode doesn't call the swapchain(because there may not be one if ddraw hasn't created the primary surface yet).

2 2