]> Dogcows Code - chaz/openbox/blobdiff - openbox/event.c
bump version
[chaz/openbox] / openbox / event.c
index 6b372a85ca42a6c0ae3d06f1b5e18ebe9fc31aa0..ed05a58d8c58e28cb1809dc2f433010c856f9e45 100644 (file)
@@ -77,6 +77,12 @@ typedef struct
     Time time;
 } ObFocusDelayData;
 
+typedef struct
+{
+    gulong start; /* inclusive */
+    gulong end;   /* inclusive */
+} ObSerialRange;
+
 static void event_process(const XEvent *e, gpointer data);
 static void event_handle_root(XEvent *e);
 static gboolean event_handle_menu_keyboard(XEvent *e);
@@ -96,8 +102,9 @@ static void focus_delay_client_dest(ObClient *client, gpointer data);
 /* The time for the current event being processed */
 Time event_curtime = CurrentTime;
 
-static guint ignore_enter_focus = 0;
 static gboolean focus_left_screen = FALSE;
+/*! A list of ObSerialRanges which are to be ignored for mouse enter events */
+static GSList *ignore_serials = NULL;
 
 #ifdef USE_SM
 static void ice_handler(gint fd, gpointer conn)
@@ -492,7 +499,7 @@ static void event_process(const XEvent *ec, gpointer data)
 
             focus_left_screen = FALSE;
 
-            focus_fallback(FALSE, FALSE);
+            focus_fallback(FALSE, config_focus_under_mouse, TRUE);
 
             /* We don't get a FocusOut for this case, because it's just moving
                from our Inferior up to us. This happens when iconifying a
@@ -544,7 +551,7 @@ static void event_process(const XEvent *ec, gpointer data)
                 */
 
                 if (!focus_left_screen)
-                    focus_fallback(FALSE, FALSE);
+                    focus_fallback(FALSE, config_focus_under_mouse, TRUE);
             }
         }
         else if (!client)
@@ -600,7 +607,7 @@ static void event_process(const XEvent *ec, gpointer data)
                 ob_debug_type(OB_DEBUG_FOCUS,
                               "Focus went to an unmanaged window 0x%x !\n",
                               ce.xfocus.window);
-                focus_fallback(TRUE, FALSE);
+                focus_fallback(TRUE, config_focus_under_mouse, TRUE);
             }
         }
 
@@ -716,6 +723,7 @@ static void event_handle_root(XEvent *e)
         } else if (msgtype == prop_atoms.net_showing_desktop) {
             screen_show_desktop(e->xclient.data.l[0] != 0, NULL);
         } else if (msgtype == prop_atoms.ob_control) {
+            ob_debug("OB_CONTROL: %d\n", e->xclient.data.l[0]);
             if (e->xclient.data.l[0] == 1)
                 ob_reconfigure();
             else if (e->xclient.data.l[0] == 2)
@@ -723,8 +731,10 @@ static void event_handle_root(XEvent *e)
         }
         break;
     case PropertyNotify:
-        if (e->xproperty.atom == prop_atoms.net_desktop_names)
+        if (e->xproperty.atom == prop_atoms.net_desktop_names) {
+            ob_debug("UPDATE DESKTOP NAMES\n");
             screen_update_desktop_names();
+        }
         else if (e->xproperty.atom == prop_atoms.net_desktop_layout)
             screen_update_layout();
         break;
@@ -940,7 +950,7 @@ static void event_handle_client(ObClient *client, XEvent *e)
                corresponding enter events. Pretend like the animating window
                doesn't even exist..! */
             if (frame_iconify_animating(client->frame))
-                event_ignore_all_queued_enters();
+                event_end_ignore_all_enters(event_start_ignore_all_enters());
 
             ob_debug_type(OB_DEBUG_FOCUS,
                           "%sNotify mode %d detail %d on %lx\n",
@@ -1057,6 +1067,7 @@ static void event_handle_client(ObClient *client, XEvent *e)
 
         if (e->xconfigurerequest.value_mask & CWStackMode) {
             ObClient *sibling = NULL;
+            gulong ignore_start;
 
             /* get the sibling */
             if (e->xconfigurerequest.value_mask & CWSibling) {
@@ -1068,8 +1079,13 @@ static void event_handle_client(ObClient *client, XEvent *e)
             }
 
             /* activate it rather than just focus it */
+            if (!config_focus_under_mouse)
+                ignore_start = event_start_ignore_all_enters();
             stacking_restack_request(client, sibling,
-                                     e->xconfigurerequest.detail, TRUE);
+                                     e->xconfigurerequest.detail,
+                                     TRUE);
+            if (!config_focus_under_mouse)
+                event_end_ignore_all_enters(ignore_start);
 
             /* if a stacking change moves the window without resizing */
             move = TRUE;
@@ -1164,13 +1180,14 @@ static void event_handle_client(ObClient *client, XEvent *e)
                the window is actually being changed then configure it and
                send a configure notify to them */
             if (move || !RECT_EQUAL_DIMS(client->area, x, y, w, h)) {
+                gulong ignore_start;
+
                 ob_debug("Granting ConfigureRequest x %d y %d w %d h %d\n",
                          x, y, w, h);
+                ignore_start = event_start_ignore_all_enters();
                 client_configure(client, x, y, w, h, FALSE, TRUE);
+                event_end_ignore_all_enters(ignore_start);
             }
-
-            /* ignore enter events caused by these like ob actions do */
-            event_ignore_all_queued_enters();
         }
         break;
     }
@@ -1255,6 +1272,8 @@ static void event_handle_client(ObClient *client, XEvent *e)
                 client_set_desktop(client, (unsigned)e->xclient.data.l[0],
                                    FALSE);
         } else if (msgtype == prop_atoms.net_wm_state) {
+            gulong ignore_start;
+
             /* can't compress these */
             ob_debug("net_wm_state %s %ld %ld for 0x%lx\n",
                      (e->xclient.data.l[0] == 0 ? "Remove" :
@@ -1262,11 +1281,14 @@ static void event_handle_client(ObClient *client, XEvent *e)
                       e->xclient.data.l[0] == 2 ? "Toggle" : "INVALID"),
                      e->xclient.data.l[1], e->xclient.data.l[2],
                      client->window);
-            client_set_state(client, e->xclient.data.l[0],
-                             e->xclient.data.l[1], e->xclient.data.l[2]);
 
             /* ignore enter events caused by these like ob actions do */
-            event_ignore_all_queued_enters();
+            if (!config_focus_under_mouse)
+                ignore_start = event_start_ignore_all_enters();
+            client_set_state(client, e->xclient.data.l[0],
+                             e->xclient.data.l[1], e->xclient.data.l[2]);
+            if (!config_focus_under_mouse)
+                event_end_ignore_all_enters(ignore_start);
         } else if (msgtype == prop_atoms.net_close_window) {
             ob_debug("net_close_window for 0x%lx\n", client->window);
             client_close(client);
@@ -1327,6 +1349,7 @@ static void event_handle_client(ObClient *client, XEvent *e)
                 moveresize_end(TRUE);
         } else if (msgtype == prop_atoms.net_moveresize_window) {
             gint ograv, x, y, w, h;
+            gulong ignore_start;
 
             ograv = client->gravity;
 
@@ -1371,12 +1394,12 @@ static void event_handle_client(ObClient *client, XEvent *e)
 
             client_find_onscreen(client, &x, &y, w, h, FALSE);
 
+            /* ignore enter events caused by these like ob actions do */
+            ignore_start = event_start_ignore_all_enters();
             client_configure(client, x, y, w, h, FALSE, TRUE);
+            event_end_ignore_all_enters(ignore_start);
 
             client->gravity = ograv;
-
-            /* ignore enter events caused by these like ob actions do */
-            event_ignore_all_queued_enters();
         } else if (msgtype == prop_atoms.net_restack_window) {
             if (e->xclient.data.l[0] != 2) {
                 ob_debug_type(OB_DEBUG_APP_BUGS,
@@ -1405,9 +1428,16 @@ static void event_handle_client(ObClient *client, XEvent *e)
                     e->xclient.data.l[2] == TopIf ||
                     e->xclient.data.l[2] == Opposite)
                 {
+                    gulong ignore_start;
+
+                    if (!config_focus_under_mouse)
+                        ignore_start = event_start_ignore_all_enters();
                     /* just raise, don't activate */
                     stacking_restack_request(client, sibling,
                                              e->xclient.data.l[2], FALSE);
+                    if (!config_focus_under_mouse)
+                        event_end_ignore_all_enters(ignore_start);
+
                     /* send a synthetic ConfigureNotify, cuz this is supposed
                        to be like a ConfigureRequest. */
                     client_reconfigure(client);
@@ -1831,49 +1861,49 @@ void event_halt_focus_delay()
     ob_main_loop_timeout_remove(ob_main_loop, focus_delay_func);
 }
 
-static Bool event_look_for_enters(Display *d, XEvent *e, XPointer arg)
+gulong event_start_ignore_all_enters()
 {
-    if (e->type == EnterNotify &&
-        /* these types aren't used for focusing */
-        !(e->xcrossing.mode == NotifyGrab ||
-          e->xcrossing.mode == NotifyUngrab ||
-          e->xcrossing.detail == NotifyInferior))
-    {
-        ObWindow *win;
-
-        /* found an enter for that leave, ignore it if it's going to
-           another window */
-        win = g_hash_table_lookup(window_map, &e->xany.window);
-        if (win && WINDOW_IS_CLIENT(win))
-            ++ignore_enter_focus;
-    }
-    return False; /* don't disrupt the queue order, just count them */
+    XSync(ob_display, FALSE);
+    return LastKnownRequestProcessed(ob_display);
 }
 
-void event_ignore_all_queued_enters()
+void event_end_ignore_all_enters(gulong start)
 {
-    XEvent e;
+    ObSerialRange *r;
 
+    g_assert(start != 0);
     XSync(ob_display, FALSE);
 
-    /* count the events without disrupting them */
-    ignore_enter_focus = 0;
-    XCheckIfEvent(ob_display, &e, event_look_for_enters, NULL);
+    r = g_new(ObSerialRange, 1);
+    r->start = start;
+    r->end = LastKnownRequestProcessed(ob_display);
+    ignore_serials = g_slist_prepend(ignore_serials, r);
+
+    /* increment the serial so we don't ignore events we weren't meant to */
+    XSync(ob_display, FALSE);
 }
 
 static gboolean is_enter_focus_event_ignored(XEvent *e)
 {
+    GSList *it, *next;
+
     g_assert(e->type == EnterNotify &&
              !(e->xcrossing.mode == NotifyGrab ||
                e->xcrossing.mode == NotifyUngrab ||
                e->xcrossing.detail == NotifyInferior));
 
-    ob_debug_type(OB_DEBUG_FOCUS, "# enters ignored: %d\n",
-                  ignore_enter_focus);
+    for (it = ignore_serials; it; it = next) {
+        ObSerialRange *r = it->data;
 
-    if (ignore_enter_focus) {
-        --ignore_enter_focus;
-        return TRUE;
+        next = g_slist_next(it);
+
+        if ((glong)(e->xany.serial - r->end) > 0) {
+            /* past the end */
+            ignore_serials = g_slist_delete_link(ignore_serials, it);
+            g_free(r);
+        }
+        else if ((glong)(e->xany.serial - r->start) >= 0)
+            return TRUE;
     }
     return FALSE;
 }
This page took 0.029965 seconds and 4 git commands to generate.