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:
> if (!devmode->dmPelsHeight) devmode->dmPelsHeight = current_mode->dmPelsHeight;
>      }
>
> -    return devmode;
> +    if (is_detached_mode( devmode )) return devmode;
> +
> +    if ((mode = find_display_mode( modes, devmode )))
> +    {
> +        mode->dmFields |= DM_POSITION;
> +        mode->dmPosition = devmode->dmPosition;
> +        return mode;

You can remove this line and return mode directly at the end. I see you already do this in 7/7. So might as well do it here to reduce the diff later.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/576#note_5635