add some comments and asserts to make sure I don't ever add this bug again

diff --git a/openbox/focus.c b/openbox/focus.c
index ad18d9748aea81f8a0e18c1893826393790aed05..b8477bcb148c2cb4297baa3524c1381c67016b1c 100644 (file)
--- a/openbox/focus.c
+++ b/openbox/focus.c
@@ -34,7 +34,6 @@ void focus_startup()
     XSetWindowAttributes attrib;
 
     focus_client = NULL;
-    focus_cycle_popup = popup_new(TRUE);
 
     attrib.override_redirect = TRUE;
     focus_backup = XCreateWindow(ob_display, ob_root,
@@ -43,6 +42,10 @@ void focus_startup()
                                  CWOverrideRedirect, &attrib);
     XMapRaised(ob_display, focus_backup);
 
+    /* do this *after* focus_backup is created, since it is used for
+       stacking */
+    focus_cycle_popup = popup_new(TRUE);
+
     /* start with nothing focused */
     focus_set_client(NULL);
 }

diff --git a/openbox/openbox.c b/openbox/openbox.c
index d9ce9a5d8933c1c42b7d065c81fb2cdcf43ccc3a..a2f9688d719b0743a727fc78076496e9a7aa2218 100644 (file)
--- a/openbox/openbox.c
+++ b/openbox/openbox.c
@@ -197,6 +197,9 @@ int main(int argc, char **argv)
        timer_startup();
        event_startup();
         grab_startup();
+        /* focus_backup is used for stacking, so this needs to come before
+           anything that calls stacking_add */
+       focus_startup();
         window_startup();
         plugin_startup();
         /* load the plugins specified in the pluginrc */
@@ -218,7 +221,6 @@ int main(int argc, char **argv)
 
         frame_startup();
         moveresize_startup();
-       focus_startup();
        screen_startup();
         group_startup();
        client_startup();

diff --git a/openbox/stacking.c b/openbox/stacking.c
index 2fa4b2e6d9dc8c83d15c2641193da527ccf9c6fa..643b3118b6edc00caffd5178c003bd21df252fb4 100644 (file)
--- a/openbox/stacking.c
+++ b/openbox/stacking.c
@@ -61,6 +61,8 @@ static void do_restack(GList *wins, GList *before)
 
     for (i = 1, it = wins; it; ++i, it = g_list_next(it)) {
         win[i] = window_top(it->data);
+        g_assert(win[i] != None); /* better not call stacking shit before
+                                     setting your top level window value */
         stacking_list = g_list_insert_before(stacking_list, before, it->data);
     }
 
@@ -253,7 +255,10 @@ void stacking_lower(ObWindow *window)
 void stacking_add(ObWindow *win)
 {
     StackLayer l;
-    GList *wins, *it;
+    GList *wins;
+
+    g_assert(focus_backup != None); /* make sure I dont break this in the
+                                       future */
 
     l = window_layer(win);
     wins = g_list_append(NULL, win); /* list of 1 element */