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 ); } ```