Re: [PATCH v2 0/3] MR4479: winegstreamer: Implement wg_muxer_finalize.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/media_sink.c:
> -
> - if (FAILED(hr = async_command_create(ASYNC_FINALIZE, &command)))
> -
>    return hr;
> -
> - if (FAILED(hr = MFCreateAsyncResult(NULL, callback, state, &result)))
> - {
> -
>    free(command);
> -
>    return hr;
> - }
> - IMFAsyncResult_AddRef((command->u.finalize.result = result));
> -
> - if (FAILED(hr = MFPutWorkItem(MFASYNC_CALLBACK_QUEUE_STANDARD,
> -
>        &media_sink->async_callback, &command->IUnknown_iface)))
> -
>    IUnknown_Release(&command->IUnknown_iface);
> -
> - return hr;

```suggestion:-14+0
    if (FAILED(hr = async_command_create(ASYNC_FINALIZE, &command)))
        return hr;

if (FAILED(hr = MFCreateAsyncResult(NULL, callback, state, &result)))
    {
        IUnknown_Release(&command->IUnknown_iface);
        return hr;
    }
    IMFAsyncResult_AddRef((command->u.finalize.result = result));

hr = MFPutWorkItem(MFASYNC_CALLBACK_QUEUE_STANDARD,
            &media_sink->async_callback, &command->IUnknown_iface));
    IUnknown_Release(&command->IUnknown_iface);

return hr;
```