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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2778#note_37790