Rémi Bernon (@rbernon) commented about dlls/winex11.drv/window.c:
+ { + new_width = data->pending_state.rect.right - data->pending_state.rect.left; + new_height = data->pending_state.rect.bottom - data->pending_state.rect.top; + } + else + { + new_width = new_rect.right - new_rect.left; + new_height = new_rect.bottom - new_rect.top; + } + if (swp_flags & SWP_NOMOVE) + { + new_rect.left = data->pending_state.rect.left; + new_rect.top = data->pending_state.rect.top; + } + 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.
```suggestion:-17+0 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 ); } ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7116#note_91691