From cfd06718fc5d43109454407690c399ed959bf929 Mon Sep 17 00:00:00 2001 From: Tony Crisci Date: Sun, 18 May 2014 03:12:39 -0400 Subject: [PATCH] Bugfix: double X render on manage When a con is being managed, tree_render should only be called once to push the changes to the rendering server to prevent wasting resources. --- src/manage.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/manage.c b/src/manage.c index c8253f46..87f7653e 100644 --- a/src/manage.c +++ b/src/manage.c @@ -481,17 +481,17 @@ void manage_window(xcb_window_t window, xcb_get_window_attributes_cookie_t cooki ws->rect = ws->parent->rect; render_con(ws, true); } - tree_render(); + render_con(croot, false); /* Send an event about window creation */ ipc_send_window_event("new", nc); /* Defer setting focus after the 'new' event has been sent to ensure the * proper window event sequence. */ - if (set_focus) { + if (set_focus) con_focus(nc); - tree_render(); - } + + tree_render(); /* Windows might get managed with the urgency hint already set (Pidgin is * known to do that), so check for that and handle the hint accordingly.