From: Dana Jansens <danakj@orodu.net>
Date: Tue, 7 Aug 2007 02:12:20 +0000 (-0400)
Subject: more clear code?
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;ds=sidebyside;h=d3ff019f53f02afe0c7116a99f65ffb8d622e889;p=chaz%2Fopenbox

more clear code?
---

diff --git a/openbox/place.c b/openbox/place.c
index 9892836e..851e4f8f 100644
--- a/openbox/place.c
+++ b/openbox/place.c
@@ -182,7 +182,7 @@ static GSList* area_remove(GSList *list, Rect *a)
 
         if (!RECT_INTERSECTS_RECT(*r, *a)) {
             result = g_slist_prepend(result, r);
-            r = NULL; /* dont free it */
+            /* dont free r, it's moved to the result list */
         } else {
             Rect isect, extra;
 
@@ -217,9 +217,10 @@ static GSList* area_remove(GSList *list, Rect *a)
                          r->width, RECT_BOTTOM(*r) - RECT_BOTTOM(isect));
                 result = area_add(result, &extra);
             }
-        }
 
-        g_free(r);
+            /* 'r' is not being added to the result list, so free it */
+            g_free(r);
+        }
     }
     g_slist_free(list);
     return result;