Re: [PATCH 0/6] MR5159: Overwatch highlight/POTG encoding (part 1)

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/media_sink.c:
> if (!stream_sink->type)
>          return MF_E_NOT_INITIALIZED;
>
> -    IMFMediaType_AddRef((*type = stream_sink->type));
> -
> -    return S_OK;
> +    hr = MFCreateMediaType(&type_copy);
> +    if (FAILED(hr))
> +        return hr;
> +    hr = IMFMediaType_CopyAllItems(stream_sink->type, (IMFAttributes*)type_copy);
> +    if (FAILED(hr))
> +        IMFMediaType_Release(type_copy);
> +    else
> +        *type = type_copy;

```suggestion:-7+0
    if (FAILED(hr = MFCreateMediaType(&type_copy)))
        return hr;
    if (FAILED(hr = IMFMediaType_CopyAllItems(stream_sink->type, (IMFAttributes*)type_copy)))
        IMFMediaType_Release(type_copy);
    else
        *type = type_copy;
```

Nit: lets save some lines.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5159#note_62627