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

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/image.c:
> +    {
> +        if (paramsize != size)
> +            return InvalidParameter;
> +    }
> +
> +    switch (effect->type)
> +    {
> +    case BlurEffect:
> +        alloc_copy(params, effect->p.blurparams, size);
> +        break;
> +    case SharpenEffect:
> +        alloc_copy(params, effect->p.sharpenparams, size);
> +        break;
> +    case TintEffect:
> +        alloc_copy(params, effect->p.tintparams, size);
> +        break;
> I think all the cases except RedEyeCorrectionEffect can be combined into one case. I don't think the `default:` case should be reachable.

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