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, ¶m_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, ¶m_size);

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

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4658#note_55704