X-Git-Url: https://git.brokenzipper.com/gitweb?a=blobdiff_plain;ds=sidebyside;f=openbox%2Fevent.c;h=096f6272bd608a1a7bf8d99218df6476c69bd872;hb=dc43b74abdfdf9f8d847ba562accd8ea6dcd52b1;hp=6f819cc1f4c0838524d03fdd590d416e95fb3e23;hpb=7c1fb96bd33a97c9be55e5fcfae6cf7d625acbb6;p=chaz%2Fopenbox

diff --git a/openbox/event.c b/openbox/event.c
index 6f819cc1..096f6272 100644
--- a/openbox/event.c
+++ b/openbox/event.c
@@ -71,7 +71,6 @@ typedef struct
 } ObFocusDelayData;
 
 static void event_process(const XEvent *e, gpointer data);
-static void event_client_dest(ObClient *client, gpointer data);
 static void event_handle_root(XEvent *e);
 static void event_handle_menu(XEvent *e);
 static void event_handle_dock(ObDock *s, XEvent *e);
@@ -164,7 +163,6 @@ void event_startup(gboolean reconfig)
 #endif
 
     client_add_destructor(focus_delay_client_dest, NULL);
-    client_add_destructor(event_client_dest, NULL);
 }
 
 void event_shutdown(gboolean reconfig)
@@ -176,7 +174,6 @@ void event_shutdown(gboolean reconfig)
 #endif
 
     client_remove_destructor(focus_delay_client_dest);
-    client_remove_destructor(event_client_dest);
     XFreeModifiermap(modmap);
 }
 
@@ -395,11 +392,8 @@ static gboolean event_ignore(XEvent *e, ObClient *client)
     switch(e->type) {
     case FocusIn:
     case FocusOut:
-        if (!wanted_focusevent(e)) {
-            ob_debug_type(OB_DEBUG_FOCUS, "focus event ignored\n");
+        if (!wanted_focusevent(e))
             return TRUE;
-        }
-        ob_debug_type(OB_DEBUG_FOCUS, "focus event used;\n");
         break;
     }
     return FALSE;
@@ -442,29 +436,6 @@ static void event_process(const XEvent *ec, gpointer data)
             }
         }
 
-    if (e->type == FocusIn || e->type == FocusOut) {
-        gint mode = e->xfocus.mode;
-        gint detail = e->xfocus.detail;
-        Window window = e->xfocus.window;
-        if (detail == NotifyVirtual) {
-            ob_debug_type(OB_DEBUG_FOCUS,
-                          "FOCUS %s NOTIFY VIRTUAL window 0x%x\n",
-                          (e->type == FocusIn ? "IN" : "OUT"), window);
-        }
-
-        else if (detail == NotifyNonlinearVirtual) {
-            ob_debug_type(OB_DEBUG_FOCUS,
-                          "FOCUS %s NOTIFY NONLINVIRTUAL window 0x%x\n",
-                          (e->type == FocusIn ? "IN" : "OUT"), window);
-        }
-
-        else
-            ob_debug_type(OB_DEBUG_FOCUS,
-                          "UNKNOWN FOCUS %s (d %d, m %d) window 0x%x\n",
-                          (e->type == FocusIn ? "IN" : "OUT"),
-                          detail, mode, window);
-    }
-
     event_set_curtime(e);
     event_hack_mods(e);
     if (event_ignore(e, client)) {
@@ -485,18 +456,22 @@ static void event_process(const XEvent *ec, gpointer data)
         if (e->xfocus.detail == NotifyPointerRoot ||
                    e->xfocus.detail == NotifyDetailNone) {
             ob_debug_type(OB_DEBUG_FOCUS, "Focus went to root\n");
-            /* Focus has been reverted to the root window or nothing, so fall
-               back to something other than the window which just had it. */
-            focus_fallback(FALSE);
+            /* Focus has been reverted to the root window or nothing
+               FocusOut events come after UnmapNotify, so we don't need to
+               worry about focusing an invalid window
+             */
+            focus_fallback(TRUE);
         } else if (e->xfocus.detail == NotifyInferior) {
             ob_debug_type(OB_DEBUG_FOCUS, "Focus went to parent\n");
             /* Focus has been reverted to parent, which is our frame window,
-               or the root window, so fall back to something other than the
-               window which had it. */
-            focus_fallback(FALSE);
+               or the root window
+               FocusOut events come after UnmapNotify, so we don't need to
+               worry about focusing an invalid window
+            */
+            focus_fallback(TRUE);
         } else if (client && client != focus_client) {
-            focus_set_client(client);
             frame_adjust_focus(client->frame, TRUE);
+            focus_set_client(client);
             client_calc_layer(client);
         }
     } else if (e->type == FocusOut) {
@@ -510,6 +485,8 @@ static void event_process(const XEvent *ec, gpointer data)
             /* There is no FocusIn, this means focus went to a window that
                is not being managed, or a window on another screen. */
             ob_debug_type(OB_DEBUG_FOCUS, "Focus went to a black hole !\n");
+            /* nothing is focused */
+            focus_set_client(NULL);
         } else if (ce.xany.window == e->xany.window) {
             /* If focus didn't actually move anywhere, there is nothing to do*/
             nomove = TRUE;
@@ -528,9 +505,8 @@ static void event_process(const XEvent *ec, gpointer data)
         }
 
         if (client && !nomove) {
-            /* This client is no longer focused, so show that */
-            focus_hilite = NULL;
             frame_adjust_focus(client->frame, FALSE);
+            /* focus_set_client has already been called for sure */
             client_calc_layer(client);
         }
     } else if (group)
@@ -594,8 +570,7 @@ static void event_process(const XEvent *ec, gpointer data)
                     mouse_event(client, e);
                 } else if (e->type == KeyPress) {
                     keyboard_event((focus_cycle_target ? focus_cycle_target :
-                                    (focus_hilite ? focus_hilite : client)),
-                                   e);
+                                    client), e);
                 }
             }
         }
@@ -1376,12 +1351,6 @@ static void focus_delay_client_dest(ObClient *client, gpointer data)
                                      client, FALSE);
 }
 
-static void event_client_dest(ObClient *client, gpointer data)
-{
-    if (client == focus_hilite)
-        focus_hilite = NULL;
-}
-
 void event_halt_focus_delay()
 {
     ob_main_loop_timeout_remove(ob_main_loop, focus_delay_func);