Re: [PATCH v3 0/7] MR551: win32u: Complete support for display modes in nulldrv.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/winemac.drv/display.c:
> -
> - display_idx = wcstol(devname + 11, &end, 10) - 1;
> - if (display_idx >= num_displays)
> - {
> -
>    macdrv_free_displays(displays);
> -
>    return FALSE;
> - }
> -
> - display_id = displays[display_idx].displayID;
> -
> - pthread_mutex_lock(&cached_modes_mutex);
> -
> - if (mode == 0 || !cached_modes || flags != cached_modes_flags || display_id != cached_modes_display_id)
> - {
> -
>    if (cached_modes) CFRelease(cached_modes);
> -
>    cached_modes = copy_display_modes(display_id, (flags & EDS_RAWMODE) != 0);

Now you're including unsupported modes even if EDS_RAWMODE is not specified because display_get_modes() uses copy_display_modes(display_id, TRUE).