Re: [PATCH v15 0/4] MR4661: gdiplus: Implement Gdiplus effects

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/tests/image.c:
> stat = pGdipCreateEffect(BlurEffectGuid, NULL);
>      expect(InvalidParameter, stat);
>
> +    stat = pGdipGetEffectParameterSize(NULL, NULL);
> +    expect(InvalidParameter, stat);
> +
> +    stat = pGdipGetEffectParameterSize(effect, NULL);
> +    expect(InvalidParameter, stat);
> +
>      stat = pGdipCreateEffect(noneffect, &effect);
>      expect(Win32Error, stat);
>      ok(effect == NULL, "Expected effect to be NULL\n");
>
> -    for(i=0; i < ARRAY_SIZE(effectlist); i++)
> +    size = 0;
> +    stat = pGdipGetEffectParameterSize(effect, &size);

effect is supposed to be NULL here, so it doesn't make sense to pass it in by name.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4661#note_58017