From 4f11c53a6d948be29a69b4b3b0a40679e9dd01dd Mon Sep 17 00:00:00 2001
From: Christopher Jeffrey <chjjeffrey@gmail.com>
Date: Sat, 8 Sep 2012 21:13:56 -0500
Subject: avoid allocating a new win struct if possible

---
 compton.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/compton.c b/compton.c
index 021174804..b72dc59bd 100644
--- a/compton.c
+++ b/compton.c
@@ -1573,16 +1573,15 @@ set_opacity(Display *dpy, win *w, unsigned long opacity) {
 
 static void
 add_win(Display *dpy, Window id, Window prev, Bool override_redirect) {
+  if (find_win(dpy, id)) {
+    return;
+  }
+
   win *new = malloc(sizeof(win));
   win **p;
 
   if (!new) return;
 
-  if (find_win(dpy, id)) {
-    free(new);
-    return;
-  }
-
   if (prev) {
     for (p = &list; *p; p = &(*p)->next) {
       if ((*p)->id == prev && !(*p)->destroyed)
-- 
cgit v1.2.1