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

[Rémi Bernon]

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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3528#note_41602