Re: [PATCH 0/1] MR7116: winex11: Respect swp_flags when syncing window position.

[Gabriel Ivăncescu (＠insn)]

> Merge
> request https://gitlab.winehq.org/wine/wine/-/merge_requests/7116 was reviewed by Rémi Bernon
>
> --
>
> Rémi Bernon started a new discussion on
> dlls/winex11.drv/window.c: https://gitlab.winehq.org/wine/wine/-/merge_requests/7116#note_91691
> > - }
> > - new_rect.right = new_rect.left + new_width;
> > - new_rect.bottom = new_rect.top + new_height;
>
> What about shortening it like this? Also using `desired_state` is
> probably better here, in case window config request is being delayed.
>     new_rect = data->rects.visible;
>     if (swp_flags & SWP_NOSIZE)
>     {
>         new_rect.right = new_rect.left +
> data->desired_state.rect.right - data->desired_state.rect.left;
> new_rect.bottom = new_rect.top + data->desired_state.rect.bottom -
> data->desired_state.rect.top; } if (swp_flags & SWP_NOMOVE)
>     {
>         OffsetRect( &new_rect, data->desired_state.rect.left -
> new_rect.left, data->desired_state.rect.top - new_rect.top );
>     }

Sure. For `desired_state`, I'm a bit confused of how it all
works, I just took it from `window_set_config` old_rect. But yes that
works too.