> -    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);

Should there be an ok(TRUE) in the else clause to keep track that the
test succeeded?

 - Allan


Reply via email to