Alarm trigger queueing for the GUI part.

2000-12-21  Federico Mena Quintero  <federico@helixcode.com>

	Alarm trigger queueing for the GUI part.

	* gui/alarm-notify.[ch]: New files with the high-level alarm
	notification system; mostly moved over from gnome-cal.c.  The
	low-level timer stuff is still in alarm.[ch].

	* gui/alarm-notify.c (alarm_notify_init): New function to
	initialize the alarm notification system.
	(alarm_notify_done): New function to shut down the alarm
	notification system.
	(alarm_notify_add_client): New function to start monitoring a
	calendar client for alarm notification.
	(alarm_notify_remove_client): New function to stop monitoring a
	client.

	* gui/alarm.h (AlarmDestroyNotify): Also pass in the alarm ID so
	the callback may know which ID is being destroyed.

	* gui/alarm.c (clear_itimer): New function.
	(pop_alarm): Use clear_itimer().
	(alarm_done): New function to shut down the timer system.
	(alarm_add): Add some preconditions.  Do not call the destroy
	notification function if we could not create the alarm.
	(alarm_ready): Pass the alarm ID to the destroy notify function.
	(alarm_remove): Likewise.  Also, add some preconditions.

	* gui/gnome-cal.c: Removed the alarm notification functions from
	here since they are now in alarm-notify.c.
	(gnome_calendar_construct): Register the client with
	alarm_notify_add_client().
	(gnome_calendar_destroy): Use alarm_notify_remove_client() to
	unregister the client.
	(obj_updated_cb): Do not do any alarm-related stuff.
	(obj_removed_cb): Likewise.

	* gui/main.c (main): Shut down the alarm timer system.
	(main): Initialize and shut down the alarm notification system.

	* gui/Makefile.am (evolution_calendar_SOURCES): Added
	alarm-notify.[ch] to the list of sources.

	* gui/calendar-model.c (calendar_model_set_cal_client): Only
	connect to the "cal_loaded" signal if the client is not already
	loaded.

	* gui/e-day-view.c (e_day_view_set_cal_client): Likewise.

	* gui/e-week-view.c (e_week_view_set_cal_client): Likewise.

	* gui/e-itip-control.c (update_calendar): Connect to "cal_loaded"
	before issuing the load request.

svn path=/trunk/; revision=7130

1 files changed, 71 insertions, 20 deletions

diff --git a/calendar/gui/alarm-notify/alarm.c b/calendar/gui/alarm-notify/alarm.c
index 087ec518bc..a0fe90f337 100644
--- a/calendar/gui/alarm-notify/alarm.c
+++ b/calendar/gui/alarm-notify/alarm.c
@@ -1,4 +1,4 @@
-/* Evolution calendar - alarm notification support
+/* Evolution calendar - Low-level alarm timer mechanism
  *
  * Copyright (C) 2000 Helix Code, Inc.
  *
@@ -21,15 +21,19 @@
  */
 
 #include <config.h>
+#include <unistd.h>
 #include <time.h>
-#include <gnome.h>
 #include <fcntl.h>
 #include <signal.h>
 #include <sys/time.h>
+#include <gdk/gdk.h>
 #include "alarm.h"
 
 
 
+/* Whether the timer system has been initialized */
+static gboolean alarm_inited;
+
 /* The pipes used to notify about an alarm */
 static int alarm_pipes [2];
 
@@ -72,6 +76,13 @@ setup_itimer (time_t diff)
 	return (v == 0) ? TRUE : FALSE;
 }
 
+/* Clears the itimer we have pending */
+static gboolean
+clear_itimer (void)
+{
+	return setup_itimer (0);
+}
+
 /* Removes the head alarm, returns it, and schedules the next alarm in the
  * queue.
  */
@@ -106,20 +117,10 @@ pop_alarm (void)
 			 * will fail?
 			 */
 		}
-	} else {
-		struct itimerval itimer;
-		int v;
-
-		itimer.it_interval.tv_sec = 0;
-		itimer.it_interval.tv_usec = 0;
-		itimer.it_value.tv_sec = 0;
-		itimer.it_value.tv_usec = 0;
-
-		v = setitimer (ITIMER_REAL, &itimer, NULL);
-		if (v != 0)
+	} else
+		if (!clear_itimer ())
 			g_message ("pop_alarm(): Could not clear the timer!  "
 				   "Weird things may happen.");
-	}
 
 	return ar;
 }
@@ -144,7 +145,7 @@ alarm_ready (gpointer data, gint fd, GdkInputCondition cond)
 	(* ar->alarm_fn) (ar, ar->trigger, ar->data);
 
 	if (ar->destroy_notify_fn)
-		(* ar->destroy_notify_fn) (ar->data);
+		(* ar->destroy_notify_fn) (ar, ar->data);
 
 	g_free (ar);
 }
@@ -216,6 +217,10 @@ alarm_add (time_t trigger, AlarmFunction alarm_fn, gpointer data,
 	time_t now;
 	AlarmRecord *ar;
 
+	g_return_val_if_fail (alarm_inited, NULL);
+	g_return_val_if_fail (trigger != -1, NULL);
+	g_return_val_if_fail (alarm_fn != NULL, NULL);
+
 	now = time (NULL);
 	if (trigger < now)
 		return NULL;
@@ -229,9 +234,6 @@ alarm_add (time_t trigger, AlarmFunction alarm_fn, gpointer data,
 	g_print ("alarm_add(): Adding alarm for %s\n", ctime (&trigger));
 
 	if (!queue_alarm (now, ar)) {
-		if (ar->destroy_notify_fn)
-			(* ar->destroy_notify_fn) (ar->data);
-
 		g_free (ar);
 		ar = NULL;
 	}
@@ -252,6 +254,7 @@ alarm_remove (gpointer alarm)
 	AlarmRecord *old_head;
 	GList *l;
 
+	g_return_if_fail (alarm_inited);
 	g_return_if_fail (alarm != NULL);
 
 	ar = alarm;
@@ -272,7 +275,7 @@ alarm_remove (gpointer alarm)
 	}
 
 	if (ar->destroy_notify_fn)
-		(* ar->destroy_notify_fn) (ar->data);
+		(* ar->destroy_notify_fn) (ar, ar->data);
 
 	g_free (ar);
 }
@@ -280,7 +283,7 @@ alarm_remove (gpointer alarm)
 /**
  * alarm_init:
  *
- * Initializes the alarm notification system.  This must be called near the
+ * Initializes the alarm timer mechanism.  This must be called near the
  * beginning of the program.
  **/
 void
@@ -289,6 +292,8 @@ alarm_init (void)
 	struct sigaction sa;
 	int flags;
 
+	g_return_if_fail (alarm_inited == FALSE);
+
 	pipe (alarm_pipes);
 
 	/* set non blocking mode */
@@ -302,4 +307,50 @@ alarm_init (void)
 	sigemptyset (&sa.sa_mask);
 	sa.sa_flags = SA_RESTART;
 	sigaction (SIGALRM, &sa, NULL);
+
+	alarm_inited = TRUE;
+}
+
+/**
+ * alarm_done:
+ * 
+ * Terminates the alarm timer mechanism.  This should be called at the end of
+ * the program.
+ **/
+void
+alarm_done (void)
+{
+	GList *l;
+
+	g_return_if_fail (alarm_inited);
+
+	if (!clear_itimer ())
+		g_message ("alarm_done(): Could not clear the timer!  "
+			   "Weird things may happen.");
+
+	for (l = alarms; l; l = l->next) {
+		AlarmRecord *ar;
+
+		ar = l->data;
+
+		if (ar->destroy_notify_fn)
+			(* ar->destroy_notify_fn) (ar, ar->data);
+
+		g_free (ar);
+	}
+
+	g_list_free (alarms);
+	alarms = NULL;
+
+	if (close (alarm_pipes[0]) != 0)
+		g_message ("alarm_done(): Could not close the input pipe for notification");
+
+	alarm_pipes[0] = -1;
+
+	if (close (alarm_pipes[1]) != 0)
+		g_message ("alarm_done(): Could not close the output pipe for notification");
+
+	alarm_pipes[1] = -1;
+
+	alarm_inited = FALSE;
 }