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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7370#note_95928