Huw Davies (@huw) commented about dlls/mmdevapi/client.c:
- return E_NOTIMPL;
- UINT32 dummy;
- HRESULT hr;
- TRACE("(%p)->(%p, %p)\n", This, cur_format, cur_period_frames);
- if (!cur_format || !cur_period_frames)
return E_POINTER;
- if (FAILED(hr = client_GetMixFormat(iface, cur_format)))
return hr;
- if (FAILED(hr = client_GetSharedModeEnginePeriod(iface, *cur_format, cur_period_frames, &dummy, &dummy, &dummy)))
return hr;
- return hr;
Any reason not to return directly from `client_GetSharedModeEnginePeriod()`?