Re: [PATCH 0/8] MR3009: wine{alsa,coreaudio,oss,pulse}: Move AudioClient's GetDevicePeriod, GetMixFormat, IsFormatSupported, GetCurrentPadding, GetStreamLatency, GetBufferSize into mmdevapi.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/mmdevapi/client.c:
> - struct audio_client *This = impl_from_IAudioClient3(iface);
> - struct get_device_period_params params;
> -
> - TRACE("(%p)->(%p, %p)\n", This, defperiod, minperiod);
> -
> - if (!defperiod && !minperiod)
> -
>    return E_POINTER;
> -
> - params.device     = This->device_name;
> - params.flow       = This->dataflow;
> - params.def_period = defperiod;
> - params.min_period = minperiod;
> -
> - WINE_UNIX_CALL(get_device_period, &params);
> -
> - return S_OK;

Why are we returning `S_OK` unconditionally here?  All of the drivers return `params.result`.