Re: [PATCH v7 0/2] MR2778: mfmediaengine: Add support for video effects.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> -
>    goto done;
> -
> - if (FAILED(hr = IMFMediaType_GetItem(type, &MF_MT_FRAME_SIZE, &propvar)))
> -
>    goto done;
> -
> - impl->frame_width = width = propvar.uhVal.HighPart;
> - impl->frame_height = height = propvar.uhVal.LowPart;
> -
> - if (impl->media_type_in) IMFMediaType_Release(impl->media_type_in);
> - impl->media_type_in = type;
> - IMFMediaType_AddRef(impl->media_type_in);
> -
> - /* Idk why, but Windows apparently expects this function to also set an output type... */
> - if (impl->media_type_out) IMFMediaType_Release(impl->media_type_out);
> - impl->media_type_out = type;
> - IMFMediaType_AddRef(impl->media_type_out);

This one is a strange requirement.