Rémi Bernon (@rbernon) commented about dlls/winex11.drv/window.c:
case MAKELONG(WithdrawnState, NormalState): case MAKELONG(IconicState, NormalState): if (data->embedded) set_xembed_flags( data, XEMBED_MAPPED );
else XMapWindow( data->display, data->whole_window );
else
{
XMapWindow( data->display, data->whole_window );
update_net_wm_fullscreen_monitors( data );
}
I don't understand what difference it makes, or even how this is better? When the window is mapped, window managers should check whether it has the property set already, (and whether it is fullscreen) and apply the proper changes.