Re: [PATCH v2 0/4] MR406: win32u: Introduce new add_modes device manager callback.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/winemac.drv/display.c:
> -
>    return NULL;
> - }
> -
> - count = CFArrayGetCount(modes);
> - for (i = 0; i < count; i++)
> - {
> -
>    CGDisplayModeRef mode = (CGDisplayModeRef)CFArrayGetValueAtIndex(modes, i);
> -
>    display_mode_to_devmode(display_id, mode, devmodes + i);
> - }
> -
> - if (!modes_has_16bpp) for (i = 0; i < count; i++)
> - {
> -
>    /* We only synthesize modes from those having the default bpp. */
> -
>    if (devmodes[i].dmBitsPerPel != default_bpp) continue;
> -
>    devmodes[*modes_count] = devmodes[i];
> -
>    devmodes[*modes_count++].dmBitsPerPel = 16;

This doesn't do what you probably meant.  Likewise below.