Re: [PATCH v4 0/2] MR4658: gidplus: Add GdipGetEffectParameterSize stub, fix GdipDeleteEffect and update test.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/tests/image.c:
> - for(i=0; i < ARRAY_SIZE(effectlist); i++)
> - param_size = 0;
> - stat = pGdipGetEffectParameterSize(NULL, &param_size);
> - expect(InvalidParameter, stat);
> - expect(0, param_size);
> -
> - for (i = 0; i < ARRAY_SIZE(td); i++)
> {
> -
>    stat = pGdipCreateEffect(*effectlist[i], &effect);
> -
>    todo_wine expect(Ok, stat);
> -
>    stat = pGdipCreateEffect(*(td[i].effect), &effect);
> -
>    todo_wine_if(td[i].todo) expect(Ok, stat);
>    if (stat == Ok)
>    {
> -
>        stat = pGdipGetEffectParameterSize(effect, &param_size);

I think `param_size` should be set to 0 before each call so the tests are independent.