Re: [PATCH v6 0/5] MR4661: Draft: gdiplus: Implement Gdiplus effects

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/image.c:
> TRACE("(%p,%p,%u)\n", effect, params, size);
>
> -    if(!(calls++))
> -        FIXME("not implemented\n");
> +    if (!effect || !params || !size)
> +        return InvalidParameter;
>
> -    return NotImplemented;
> +    ef = (GpEffect *)effect;
> +    switch (ef->type)
> +    {
> +    case BlurEffect:
> +        paramsize = sizeof(struct BlurParams);
> +        if (paramsize != size)
> +            return InvalidParameter;
> +        break;

The mapping of type to paramsize shouldn't be duplicated here.

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