Re: [PATCH 0/1] MR4063: winex11.drv: Set _NET_WM_FULLSCREEN_MONITORS only when necessary.

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about dlls/winex11.drv/window.c:
> +    {
> +        unsigned long count, remaining;
> +        BOOL prop_found = FALSE;
> +        long *prop_data;
> +        int format;
> +        Atom type;
> +
> +        if (!XGetWindowProperty( data->display, data->whole_window,
> +                                 x11drv_atom(_NET_WM_FULLSCREEN_MONITORS), 0, ~0, False,
> +                                 XA_CARDINAL, &type, &format, &count, &remaining,
> +                                 (unsigned char **)&prop_data ))
> +        {
> +            if (type == XA_CARDINAL && format == 32 && count == 4)
> +                prop_found = TRUE;
> +            XFree( prop_data );
> +        }
> It shouldn't be necessary to query the property, we should be able to remember whether we already set it, without requiring a server round-trip.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4063#note_48316