Re: [PATCH 0/2] MR3369: mf/tests: Test sample processing for MPEG4 media sink.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mf/tests/mf.c:
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - hr = IMFSample_SetSampleDuration(input_sample, 10000000);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - hr = IMFStreamSink_ProcessSample(stream_sink, input_sample);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> -
> - /* Finalize media sink. */
> - hr = IMFPresentationClock_Stop(clock);
> - ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
> - WaitForSingleObject(stream_event_callback.stop_event, 3000);
> -
> - finalize_callback.IMFAsyncCallback_iface.lpVtbl = &test_media_sink_callback_finalize_vtbl;
> - finalize_callback.finalize_event = CreateEventW(NULL, FALSE, FALSE, NULL);
> - finalize_callback.media_sink = media_sink;
> - hr = IMFFinalizableMediaSink_BeginFinalize((IMFFinalizableMediaSink *)media_sink,
> -
>        &finalize_callback.IMFAsyncCallback_iface, NULL);

This is not a good way to cast an interface pointer.