Ken Thomases : winmm/tests: Don' t complain for Wine drivers that allow multiple wave-out opens.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 8b086d6050704c4722f7700c9ed64f597133468a
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=8b086d6050704c4722f7700c9e...

Author: Ken Thomases <ken(a)codeweavers.com>
Date:   Fri Oct 16 04:10:09 2009 -0500

winmm/tests: Don't complain for Wine drivers that allow multiple wave-out opens.

---

 dlls/winmm/tests/wave.c |   14 +++++++++-----
 1 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/dlls/winmm/tests/wave.c b/dlls/winmm/tests/wave.c
index 3e3afbd..98a16f5 100644
--- a/dlls/winmm/tests/wave.c
+++ b/dlls/winmm/tests/wave.c
@@ -59,12 +59,16 @@ static void test_multiple_waveopens(void)
     }
 
     ret = waveOutOpen(&handle2, 0, &wfx, 0, 0, 0);
-    /* In windows this is most likely allowed, in wine an application can use the waveout
-     * interface, but so can directsound.. this causes problems if directsound goes active
+    /* Modern Windows allows for wave-out devices to be opened multiple times.
+     * Some Wine audio drivers allow that and some don't.  To avoid false alarms
+     * for those that do, don't "todo_wine ok(...)" on success.
      */
-    todo_wine ok(ret == MMSYSERR_NOERROR || broken(ret == MMSYSERR_ALLOCATED), /* winME */
-                 "waveOutOpen returns: %x\n", ret);
-    if (ret == MMSYSERR_NOERROR)
+    if (ret != MMSYSERR_NOERROR)
+    {
+        todo_wine ok(ret == MMSYSERR_NOERROR || broken(ret == MMSYSERR_ALLOCATED), /* winME */
+                     "second waveOutOpen returns: %x\n", ret);
+    }
+    else
         waveOutClose(handle2);
 
     waveOutClose(handle1);