Re: [PATCH v3 0/6] MR2568: win32u: Support setting the current mode using gdi_device_manager.

[Rémi Bernon]

Rémi Bernon (@rbernon) commented about dlls/winemac.drv/display.c:
> device_manager->add_monitor( &gdi_monitor, param );
>              }
>
> +            if (macdrv_get_displays(&displays, &display_count)) break;
> +
> +            /* Get the current mode */
> +            for (display = displays; display < displays + display_count; display++)
> +            {
> +                if (display->displayID == adapter->id)
> +                {
> +                    display_get_current_mode(display, ¤t_mode);
> +                    break;
> +                }
> +            }
> +
> +            macdrv_free_displays(displays);

It doesn't seem very optimal to get the displays every iteration?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2568#note_28726