Re: [PATCH v8 0/3] MR231: SetWindowPos() parent window exposure fixes (#53153)

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about server/window.c:
> win->visible_rect = *visible_rect;
>      win->surface_rect = *surface_rect;
>      win->client_rect  = *client_rect;
> -    if (!(swp_flags & SWP_NOZORDER) && win->parent) link_window( win, previous );
> +    if (!(swp_flags & SWP_NOZORDER) && win->parent)
> +    {
> +        struct list *old_prev = win->is_linked ? win->entry.prev : NULL;
> +        link_window( win, previous );
> +        if (old_prev != win->entry.prev) zorder_changed = 1;

I think that it would be cleaner to return if z-order changed from `link_window()` would be cleaner.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/231#note_12988