From ea9d8a895bbb7919a9f2f44fd57dd426e45134cc Mon Sep 17 00:00:00 2001 From: Carlos Garnacho Date: Mon, 25 Feb 2019 18:47:59 +0100 Subject: [PATCH] wayland: Drop error trap Code underneath seems to handle errors properly, or be x11-agnostic entirely, this is apparently here to save a few XSync()s on X11. Just drop this windowing dependent bit to make things cleaner. https://gitlab.gnome.org/GNOME/mutter/merge_requests/420 --- src/wayland/meta-window-wayland.c | 28 +++++++--------------------- 1 file changed, 7 insertions(+), 21 deletions(-) diff --git a/src/wayland/meta-window-wayland.c b/src/wayland/meta-window-wayland.c index 8191adadd..5ce67c689 100644 --- a/src/wayland/meta-window-wayland.c +++ b/src/wayland/meta-window-wayland.c @@ -655,7 +655,6 @@ meta_window_wayland_new (MetaDisplay *display, MetaWaylandSurface *surface) { XWindowAttributes attrs = { 0 }; - MetaWindow *window; /* * Set attributes used by _meta_window_shared_new, don't bother trying to fake @@ -670,26 +669,13 @@ meta_window_wayland_new (MetaDisplay *display, attrs.map_state = IsUnmapped; attrs.override_redirect = False; - /* XXX: Note: In the Wayland case we currently still trap X errors while - * creating a MetaWindow because we will still be making various redundant - * X requests (passing a window xid of None) until we thoroughly audit all - * the code to make sure it knows about non X based clients... - */ - meta_x11_error_trap_push (display->x11_display); /* Push a trap over all of window - * creation, to reduce XSync() calls - */ - - window = _meta_window_shared_new (display, - META_WINDOW_CLIENT_TYPE_WAYLAND, - surface, - None, - WithdrawnState, - META_COMP_EFFECT_CREATE, - &attrs); - - meta_x11_error_trap_pop (display->x11_display); /* pop the XSync()-reducing trap */ - - return window; + return _meta_window_shared_new (display, + META_WINDOW_CLIENT_TYPE_WAYLAND, + surface, + None, + WithdrawnState, + META_COMP_EFFECT_CREATE, + &attrs); } static gboolean