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()?

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