From: Tony Crisci <tony@dubstepdish.com>
Date: Sun, 18 May 2014 07:12:39 +0000 (-0400)
Subject: Bugfix: double X render on manage
X-Git-Tag: 4.8~34
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=cfd06718fc5d43109454407690c399ed959bf929;p=i3%2Fi3

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.
---

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.