From: Dana Jansens <danakj@orodu.net>
Date: Tue, 20 Apr 2010 19:22:58 +0000 (-0400)
Subject: add asserts to make sure we don't add things to stacking list that are not managed
X-Git-Url: https://git.brokenzipper.com/gitweb?a=commitdiff_plain;h=72fe65fd2284e2f592e71c33d50dd3b7bc9ea569;p=chaz%2Fopenbox

add asserts to make sure we don't add things to stacking list that are not managed
---

diff --git a/openbox/stacking.c b/openbox/stacking.c
index cb710e5e..405b7bd4 100644
--- a/openbox/stacking.c
+++ b/openbox/stacking.c
@@ -438,8 +438,11 @@ void stacking_add(ObWindow *win)
 {
     g_assert(screen_support_win != None); /* make sure I dont break this in the
                                              future */
+    /* don't add windows that are being unmanaged ! */
+    if (WINDOW_IS_CLIENT(win)) g_assert(WINDOW_AS_CLIENT(win)->managed);
 
     stacking_list = g_list_append(stacking_list, win);
+
     stacking_raise(win);
 }
 
@@ -498,6 +501,9 @@ void stacking_add_nonintrusive(ObWindow *win)
 
     client = WINDOW_AS_CLIENT(win);
 
+    /* don't add windows that are being unmanaged ! */
+    g_assert(client->managed);
+
     /* insert above its highest parent (or its highest child !) */
     it_below = find_highest_relative(client);