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.