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

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/image.c:
> +    }
> +    else if (IsEqualGUID(&guid, &LevelsEffectGuid))
> +    {
> +        ef->type = LevelsEffect;
> +    }
> +    else if (IsEqualGUID(&guid, &ColorCurveEffectGuid))
> +    {
> +        ef->type = ColorCurveEffect;
> +    }
> +    else
> +    {
> +        ef->type = NoneEffect;
> +        status = Win32Error;
> +    }
> +
> +    *effect = (CGpEffect *)ef;
> Why not just define `struct CGpEffect` with our private structure as we do with other types?

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