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

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/tests/mfmediaengine.c:
> -
>    IMFMediaType **type)
>
> +{
> - static const GUID *types[] = { &MFMediaType_Video };
> - HRESULT hr;
> -
> - if (id)
> -
>    return MF_E_INVALIDSTREAMNUMBER;
> -
> - if (index > ARRAY_SIZE(types) - 1)
> -
>    return MF_E_NO_MORE_TYPES;
> -
> - if (SUCCEEDED(hr = MFCreateMediaType(type)))
> -
>    hr = IMFMediaType_SetGUID(*type, &MF_MT_MAJOR_TYPE, types[index]);
> -
> - return hr;
>
> +}

That's fine. You could as well use MFMediaType_Video without types array, but then it's possible we'll reuse same test transform for audio effect test later.