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

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/quartz/tests/mpegvideo.c:
> +    ok(hr == S_OK, "Got hr %#lx.\n", hr);
> +
> +    size = IMediaSample_GetSize(sample);
> +    ok(size == 256, "Got size %ld.\n", size);
> +
> +    hr = IMediaSample_SetTime(sample, NULL, NULL);
> +    ok(hr == S_OK, "Got hr %#lx.\n", hr);
> +
> +    sink->expected_start_time = 0;
> +    sink->expected_stop_time = 0;
> +    sink->wine_wrong_stop_time = FALSE;
> +    hr = IMediaSample_SetTime(sample, &sink->expected_start_time, &sink->expected_stop_time);
> +    ok(hr == S_OK, "Got hr %#lx.\n", hr);
> +
> +    test_send_video(input, sample);
> +    ok(sink->got_sample >= 1, "Got %u calls to Receive().\n", sink->got_sample);
> Shouldn't we get an exact number of frames here? Or is that not consistent somehow?

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