Re: DSound/winepulse review

2012-10-26 Thread Maarten Lankhorst
Added wine-devel to cc Op 26-10-12 13:12, joerg-cyril.hoe...@t-systems.com schreef: > Maarten, > > I'll try and come up with a review of your winepulse patches yet I'm not > familiar with the PA API. > > Regarding your dsound ones, I'm sorry I've not enough experience > with the DSound API to be

Re: [PATCH 3/3] winmm: Call IsFormatSupported for WAVE_FORMAT_QUERY only.

2012-10-26 Thread Andrew Eikum
Ran tests on all four backends and everything seems okay. On Fri, Oct 26, 2012 at 10:31:33AM +0200, joerg-cyril.hoe...@t-systems.com wrote: > Hi, > > With the previous code in place, we can derive sane values for > WAVE_FORMAT_QUERY. > > Given the amount of work and side-effects that the audio

ntdll: Do not execute callbacks past DeleteTimer(INVALID_HANDLE_VALUE)

2012-10-26 Thread Joerg-Cyril.Hoehle
Hi, Your paranoid android wrote: [what am I? A human answering a bot!?!] >=== WVISTAADM (32 bit sync) === >sync.c:946: Test failed: DeleteTimerQueueTimer That's a flaky test already known from: http://test.winehq.org/data/6ae3418aaf873b85f4508a8920e565d039e7b817/2000_fg-win2000-ie6/kernel32:sync

Re: ntdll: Do not execute callbacks past DeleteTimer(INVALID_HANDLE_VALUE)

2012-10-26 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=22495 Your paranoid android