Fix shadowed variables
[mikachu/openbox.git] / openbox / mainloop.c
index 9ee2075..7c6a956 100644 (file)
@@ -1,4 +1,24 @@
+/* -*- indent-tabs-mode: nil; tab-width: 4; c-basic-offset: 4; -*-
+
+   mainloop.c for the Openbox window manager
+   Copyright (c) 2006        Mikael Magnusson
+   Copyright (c) 2003-2007   Dana Jansens
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
+
+   See the COPYING file for a copy of the GNU General Public License.
+*/
+
 #include "mainloop.h"
+#include "event.h"
 
 #include <stdio.h>
 #include <stdlib.h>
@@ -75,6 +95,7 @@ struct _ObMainLoopTimer
     gulong delay;
     GSourceFunc func;
     gpointer data;
+    GEqualFunc equal;
     GDestroyNotify destroy;
 
     /* The timer needs to be freed */
@@ -195,8 +216,6 @@ void ob_main_loop_destroy(ObMainLoop *loop)
 
         /* only do this if we're the last loop destroyed */
         if (!all_loops) {
-            guint i;
-
             /* grab all the signals that cause core dumps */
             for (i = 0; i < NUM_CORE_SIGNALS; ++i) {
                 if (all_signals[core_signals[i]].installed) {
@@ -244,7 +263,7 @@ void ob_main_loop_run(ObMainLoop *loop)
             for (i = 0; i < NUM_SIGNALS; ++i) {
                 while (loop->signals_fired[i]) {
                     for (it = loop->signal_handlers[i];
-                         it; it = g_slist_next(it)) {
+                            it; it = g_slist_next(it)) {
                         ObMainLoopSignalHandlerType *h = it->data;
                         h->func(i, h->data);
                     }
@@ -262,11 +281,12 @@ void ob_main_loop_run(ObMainLoop *loop)
                     ObMainLoopXHandlerType *h = it->data;
                     h->func(&e, h->data);
                 }
-            } while (XPending(loop->display));
+            } while (XPending(loop->display) && loop->run);
         } else {
             /* this only runs if there were no x events received */
 
             timer_dispatch(loop, (GTimeVal**)&wait);
+
             selset = loop->fd_set;
             /* there is a small race condition here. if a signal occurs
                between this if() and the select() then we will not process
@@ -340,7 +360,8 @@ static void sighandler(gint sig)
             /* XXX special case for signals that default to core dump.
                but throw some helpful output here... */
 
-            fprintf(stderr, "Fuck yah. Core dump. (Signal=%d)\n", sig);
+            fprintf(stderr, "How are you gentlemen? All your base are"
+                    " belong to us. (Openbox received signal %d)\n", sig);
 
             /* die with a core dump */
             abort();
@@ -476,39 +497,41 @@ void ob_main_loop_fd_remove(ObMainLoop *loop,
 #define NEAREST_TIMEOUT(loop) \
     (((ObMainLoopTimer*)(loop)->timers->data)->timeout)
 
-static long timecompare(GTimeVal *a, GTimeVal *b)
+static glong timecompare(GTimeVal *a, GTimeVal *b)
 {
-    long r;
+    glong r;
 
     if ((r = b->tv_sec - a->tv_sec)) return r;
     return b->tv_usec - a->tv_usec;
-    
+
 }
 
 static void insert_timer(ObMainLoop *loop, ObMainLoopTimer *ins)
 {
     GSList *it;
     for (it = loop->timers; it; it = g_slist_next(it)) {
-       ObMainLoopTimer *t = it->data;
-        if (timecompare(&ins->timeout, &t->timeout) <= 0) {
-           loop->timers = g_slist_insert_before(loop->timers, it, ins);
-           break;
-       }
+        ObMainLoopTimer *t = it->data;
+        if (timecompare(&ins->timeout, &t->timeout) >= 0) {
+            loop->timers = g_slist_insert_before(loop->timers, it, ins);
+            break;
+        }
     }
     if (it == NULL) /* didnt fit anywhere in the list */
-       loop->timers = g_slist_append(loop->timers, ins);
+        loop->timers = g_slist_append(loop->timers, ins);
 }
 
 void ob_main_loop_timeout_add(ObMainLoop *loop,
                               gulong microseconds,
                               GSourceFunc handler,
                               gpointer data,
+                              GEqualFunc cmp,
                               GDestroyNotify notify)
 {
     ObMainLoopTimer *t = g_new(ObMainLoopTimer, 1);
     t->delay = microseconds;
     t->func = handler;
     t->data = data;
+    t->equal = cmp;
     t->destroy = notify;
     t->del_me = FALSE;
     g_get_current_time(&loop->now);
@@ -530,16 +553,18 @@ void ob_main_loop_timeout_remove(ObMainLoop *loop,
     }
 }
 
-void ob_main_loop_timeout_remove_data(ObMainLoop *loop,
-                                      GSourceFunc handler,
-                                      gpointer data)
+void ob_main_loop_timeout_remove_data(ObMainLoop *loop, GSourceFunc handler,
+                                      gpointer data, gboolean cancel_dest)
 {
     GSList *it;
 
     for (it = loop->timers; it; it = g_slist_next(it)) {
         ObMainLoopTimer *t = it->data;
-        if (t->func == handler && t->data == data)
+        if (t->func == handler && t->equal(t->data, data)) {
             t->del_me = TRUE;
+            if (cancel_dest)
+                t->destroy = NULL;
+        }
     }
 }
 
@@ -573,34 +598,36 @@ static void timer_dispatch(ObMainLoop *loop, GTimeVal **wait)
     g_get_current_time(&loop->now);
 
     for (it = loop->timers; it; it = next) {
-       ObMainLoopTimer *curr;
-        
+        ObMainLoopTimer *curr;
+
         next = g_slist_next(it);
 
-       curr = it->data;
+        curr = it->data;
 
-       /* since timer_stop doesn't actually free the timer, we have to do our
-          real freeing in here.
-       */
-       if (curr->del_me) {
+        /* since timer_stop doesn't actually free the timer, we have to do our
+           real freeing in here.
+        */
+        if (curr->del_me) {
             /* delete the top */
-           loop->timers = g_slist_delete_link(loop->timers, it); 
-           g_free(curr);
-           continue;
-       }
-       
-       /* the queue is sorted, so if this timer shouldn't fire, none are 
-          ready */
-        if (timecompare(&NEAREST_TIMEOUT(loop), &loop->now) <= 0)
-           break;
-
-       /* we set the last fired time to delay msec after the previous firing,
-          then re-insert.  timers maintain their order and may trigger more
-          than once if they've waited more than one delay's worth of time.
-       */
-       loop->timers = g_slist_delete_link(loop->timers, it);
-       g_time_val_add(&curr->last, curr->delay);
-       if (curr->func(curr->data)) {
+            loop->timers = g_slist_delete_link(loop->timers, it);
+            if (curr->destroy)
+                curr->destroy(curr->data);
+            g_free(curr);
+            continue;
+        }
+
+        /* the queue is sorted, so if this timer shouldn't fire, none are
+           ready */
+        if (timecompare(&NEAREST_TIMEOUT(loop), &loop->now) < 0)
+            break;
+
+        /* we set the last fired time to delay msec after the previous firing,
+           then re-insert.  timers maintain their order and may trigger more
+           than once if they've waited more than one delay's worth of time.
+        */
+        loop->timers = g_slist_delete_link(loop->timers, it);
+        g_time_val_add(&curr->last, curr->delay);
+        if (curr->func(curr->data)) {
             g_time_val_add(&curr->timeout, curr->delay);
             insert_timer(loop, curr);
         } else {
@@ -613,13 +640,13 @@ static void timer_dispatch(ObMainLoop *loop, GTimeVal **wait)
     }
 
     if (fired) {
-       /* if at least one timer fires, then don't wait on X events, as there
-          may already be some in the queue from the timer callbacks.
-       */
-       loop->ret_wait.tv_sec = loop->ret_wait.tv_usec = 0;
-       *wait = &loop->ret_wait;
+        /* if at least one timer fires, then don't wait on X events, as there
+           may already be some in the queue from the timer callbacks.
+        */
+        loop->ret_wait.tv_sec = loop->ret_wait.tv_usec = 0;
+        *wait = &loop->ret_wait;
     } else if (nearest_timeout_wait(loop, &loop->ret_wait))
-       *wait = &loop->ret_wait;
+        *wait = &loop->ret_wait;
     else
-       *wait = NULL;
+        *wait = NULL;
 }