January 2012 - Wine-devel - list.winehq.org

Re: kernel32: add stubs for GetConsoleCommandHistoryLengthA/W
by André Hentschel 16 Jan '12

16 Jan '12

Am 16.01.2012 18:56, schrieb Austin English: > http://bugs.winehq.org/show_bug.cgi?id=29620 just want to mention source.winehq.org/git/wine.git/blob?h=fdc45524c8b6c70070a03b9efdde4230114f1… here, did you noticed that? -- Best Regards, André Hentschel

1 0

Long translation patch
by Gustavo Henrique Milaré 16 Jan '12

16 Jan '12

Hello, Long time no see :D I've been updating the translation of Wine to Brazilian Portuguese for the last couple of days. I have checked and made extensive tests (I opened the programs, used XN Resource Editor on the resources of the dlls and made a fast review of the diff file). It turns out the patch is very large (around 6500 lines). Sorry if this is a lame question, but I guess I'll need to break it in 15 patches or so, right? Should I send all patches in the same day? I don't want to give the patch reviewers a headache :) Cheers, Gustavo

5 5

RFC: wined3d: Let GetRasterStatus return D3D_OK, again.
by Akihiro Sagawa 16 Jan '12

16 Jan '12

Hi, Attached patch lets GetRasterStatus in wined3d return D3D_OK with pseudo-raster status by QueryPerformanceCounter. Could someone please try the patch with retail version of StarCraft 2? This patch fixes bug 26917[1]. But I'm afraid another problem, especially for SC2. Because the current behavior was introduced to fix bug 23556[2] related SC2. I've already tried for SC2 current demo version. It looks fine. However, I couldn't find GetRasterStatus call in trace logs. Therefore, I'd like to test with retail version which I don't have. I also welcome comments or problem reports about my patch. [1] http://bugs.winehq.org/show_bug.cgi?id=26917 [2] http://bugs.winehq.org/show_bug.cgi?id=23556 -- Akihiro Sagawa

2 1

Re: [PATCH 1/2] winmm: Only start the devices thread when necessary
by Alexandre Julliard 16 Jan '12

16 Jan '12

Andrew Eikum <aeikum(a)codeweavers.com> writes: > Fixes bug 28042. > > dlls/winmm/waveform.c | 106 ++++++++++++++++++++++++++++++++++++------------- > 1 files changed, 78 insertions(+), 28 deletions(-) It doesn't work here: ../../../tools/runtest -q -P wine -M quartz.dll -T ../../.. -p quartz_test.exe.so filtergraph.c && touch filtergraph.ok wine: Unhandled page fault on read access to 0x69187f74 at address 0x558812f9 (thread 0009), starting debugger... Unhandled exception: page fault on read access to 0x69187f74 in 32-bit code (0x558812f9). Register dump: CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b EIP:558812f9 ESP:0032e840 EBP:0032f778 EFLAGS:00010206( R- -- I - -P- ) EAX:0012ff90 EBX:5588f200 ECX:69187f60 EDX:0012fe30 ESI:00000000 EDI:0032ecdc Stack dump: 0x0032e840: 0012ff90 5588816c 0032f504 0032f754 0x0032e850: 55887f4c 0032f504 0032f6e0 00000000 0x0032e860: 00000000 00000000 00000000 00000000 0x0032e870: 0012fda8 0012fe30 0032e908 68352582 0x0032e880: 000000a0 00000000 6834ed1d 683b710c 0x0032e890: 0032eb18 00000000 0032e978 0032f834 Backtrace: =>0 0x558812f9 DEVENUM_ICreateDevEnum_CreateClassEnumerator+0x10c9(iface=0x55891e4c, clsidDeviceClass=0x32f834, ppEnumMoniker=0x32f870, dwFlags=0) [/home/julliard/wine/wine/dlls/devenum/createdevenum.c:727] in devenum (0x0032f778) 1 0x69104bb2 (0x0032f898) 2 0x69101229 (0x0032faa8) 3 0x6876955a test_render_filter_priority+0x1589() [/home/julliard/wine/wine/dlls/quartz/tests/filtergraph.c:1846] in quartz_test (0x0032fcd8) 4 0x68769fa3 func_filtergraph+0x562() [/home/julliard/wine/wine/dlls/quartz/tests/filtergraph.c:1895] in quartz_test (0x0032fd58) 5 0x68763392 main+0x3a1(argc=*** Invalid address 0x69187f60 *** , argv=*** Invalid address 0x69187f64 *** Internal symbol error: unable to access memory location 0x69187f64) [/home/julliard/wine/wine/dlls/quartz/tests/../../../include/wine/test.h:556] in quartz_test (0x0032fe28) 6 0x6876f865 __wine_spec_exe_entry+0x74(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/winecrt0/exe_entry.c:36] in quartz_test (0x0032fe70) 7 0x684895bc call_process_entry+0xb() in kernel32 (0x0032fe88) 8 0x6848a73f start_process+0x5e(peb=0x7ffdf000) [/home/julliard/wine/wine/dlls/kernel32/process.c:1084] in kernel32 (0x0032fec8) 9 0x68376de0 call_thread_func_wrapper+0xb() in ntdll (0x0032fed8) 10 0x68379acd call_thread_func+0x7c(entry=0x6848a6e0, arg=0x7ffdf000, frame=0x32ffc8) [/home/julliard/wine/wine/dlls/ntdll/signal_i386.c:2532] in ntdll (0x0032ffa8) 11 0x68376dbe call_thread_entry_point+0x11() in ntdll (0x0032ffc8) 12 0x6834eece start_process+0x1d(kernel_start=0x6848a6e0) [/home/julliard/wine/wine/dlls/ntdll/loader.c:2612] in ntdll (0x0032ffe8) 13 0x68028a9d wine_call_on_stack+0x1c() in libwine.so.1 (0x00000000) 0x558812f9 DEVENUM_ICreateDevEnum_CreateClassEnumerator+0x10c9 [/home/julliard/wine/wine/dlls/devenum/createdevenum.c:727] in devenum: call *0x14(%ecx) 727 res = IFilterMapper2_RegisterFilter(pMapper, -- Alexandre Julliard julliard(a)winehq.org

1 0

Wine 1.3.37 Gecko engine fails on Arch Linux x86
by Saulius Krasuckas 15 Jan '12

15 Jan '12

I decided to run latest Winetest on my linux box and chose to install Gecko package during prefix setup: $ rm -rf ~/.wine $ wine start wine: created the configuration directory '/home/s2/.wine' fixme:urlmon:DownloadBSC_OnProgress Unsupported status 3 fixme:wininet:InternetLockRequestFile STUB fixme:storage:create_storagefile Storage share mode not implemented. err:mscoree:LoadLibraryShim error reading registry key for installroot err:mscoree:LoadLibraryShim error reading registry key for installroot err:mscoree:LoadLibraryShim error reading registry key for installroot err:mscoree:LoadLibraryShim error reading registry key for installroot err:mscoree:LoadLibraryShim error reading registry key for installroot err:mscoree:LoadLibraryShim error reading registry key for installroot err:module:import_dll Loading library ssl3.dll (which is needed by L"C:\\windows\\system32\\gecko\\1.4\\wine_gecko\\xul.dll") failed (error c0000020). Could not load wine-gecko. HTML rendering will be disabled. wine: configuration in '/home/s2/.wine' has been updated. Another try produced a crash: $ rm -rf ~/.wine $ wine start wine: created the configuration directory '/home/s2/.wine' fixme:urlmon:DownloadBSC_OnProgress Unsupported status 3 fixme:wininet:InternetLockRequestFile STUB wine: Unhandled page fault on read access to 0x149cb074 at address 0x7def0a3a (thread 0018), starting debugger... err:seh:raise_exception Unhandled exception code c0000005 flags 0 addr 0x7bc49a59 Could not load wine-gecko. HTML rendering will be disabled. wine: configuration in '/home/s2/.wine' has been updated. Should I report a bug? S.

4 5

Re: [4/5] d3drm: Add IDirect3DRMWinDevice interface
by Nikolay Sivov 15 Jan '12

15 Jan '12

On 1/15/2012 19:04, André Hentschel wrote: > --- > dlls/d3drm/device.c | 165 +++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 files changed, 165 insertions(+), 0 deletions(-) > > diff --git a/dlls/d3drm/device.c b/dlls/d3drm/device.c > index f02ce33..957b465 100644 > --- a/dlls/d3drm/device.c > +++ b/dlls/d3drm/device.c > @@ -29,17 +29,20 @@ > #include "wingdi.h" > > #include "d3drm_private.h" > +#include "d3drmwin.h" > > WINE_DEFAULT_DEBUG_CHANNEL(d3drm); > > typedef struct { > IDirect3DRMDevice2 IDirect3DRMDevice2_iface; > IDirect3DRMDevice3 IDirect3DRMDevice3_iface; > + IDirect3DRMWinDevice IDirect3DRMWinDevice_iface; > LONG ref; > } IDirect3DRMDeviceImpl; > > static const struct IDirect3DRMDevice2Vtbl Direct3DRMDevice2_Vtbl; > static const struct IDirect3DRMDevice3Vtbl Direct3DRMDevice3_Vtbl; > +static const struct IDirect3DRMWinDeviceVtbl Direct3DRMWinDevice_Vtbl; You can move instance creation call after all vtables initialization, so you won't need these forwards. > > static inline IDirect3DRMDeviceImpl *impl_from_IDirect3DRMDevice2(IDirect3DRMDevice2 *iface) > { > @@ -51,6 +54,11 @@ static inline IDirect3DRMDeviceImpl *impl_from_IDirect3DRMDevice3(IDirect3DRMDev > return CONTAINING_RECORD(iface, IDirect3DRMDeviceImpl, IDirect3DRMDevice3_iface); > } > > +static inline IDirect3DRMDeviceImpl *impl_from_IDirect3DRMWinDevice(IDirect3DRMWinDevice *iface) > +{ > + return CONTAINING_RECORD(iface, IDirect3DRMDeviceImpl, IDirect3DRMWinDevice_iface); > +} > + > HRESULT Direct3DRMDevice_create(REFIID riid, IUnknown** ppObj) > { > IDirect3DRMDeviceImpl* object; > @@ -66,6 +74,7 @@ HRESULT Direct3DRMDevice_create(REFIID riid, IUnknown** ppObj) > > object->IDirect3DRMDevice2_iface.lpVtbl =&Direct3DRMDevice2_Vtbl; > object->IDirect3DRMDevice3_iface.lpVtbl =&Direct3DRMDevice3_Vtbl; > + object->IDirect3DRMWinDevice_iface.lpVtbl =&Direct3DRMWinDevice_Vtbl; > object->ref = 1; > > if (IsEqualGUID(riid,&IID_IDirect3DRMFrame3)) > @@ -96,6 +105,10 @@ static HRESULT WINAPI IDirect3DRMDevice2Impl_QueryInterface(IDirect3DRMDevice2* > { > *object =&This->IDirect3DRMDevice3_iface; > } > + else if(IsEqualGUID(riid,&IID_IDirect3DRMWinDevice)) > + { > + *object =&This->IDirect3DRMWinDevice_iface; > + } > else > { > FIXME("interface %s not implemented\n", debugstr_guid(riid)); > @@ -1021,3 +1034,155 @@ static const struct IDirect3DRMDevice3Vtbl Direct3DRMDevice3_Vtbl = > IDirect3DRMDevice3Impl_GetStateChangeOptions, > IDirect3DRMDevice3Impl_SetStateChangeOptions > }; > + > + > +/*** IUnknown methods ***/ > +static HRESULT WINAPI IDirect3DRMWinDeviceImpl_QueryInterface(IDirect3DRMWinDevice* iface, > + REFIID riid, void** object) > +{ > + IDirect3DRMDeviceImpl *This = impl_from_IDirect3DRMWinDevice(iface); > + return IDirect3DRMDevice2_QueryInterface(&This->IDirect3DRMDevice2_iface, riid, object); > +} > + > +static ULONG WINAPI IDirect3DRMWinDeviceImpl_AddRef(IDirect3DRMWinDevice* iface) > +{ > + IDirect3DRMDeviceImpl *This = impl_from_IDirect3DRMWinDevice(iface); > + > + TRACE("(%p)\n", This); > + > + return InterlockedIncrement(&This->ref); > +} > + > +static ULONG WINAPI IDirect3DRMWinDeviceImpl_Release(IDirect3DRMWinDevice* iface) > +{ > + IDirect3DRMDeviceImpl *This = impl_from_IDirect3DRMWinDevice(iface); > + ULONG ref = InterlockedDecrement(&This->ref); > + > + TRACE("(%p)\n", This); > + > + if (!ref) > + HeapFree(GetProcessHeap(), 0, This); > + > + return ref; > +} AddRef and Release should be forwards as well.

1 0

Re: [PATCH 1/2] winecoreaudio: Fix AudioCaptureClient Get/ReleaseBuffer protocol.
by Francois Gouget 14 Jan '12

14 Jan '12

On Fri, 13 Jan 2012, Joerg-Cyril.Hoehle(a)t-systems.com wrote: [...] > - Francois' machine crashes in AudioCaptureClient_ReleaseBuffer > http://test.winehq.org/data/5eecdc80e14d2c6cf74a64ac5423dd6ce7c37a3a/mac_fg… > Mine doesn't, even without my patches. Just in case this helps understand what's going on, this Mac Mini is connected to my amplifier via an optical link. I know one consequence is that I cannot set the volume on the Mac. Maybe it has consequences on sound capture to? -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ Computers are like airconditioners They stop working properly if you open WINDOWS

1 0

Re: [3/3] user32: Move the server call up in order to perform the thread input check earlier in SetActiveWindow.
by Alexandre Julliard 14 Jan '12

14 Jan '12

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > SetForegroundWindow already does a similar thing. Yes but that's because it doesn't have to call a hook first. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: msvcp90: start rfind from given position
by Marvin 13 Jan '12

13 Jan '12

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=16431 Your paranoid android. === WVISTAADM (32 bit string) === Failure running script in VM: The specified guest user must be logged in interactively to perform this operation

3 2

Named pipe message mode
by Adam Martinson 13 Jan '12

13 Jan '12

Need some extra pairs of eyes on these; any feedback would be helpful. ~Adam

1 0