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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/winegstreamer/media_sink.c:
> static HRESULT WINAPI media_sink_SetPresentationClock(IMFFinalizableMediaSink *iface, IMFPresentationClock *clock)
>  {
> - FIXME("iface %p, clock %p stub!\n", iface, clock);
> - struct media_sink *media_sink = impl_from_IMFFinalizableMediaSink(iface);
> - HRESULT hr = S_OK;
> - return E_NOTIMPL;
> - TRACE("iface %p, clock %p\n", iface, clock);
> -
> - if (media_sink->clock)
> - {
> -
>    hr = IMFPresentationClock_RemoveClockStateSink(media_sink->clock, &media_sink->IMFClockStateSink_iface);
> -
>    if (!SUCCEEDED(hr))

```suggestion:-0+0
        if (FAILED(hr))
```

Same everywhere else.