[PATCH 2/2] mmdevapi/tests: Relax capture test that sometimes fails on Windows 8

Andrew Eikum aeikum at codeweavers.com
Mon Feb 19 08:45:35 CST 2018


Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>

On Mon, Feb 19, 2018 at 02:31:30AM -0700, Alex Henrie wrote:
> Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
> ---
>  dlls/mmdevapi/tests/capture.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/dlls/mmdevapi/tests/capture.c b/dlls/mmdevapi/tests/capture.c
> index 27a5cdc33f..0fa9f0cab8 100644
> --- a/dlls/mmdevapi/tests/capture.c
> +++ b/dlls/mmdevapi/tests/capture.c
> @@ -413,9 +413,9 @@ static void test_capture(IAudioClient *ac, HANDLE handle, WAVEFORMATEX *wfx)
>      hr = IAudioCaptureClient_GetBuffer(acc, &data, &frames, &flags, &pos, &qpc);
>      ok(hr == S_OK, "Valid IAudioCaptureClient_GetBuffer returns %08x\n", hr);
>      trace("Running position %d pad %u flags %x, amount of frames locked: %u\n",
> -          hr==S_OK ? (UINT)pos : -1, pad, flags, frames);
> +          SUCCEEDED(hr) ? (UINT)pos : -1, pad, flags, frames);
>  
> -    if(hr == S_OK){
> +    if(SUCCEEDED(hr)){
>          /* Some w7 machines signal DATA_DISCONTINUITY here following the
>           * previous AUDCLNT_S_BUFFER_EMPTY, others not.  What logic? */
>          ok(pos >= sum, "Position %u gap %d\n", (UINT)pos, (UINT)pos - sum);
> -- 
> 2.16.1
> 



More information about the wine-devel mailing list