Re: [PATCH v13 0/7] MR3938: winegstreamer: Implement CLSID_CMpegVideoCodec

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/quartz/tests/mpegvideo.c:
> +static HRESULT WINAPI testsink_Receive(struct strmbase_sink *iface, IMediaSample *sample)
> +{
> +    struct testfilter *filter = impl_from_strmbase_filter(iface->pin.filter);
> +    REFERENCE_TIME start, stop;
> +    HRESULT hr;
> +    LONG size;
> +
> +    size = IMediaSample_GetSize(sample);
> +    ok(size == 1536, "Got size %lu.\n", size);
> +    size = IMediaSample_GetActualDataLength(sample);
> +    ok(size == 1536, "Got actual size %lu.\n", size);
> +
> +    start = 0xdeadbeef;
> +    stop = 0xdeadbeef;
> +    hr = IMediaSample_GetTime(sample, &start, &stop);
> +    ok(hr == S_OK, "Got hr %#lx.\n", hr);
> Can you please also check (at least the return value of) IMediaSample::GetMediaType()? I'm rather curious if it's updated or not.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3938#note_48914