add asserts to make sure we don't add things to stacking list that are not managed
authorDana Jansens <danakj@orodu.net>
Tue, 20 Apr 2010 19:22:58 +0000 (15:22 -0400)
committerDana Jansens <danakj@orodu.net>
Tue, 20 Apr 2010 19:22:58 +0000 (15:22 -0400)
openbox/stacking.c

index cb710e5..405b7bd 100644 (file)
@@ -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);