Re: [PATCH 0/1] MR2816: mf/tests: Add tests 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 = MFCreateMPEG4MediaSink(NULL, NULL, NULL, NULL);
> - todo_wine
> - ok(hr == E_POINTER, "Unexpected hr %#lx.\n", hr);
> -
> - sink = (void *)0xdeadbeef;
> - hr = MFCreateMPEG4MediaSink(NULL, NULL, NULL, &sink);
> - todo_wine
> - ok(hr == E_POINTER, "Unexpected hr %#lx.\n", hr);
> - ok(sink == (void *)0xdeadbeef, "Unexpected pointer %p.\n", sink);
> - sink = NULL;
> -
> - hr = MFCreateMPEG4MediaSink(bytestream_empty, NULL, NULL, &sink_empty);
> - todo_wine
> - ok(hr == S_OK || broken(hr == E_INVALIDARG), "Unexpected hr %#lx.\n", hr);

When does it fail? If it's current win10+ behaviour to expect media types, just keep this broken, no need to test it further.