Re: [PATCH v4 0/5] MR7370: winex11: Pass fullscreen flag to is_window_managed.

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about dlls/winex11.drv/window.c:
>  XFlush( data->display );
>
> }
>
> +static void window_set_managed( struct x11drv_win_data *data, BOOL new_managed, BOOL new_embedded )
> +{
> - XSetWindowAttributes attr = {.override_redirect = !new_managed};
> - BOOL old_managed = data->managed, old_embedded = data->embedded;
> - UINT wm_state = data->desired_state.wm_state;

It doesn't seem an improvement to tie the two states together, and then have to pass data->embedded everywhere to avoid changing it.