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

WINMM_OpenDevice Activate failed
by GOUJON Alexandre Oct. 6, 2011

Oct. 6, 2011

Hi Andrew, I saw you modified the Audio tab of winecfg to let the user choose a device. You're really doing a great job considering the mess behind it (audio architecture & drivers). I tested the latest wine and noticed that when you click the "Test Sound" button, there is no error but if you do it again (or several times), there is a err:winmm:WINMM_OpenDevice Activate failed: 80004005 on the console. Harmless but probably worth fixing.

1 0

Re: [1/4] winscard: implementation
by André Hentschel Oct. 6, 2011

Oct. 6, 2011

Am 03.10.2011 09:31, schrieb Vincent Hardy: > First submitted on wine-devel and modified following comments from Marcus Meissner (MAX_ATR_SIZE / bytecount). > OK, some notes: "winscard: implementation" is a very bad name for a patch, especially for 4. Splitting the patches is a good idea, but you need to split it different (you are adding dead code). In the first patch you might want to add the configure.ac and library loading stuff together with only the things needed for e.g. SCardListReadersA (why no SCardListReadersW?) and then e.g. a patch for every new function. I didn't looked at the code, but i want to remember you to read http://wiki.winehq.org/SubmittingPatches -- Best Regards, André Hentschel

4 3

Re: Vista/w2k8/w7 users, please test mmdevapi base render test
by Erik Inge Bolsø Oct. 6, 2011

Oct. 6, 2011

Hi Jörg, on my W7 machine I get some failures. I'm using a semi-pro usb soundcard, Creative E-MU 0202 USB. It's set to use 24-bit, 48 kHz as standard in shared mode. In addition, I turned off the E-MU and connected my cheap C-Media USB headset and left it set to default settings (44100 hz, 16-bit). I get some failures both in exclusive and non-exclusive mode. (And how does the test handle multiple soundcards?) E-MU, Exclusive allowed: render.c:167: Returned periods: 10.0000 ms 3.0000 ms render.c:179: pwfx: 00751AA8 render.c:180: Tag: fffe render.c:181: bits: 32 render.c:182: chan: 2 render.c:183: rate: 48000 render.c:184: align: 8 render.c:185: extra: 22 render.c:190: Res: 32 render.c:191: Mask: 3 render.c:192: Alg: FLOAT render.c:308: Returned latency: 10.6666 ms render.c:381: IsSupported(1, 44100x16x2) render.c:381: IsSupported(1, 48000x16x2) render.c:381: IsSupported(0, 48000x 8x2) render.c:381: IsSupported(0, 48000x16x2) render.c:887: Test failed: Initialize failed: 8889000f render.c:890: Test failed: GetService failed: 88890001 render.c:921: this is the last test seen before the exception render: unhandled exception c0000005 at 00409E1C E-MU, Exclusive disabled: render.c:167: Returned periods: 10.0000 ms 3.0000 ms render.c:179: pwfx: 00701AA8 render.c:180: Tag: fffe render.c:181: bits: 32 render.c:182: chan: 2 render.c:183: rate: 48000 render.c:184: align: 8 render.c:185: extra: 22 render.c:190: Res: 32 render.c:191: Mask: 3 render.c:192: Alg: FLOAT render.c:308: Returned latency: 10.6666 ms render.c:413: Test failed: Initialize(1, 44100x16x2) returns 8889000e render.c:413: Test failed: Initialize(1, 48000x16x2) returns 8889000e render.c:381: IsSupported(0, 48000x 8x2) render.c:381: IsSupported(0, 48000x16x2) render.c:887: Test failed: Initialize failed: 8889000f render.c:890: Test failed: GetService failed: 88890001 render.c:921: this is the last test seen before the exception render: unhandled exception c0000005 at 00409E1C C-Media USB headphone, Exclusive enabled: render.c:167: Returned periods: 10.0000 ms 3.0000 ms render.c:179: pwfx: 00A81AA8 render.c:180: Tag: fffe render.c:181: bits: 32 render.c:182: chan: 2 render.c:183: rate: 44100 render.c:184: align: 8 render.c:185: extra: 22 render.c:190: Res: 32 render.c:191: Mask: 3 render.c:192: Alg: FLOAT render.c:308: Returned latency: 11.6100 ms render.c:381: IsSupported(1, 44100x16x2) render.c:381: IsSupported(1, 48000x16x2) render.c:381: IsSupported(0, 44100x 8x2) render.c:381: IsSupported(0, 44100x16x2) render.c:1457: Test failed: Got wrong volume: 0.000000 render.c:1480: Test failed: Got wrong volume: 0.000000 render: 705 tests executed (0 marked as todo, 2 failures), 0 skipped. C-Media USB headphone, Exclusive disabled: render.c:167: Returned periods: 10.0000 ms 3.0000 ms render.c:179: pwfx: 009E1AA8 render.c:180: Tag: fffe render.c:181: bits: 32 render.c:182: chan: 2 render.c:183: rate: 44100 render.c:184: align: 8 render.c:185: extra: 22 render.c:190: Res: 32 render.c:191: Mask: 3 render.c:192: Alg: FLOAT render.c:308: Returned latency: 11.6100 ms render.c:413: Test failed: Initialize(1, 44100x16x2) returns 8889000e render.c:413: Test failed: Initialize(1, 48000x16x2) returns 8889000e render.c:381: IsSupported(0, 44100x 8x2) render.c:381: IsSupported(0, 44100x16x2) render.c:1457: Test failed: Got wrong volume: 0.000000 render.c:1480: Test failed: Got wrong volume: 0.000000 render: 705 tests executed (0 marked as todo, 4 failures), 0 skipped. -- -erik http://useofwords.blogspot.com/

2 2

Re: mscoree: Implement ICorDebug SetManagedHandler (try 2)
by Vincent Povirk Oct. 6, 2011

Oct. 6, 2011

Still not sure there should be only one ICorDebug per runtime host. It seems like you might want to create multiple interfaces and use them independently.

1 0

Re: mscoree: Implement ICorDebug SetManagedHandler (try 2)
by Marvin Oct. 6, 2011

Oct. 6, 2011

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

1 0

Re: mscoree: Implement ICorDebug SetManagedHandler
by Alexandre Julliard Oct. 5, 2011

Oct. 5, 2011

Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> writes: > @@ -99,8 +99,28 @@ static HRESULT WINAPI CorDebug_Terminate(ICorDebug *iface) > static HRESULT WINAPI CorDebug_SetManagedHandler(ICorDebug *iface, ICorDebugManagedCallback *pCallback) > { > RuntimeHost *This = impl_from_ICorDebug( iface ); > - FIXME("stub %p %p\n", This, pCallback); > - return E_NOTIMPL; > + HRESULT hr; > + > + TRACE("%p (%p)\n", This, pCallback); > + > + if(!pCallback) > + return E_INVALIDARG; > + > + if(This->pCallback2) > + ICorDebugManagedCallback2_Release(This->pCallback2); > + > + if(This->pCallback) > + ICorDebugManagedCallback_Release(This->pCallback); > + > + hr = ICorDebugManagedCallback_QueryInterface(pCallback, &IID_ICorDebugManagedCallback2, (void**)&This->pCallback2); > + if(hr == S_OK) > + { > + This->pCallback = pCallback; > + > + ICorDebugManagedCallback_AddRef(This->pCallback); > + } You can't release the old interfaces before the new ones are in place. -- Alexandre Julliard julliard(a)winehq.org

1 0

[Question] Built-in indeo drivers
by Maxim Oct. 5, 2011

Oct. 5, 2011

Hi crews, I want to propose a solution for the well-known problem with Indeo codecs. Details here: http://www.ligos.com/index.php/home/support/indeosupport/ There is a bunch of (old) games still using indeo in their cut scenes. For that reason the installation of the Intel DLLs is required. Unfortunately, old indeo codecs prior the version 5 contain bugs and their support has been discontinued for years. Moreover, Microsoft has disabled indeo in the latest OSes due to security issues. I volunteer myself to create an open-source bult-in replacement for indeo v3,4 and 5 based on my patches for ffmpeg. The code will be reworked in order to remove ffmpeg/libav library dependencies. Only decoding will be implemented. Encoding would require a huge amount of time and it doesn't worth it... Would be such a patch appreciated? Best regards Maxim Poliakovski

2 1

Re: comctl32/tests: Removed sign comparison warning in datetime tests.
by Joris Huizer Oct. 5, 2011

Oct. 5, 2011

On 10/05/2011 04:43 PM, Marko Nikolic wrote: > static void test_mccolor_types(HWND hWndDateTime, int mccolor_type, const char* mccolor_name) > { > - LRESULT r; > - COLORREF theColor, prevColor; > + //LRESULT r; > + COLORREF theColor, prevColor, crColor; > Please avoid C++/C99-style comments in Wine code In this case you should remove the 'r' variable instead of commenting it out.

2 1

Re: netapi32: add stub for NetScheduleJobEnum
by Austin English Oct. 5, 2011

Oct. 5, 2011

On Wed, Oct 5, 2011 at 11:35, Louis Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > this fixes a crash in app SysIsdpector (http://download.eset.com/download/sysinspector/32/ENU/SysInspector.exe) You should've made this a series, otherwise the build fails: -- -Austin ---------- Forwarded message ---------- From: <buildbot(a)kegel.com> Date: Wed, Oct 5, 2011 at 11:41 Subject: Re: 79589: netapi32: add stub for NetScheduleJobEnum To: dank(a)kegel.com, wine-tests-results(a)winehq.org 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/134 (though maybe not for long, as I'm still reinstalling the buildbot periodically while experimenting) BUILD FAILED: failed shell_2 Errors: netapi32.c:23: error: lmat.h: No such file or directory make[1]: *** [depend] Error 1 make: *** [dlls/netapi32/__depend__] Error 2

1 0

Re: [PATCH 1/4 (resend)] audio: Load winmm only on demand
by Alexandre Julliard Oct. 5, 2011

Oct. 5, 2011

Andrew Eikum <aeikum(a)codeweavers.com> writes: > resend: Fix subject, to make buildbot happy. > > dlls/winealsa.drv/Makefile.in | 2 +- > dlls/winealsa.drv/midi.c | 21 ++++++++++++++++++++- > dlls/winecoreaudio.drv/Makefile.in | 2 +- > dlls/winecoreaudio.drv/coreaudio.c | 2 +- > dlls/winecoreaudio.drv/midi.c | 21 ++++++++++++++++++++- > dlls/wineoss.drv/Makefile.in | 2 +- > dlls/wineoss.drv/midi.c | 21 ++++++++++++++++++++- > 7 files changed, 64 insertions(+), 7 deletions(-) Making winmm a delayed import should work just as well, with no code changes. -- Alexandre Julliard julliard(a)winehq.org

1 0