Re: [PATCH 0/5] MR3528: winegstreamer: Implement windows api frontend for file media sink.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/media_sink.c:
> -
> - return E_NOTIMPL;
>
> +}
> -
>
> +static HRESULT WINAPI media_sink_callback_Invoke(IMFAsyncCallback *iface, IMFAsyncResult *async_result)
> +{
> - struct media_sink *media_sink = impl_from_async_callback(iface);
> - struct async_command *command;
> - IUnknown *state;
> - HRESULT hr;
> -
> - TRACE("iface %p, async_result %p.\n", iface, async_result);
> -
> - EnterCriticalSection(&media_sink->cs);
> -
> - if (FAILED(hr = IMFAsyncResult_GetState(async_result, &state)))

Fwiw, you can also use `IMFAsyncResult_GetStateNoAddRef` for a simpler getter, and you won't need to release the reference.