From: Dana Jansens <danakj@orodu.net>
Date: Mon, 1 Oct 2012 03:42:16 +0000 (-0400)
Subject: When a window is the focus target and it is removed from the focus order, reset the... 
X-Git-Url: https://git.dogcows.com/gitweb?a=commitdiff_plain;h=56a0993300e6b4939b2c4eae1cf4d444732cd043;p=chaz%2Fopenbox

When a window is the focus target and it is removed from the focus order, reset the focus cycling (Fix bug 5410)

We were only doing this if a dialog was showing. But if it's the target, then
a dialog is not relevant, we should always recompute the focus cycling. This
avoids a crash when not using a dialog and closing a window during cycling.
---

diff --git a/openbox/focus_cycle.c b/openbox/focus_cycle.c
index 6d4cc2a5..93a9a993 100644
--- a/openbox/focus_cycle.c
+++ b/openbox/focus_cycle.c
@@ -74,7 +74,7 @@ void focus_cycle_addremove(ObClient *c, gboolean redraw)
         gboolean v, s;
 
         v = focus_cycle_valid(c);
-        s = focus_cycle_popup_is_showing(c);
+        s = focus_cycle_popup_is_showing(c) || c == focus_cycle_target;
 
         if (v != s)
             focus_cycle_reorder();