Re: Support for WM spec

[Havoc Pennington <hp redhat com>]

Alexander Larsson <alla lysator liu se> writes:  
> +  xev.xclient.window = gdk_root_window;
> +  xev.xclient.display = gdk_display;
> +  xev.xclient.window = gdk_root_window;

Should be the window to be modal, not the root window, I think.

I have this same code in a couple places - maybe it should be broken
out into a function.

> +#ifdef NO_GDK_WMSPEC_SUPPORTED_YET
> +  if (!_gdk_wmspec_supported (gdk_atom_intern ("_NET_WM_ICON", FALSE)))
> +    return FALSE;
> +#endif

Possibly should set the icon anyway, in case a supported WM appears
later, as Tim pointed out for the stuff I was doing.

Havoc