Re: [PATCH 0/6] MR561: winex11.drv: Use DEVMODEW instead of x11drv_display_setting.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/winex11.drv/display.c:
> static void place_all_displays(struct x11drv_display_setting *displays, INT display_count)
>  {
>      INT left_most = INT_MAX, top_most = INT_MAX;
> -    INT placing_idx, display_idx;
>      POINT min_offset, offset;
> +    DEVMODEW *placing;
> +    INT display_idx;
>
> for (display_idx = 0; display_idx < display_count; ++display_idx)
> -        displays[display_idx].desired_mode.dmFields &= ~DM_POSITION;
> +    {
> +        DEVMODEW *mode = &displays[display_idx].mode;
> +        mode->dmFields &= ~DM_POSITION;
> +    }
> mode is only used once. It seems simpler to use displays[display_idx].mode.dmFields here.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/561#note_5397