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.