Re: [PATCH 0/7] MR576: win32u: Move display placement logic out of graphics drivers.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
>  memset( &devmode->dmDriverExtra, 0, devmode->dmSize - offsetof(DEVMODEW, dmDriverExtra) );
>
>  if (index == ENUM_REGISTRY_SETTINGS) ret = read_registry_settings( adapter_path, devmode );
> - else ret = user_driver->pEnumDisplaySettingsEx( device_name, index, devmode, flags );
> - else if (index != ENUM_CURRENT_SETTINGS) ret = user_driver->pEnumDisplaySettingsEx( device_name, index, devmode, flags );

No really an issue. But moving the getting the current display settings part right after getting registry settings seems more natural to me. So index == ENUM_CURRENT_SETTINGS here and then do EnumDisplaySettingsEx() in the else branch.