Re: [PATCH v2 0/5] MR2778: mfmediaengine: Add support for video effects.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/mfmediaengine/main.c:
> -
> - if (!engine->effects.video_effects_size)
> -
>    return S_OK;
> -
> - if(!(*nodes = calloc(engine->effects.video_effects_size, sizeof(**nodes))))
> -
>    return E_OUTOFMEMORY;
> -
> - for (i = 0; i < engine->effects.video_effects_size; ++i)
> - {
> -
>    IMFTopologyNode *node = NULL;
> -
> -
>    if (FAILED(hr = MFCreateTopologyNode(MF_TOPOLOGY_TRANSFORM_NODE, &node)))
> -
>        goto failed;
> -
> -
>    IMFTopologyNode_SetObject(node, (IUnknown *)engine->effects.video_effects[i].object);
> -
>    IMFTopologyNode_SetUINT32(node, &MF_TOPONODE_NOSHUTDOWN_ON_REMOVE, FALSE);

How did you verify this one?