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?