http://bugs.winehq.org/show_bug.cgi?id=13204
--- Comment #16 from Tomas Carnecky tom@dbservice.com 2008-05-18 14:10:19 --- The tests rely on Windows' logic. Wine has to mimic the Win32 API as accurately as possible. If the wine API behaves differently than the Win32 API, then that is a bug in wine. And the tests ensure exactly that. Developers use the tests to see how Windows behaves so they can see if wine behaves the same. Just because something seems to be correct, doesn't mean it is.
Did you test your patch with pulseaudio disabled? So that wine uses the native alsa driver and not the pulse plugin? As far as I know, the winmm wave test passes in that case. So you need to make sure that your patch passes in that situation as well.