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->flags & FLAGS_ENGINE_SHUT_DOWN)
> -
>    hr = MF_E_SHUTDOWN;
> - else if (!(tmp = realloc(engine->effects.video_effects,
> -
>    sizeof(*engine->effects.video_effects) * (engine->effects.video_effects_size + 1))))
> - {
> -
>    hr = MF_E_INVALIDREQUEST;
> - }
> - else
> - {
> -
>    tmp[engine->effects.video_effects_size].object = effect;
> -
>    IUnknown_AddRef(tmp[engine->effects.video_effects_size].object);
> -
>    tmp[engine->effects.video_effects_size].optional = is_optional;
> -
> -
>    engine->effects.video_effects = tmp;
> -
>    engine->effects.video_effects_size++;

Could you use mf_array_reserve()?