Revert "add XFlush to g_timeout callbacks"
authorDana Jansens <danakj@orodu.net>
Sat, 10 Jul 2010 03:58:21 +0000 (20:58 -0700)
committerMikael Magnusson <mikachu@gmail.com>
Thu, 16 Sep 2010 15:24:12 +0000 (17:24 +0200)
This reverts commit aa3685d16af4a565eecdc39047ee8d140ef5cd99.

Mika's commit using a GSource to handle X Events fixes this problem, so this
commit is not needed

openbox/dock.c
openbox/event.c
openbox/frame.c
openbox/keyboard.c
openbox/menuframe.c
openbox/moveresize.c
openbox/ping.c
openbox/popup.c
openbox/screen.c
openbox/startupnotify.c

index b8a5ff5..06d2280 100644 (file)
@@ -632,7 +632,6 @@ static gboolean hide_timeout(gpointer data)
 
     hide_timeout_id = 0;
 
 
     hide_timeout_id = 0;
 
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
@@ -644,7 +643,6 @@ static gboolean show_timeout(gpointer data)
 
     show_timeout_id = 0;
 
 
     show_timeout_id = 0;
 
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
index eb69d09..b59707b 100644 (file)
@@ -2073,7 +2073,6 @@ static gboolean focus_delay_func(gpointer data)
     if (client_focus(d->client) && config_focus_raise)
         stacking_raise(CLIENT_AS_WINDOW(d->client));
     event_curtime = old;
     if (client_focus(d->client) && config_focus_raise)
         stacking_raise(CLIENT_AS_WINDOW(d->client));
     event_curtime = old;
-    XFlush(obt_display);
     return FALSE; /* no repeat */
 }
 
     return FALSE; /* no repeat */
 }
 
@@ -2086,7 +2085,6 @@ static gboolean unfocus_delay_func(gpointer data)
     event_curserial = d->serial;
     focus_nothing();
     event_curtime = old;
     event_curserial = d->serial;
     focus_nothing();
     event_curtime = old;
-    XFlush(obt_display);
     return FALSE; /* no repeat */
 }
 
     return FALSE; /* no repeat */
 }
 
index bd9527b..8a9a5a6 100644 (file)
@@ -1672,7 +1672,6 @@ static gboolean flash_timeout(gpointer data)
         self->focused = FALSE;
     }
 
         self->focused = FALSE;
     }
 
-    XFlush(obt_display);
     return TRUE; /* go again */
 }
 
     return TRUE; /* go again */
 }
 
@@ -1770,11 +1769,11 @@ static gboolean frame_animate_iconify(gpointer p)
     }
 
     XMoveResizeWindow(obt_display, self->window, x, y, w, h);
     }
 
     XMoveResizeWindow(obt_display, self->window, x, y, w, h);
+    XFlush(obt_display);
 
     if (time == 0)
         frame_end_iconify_animation(self);
 
 
     if (time == 0)
         frame_end_iconify_animation(self);
 
-    XFlush(obt_display);
     return time > 0; /* repeat until we're out of time */
 }
 
     return time > 0; /* repeat until we're out of time */
 }
 
index 95433ac..5b8016d 100644 (file)
@@ -66,7 +66,6 @@ static void grab_keys(gboolean grab)
 static gboolean chain_timeout(gpointer data)
 {
     keyboard_reset_chains(0);
 static gboolean chain_timeout(gpointer data)
 {
     keyboard_reset_chains(0);
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
index fafc0f7..5708cdf 100644 (file)
@@ -1158,7 +1158,6 @@ static gboolean submenu_show_timeout(gpointer data)
 {
     g_assert(menu_frame_visible);
     menu_entry_frame_show_submenu((ObMenuEntryFrame*)data);
 {
     g_assert(menu_frame_visible);
     menu_entry_frame_show_submenu((ObMenuEntryFrame*)data);
-    XFlush(obt_display);
     return FALSE;
 }
 
     return FALSE;
 }
 
@@ -1166,7 +1165,6 @@ static gboolean submenu_hide_timeout(gpointer data)
 {
     g_assert(menu_frame_visible);
     menu_frame_hide((ObMenuFrame*)data);
 {
     g_assert(menu_frame_visible);
     menu_frame_hide((ObMenuFrame*)data);
-    XFlush(obt_display);
     return FALSE;
 }
 
     return FALSE;
 }
 
index 13f39df..2f68395 100644 (file)
@@ -462,8 +462,6 @@ static gboolean sync_timeout_func(gpointer data)
     ++waiting_for_sync; /* we timed out waiting for our sync... */
     do_resize(); /* ...so let any pending resizes through */
 
     ++waiting_for_sync; /* we timed out waiting for our sync... */
     do_resize(); /* ...so let any pending resizes through */
 
-    XFlush(obt_display);
-
     if (waiting_for_sync > SYNC_TIMEOUTS) {
         sync_timer = 0;
         return FALSE; /* don't repeat */
     if (waiting_for_sync > SYNC_TIMEOUTS) {
         sync_timer = 0;
         return FALSE; /* don't repeat */
@@ -618,7 +616,6 @@ static gboolean edge_warp_delay_func(gpointer data)
     }
     edge_warp_odd = !edge_warp_odd;
 
     }
     edge_warp_odd = !edge_warp_odd;
 
-    XFlush(obt_display);
     return TRUE; /* do repeat ! */
 }
 
     return TRUE; /* do repeat ! */
 }
 
index 90c4182..7cb4766 100644 (file)
@@ -148,7 +148,6 @@ static gboolean ping_timeout(gpointer data)
 
     ++t->waiting;
 
 
     ++t->waiting;
 
-    XFlush(obt_display);
     return TRUE; /* repeat */
 }
 
     return TRUE; /* repeat */
 }
 
index 6dd0e30..2a83d7b 100644 (file)
@@ -148,7 +148,6 @@ static gboolean popup_show_timeout(gpointer data)
     self->delay_mapped = FALSE;
     self->delay_timer = 0;
 
     self->delay_mapped = FALSE;
     self->delay_timer = 0;
 
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
index e3ad861..353d2de 100644 (file)
@@ -940,7 +940,6 @@ static gboolean hide_desktop_popup_func(gpointer data)
 {
     pager_popup_hide(desktop_popup);
     desktop_popup_timer = 0;
 {
     pager_popup_hide(desktop_popup);
     desktop_popup_timer = 0;
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
index 06fbb6c..16654cf 100644 (file)
@@ -116,7 +116,6 @@ static gboolean sn_wait_timeout(gpointer data)
     SnStartupSequence *seq = data;
     sn_waits = g_slist_remove(sn_waits, seq);
     screen_set_root_cursor();
     SnStartupSequence *seq = data;
     sn_waits = g_slist_remove(sn_waits, seq);
     screen_set_root_cursor();
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }
 
@@ -229,7 +228,6 @@ static gboolean sn_launch_wait_timeout(gpointer data)
 {
     SnLauncherContext *sn = data;
     sn_launcher_context_complete(sn);
 {
     SnLauncherContext *sn = data;
     sn_launcher_context_complete(sn);
-    XFlush(obt_display);
     return FALSE; /* don't repeat */
 }
 
     return FALSE; /* don't repeat */
 }