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

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/winemac.drv/display.c:
> - {
> -
>    /* We only synthesize modes from those having the default bpp. */
> -
>    if (devmodes[i].dmBitsPerPel != default_bpp) continue;
> -
>    devmodes[count + synth_count] = devmodes[i];
> -
>    devmodes[count + synth_count].dmBitsPerPel = 8;
> -
>    synth_count++;
> - }
> -
> - desc = create_original_display_mode_descriptor(display_id);
> - for (i = 0; i < count; i++)
> - {
> -
>    CGDisplayModeRef mode = (CGDisplayModeRef)CFArrayGetValueAtIndex(modes, i);
> -
>    if (retina_enabled && display_mode_matches_descriptor(mode, desc))
> -
>    {
> -
>        devmodes[i].dmPelsWidth *= 2;
> -
>        devmodes[i].dmPelsHeight *= 2;

Shouldn't you apply the scaling to fake 16bit and 8bit modes?