Re: [PATCH v5 0/6] MR8079: winex11: Serialize fullscreen/maximized and config changes.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/winex11.drv/window.c:
>  static const UINT fullscreen_mask = (1 << NET_WM_STATE_MAXIMIZED) | (1 << NET_WM_STATE_FULLSCREEN);
>  UINT style = NtUserGetWindowLongW( data->hwnd, GWL_STYLE ), mask = 0;
>  const RECT *old_rect = &data->pending_state.rect;
> -
>
> BOOL old_above = data->pending_state.above;
> XWindowChanges changes;
> RECT *new_rect = &rect;
>
> data->desired_state.rect = *new_rect;
> -
>
> data->desired_state.above = above;
> if (!data->whole_window) return; /* no window, nothing to update */
> - if (EqualRect( old_rect, new_rect ) && !above) return; /* rects are the same, no need to be raised, nothing to update */
> - if (EqualRect( old_rect, new_rect ) && old_above >= above) return; /* rects are the same, no need to be raised, nothing to update */

Using >= for BOOL types is weird. Let's not do that.