Re: [PATCH v7 0/5] MR9646: mf/tests: Scrubbing tests

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
> }
>  }
>
> +struct test_stream_sink
> +{
> +    IMFStreamSink IMFStreamSink_iface;
> +    IMFGetService IMFGetService_iface;
> +    IMFClockStateSink IMFClockStateSink_iface;

Close sink belongs to IMFMediaSink as a whole, it's not per-stream. Same for IMFGetService. If it's enough for this test sink to have a single stream, you can move everything into test_media_sink, but QueryInterface still has to treat them as separate objects. SAR does that for instance.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9646#note_127069