Re: [PATCH v5 0/3] MR1046: MfMediaEngine invalid format handling.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> - {
> -
>    notify.engine_fmt = tested_formats[i].engine_fmt;
> -
>    notify.output_fmt = tested_formats[i].output_fmt;
> -
>    notify.media_engine = NULL;
> -
>    notify.error = S_OK;
> -
>    media_engine = NULL;
> -
> -
>    winetest_push_context("%x, %x", tested_formats[i].engine_fmt, tested_formats[i].output_fmt);
> -
> -
>    media_engine = create_media_engine_ex(&notify.IMFMediaEngineNotify_iface,
> -
>            manager, tested_formats[i].engine_fmt);
> -
> -
>    ok(!!media_engine, "MfMediaEngine creation failed.\n");
> -
> -
>    if (!(notify.media_engine = media_engine))
> -
>        goto skip_iter;

I don't think you need to check that again or even skip tests if it's not supposed to happen.