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.