Re: [PATCH v3 0/5] MR720: winemac: Allow enumeration of non-primary display modes.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/winemac.drv/display.c:
> BOOL macdrv_GetCurrentDisplaySettings(LPCWSTR devname, LPDEVMODEW devmode)
>  {
>      struct macdrv_display *displays = NULL;
> - int num_displays;
> - int i, num_displays, display_idx;

i is unused.

Also this commit makes the console spam "002c:err:system:NtUserChangeDisplaySettings Changing L"\\.\DISPLAY2" display settings returned -2.
002c:err:display:init_registry_display_settings Failed to save registry display settings for L"\\.\DISPLAY2", returned -2.", even after patch 5/5. This is caused by macdrv_ChangeDisplaySettingsEx() still using the primary monitor available mode list even for secondary monitors. You can move the non-primary monitor check earlier and return fake success to avoid this for now.