Re: [PATCH v3 0/4] MR3554: mmdevapi: fixes for Yousician

[Chip Davis (＠cdavis5e)]

Chip Davis (@cdavis5e) commented about dlls/mmdevapi/tests/render.c:
> -
>    hr = IAudioClient3_GetSharedModeEnginePeriod(
> -
>        ac3, pwfx, &default_period, &unit_period, &min_period, &max_period);
> -
>    ok(hr == S_OK, "GetSharedModeEnginePeriod returns %08lx\n", hr);
> -
> -
>    if(hr == S_OK){
> -
>        hr = IAudioClient3_InitializeSharedAudioStream(
> -
>            ac3, AUDCLNT_SHAREMODE_SHARED, default_period, pwfx, NULL);
> -
>        ok(hr == S_OK, "InitializeSharedAudioStream returns %08lx\n", hr);
> -
>    }
> -
>    IAudioClient3_Release(ac3);
> -
>    IAudioClient_Release(ac);
> -
> -
>    hr = IMMDevice_Activate(dev, &IID_IAudioClient, CLSCTX_INPROC_SERVER,
> -
>            NULL, (void**)&ac);
> -
>    ok(hr == S_OK, "Activation failed with %08lx\n", hr);

Is this necessary?