** Fixes bug #362638

2007-12-20  Matthew Barnes  <mbarnes@redhat.com>

	** Fixes bug #362638

	* calendar/gui/alarm-notify/alarm-notify.c:
	* calendar/gui/alarm-notify/alarm-notify.h:
	* calendar/gui/alarm-notify/alarm-queue.c:
	Rewrite message passing to use GThreadPool instead of EThread.

	* mail/mail-mt.h:
	Overhaul the message passing API:
	- Define a MailMsg type as the base message struct.
	- Define types for the various callback functions.
	- Add a priority value to each message (not yet used).
	- Add a reference count to each message.
	- Define a MailMsgInfo type for the virtual function table.
	- Record the size of message sub-types in MailMsgInfo.
	- New/changed functions:
	    mail_msg_new()   - Easier to use.
	    mail_msg_ref()   - Increase reference count.
	    mail_msg_unref() - Decrease reference count.
	    mail_msg_main_loop_push()    }
	    mail_msg_unordered_push()    } Submit MailMsgs to various
	    mail_msg_fast_ordered_push() } message-processing threads.
	    mail_msg_slow_ordered_push() }

	* mail/mail-mt.c (mail_msg_new):
	Use GSlice for memory allocation.

	* mail/mail-mt.c (mail_msg_ref), (mail_msg_unref):
	New functions increment/decrement a MailMsg's reference count.

	* mail/mail-mt.c (mail_cancel_hood_add), (mail_cancel_hook_remove):
	Convert the 'cancel_hook_list' from an  EDList to a GHookList and
	modify the API accordingly.

	* mail/mail-mt.c:
	Use GThreadPools instead of EThreads.
	Use GAsyncQueues instead of EMsgPorts.

	* mail/em-composer-utils.c:
	* mail/em-folder-browser.c:
	* mail/em-folder-properties.c:
	* mail/em-folder-tree.c:
	* mail/em-folder-utils.c:
	* mail/em-folder-view.c:
	* mail/em-format-html-print.c:
	* mail/em-format-html.c:
	* mail/em-subscribe-editor.c:
	* mail/em-sync-stream.c:
	* mail/importers/elm-importer.c:
	* mail/importers/mail-importer.c:
	* mail/importers/pine-importer.c:
	* mail/mail-component.c:
	* mail/mail-folder-cache.c:
	* mail/mail-mt.c:
	* mail/mail-ops.c:
	* mail/mail-ops.h:
	* mail/mail-send-recv.c:
	* mail/mail-session.c:
	* mail/mail-vfolder.c:
	* mail/message-list.c:
	* plugins/folder-unsubscribe/folder-unsubscribe.c:
	* plugins/groupwise-features/share-folder-common.c:
	* plugins/exchange-operations/exchange-folder.c:
	* plugins/mark-all-read/mark-all-read.c:
	* plugins/mailing-list-actions/mailing-list-actions.c:
	* plugins/itip-formatter/itip-formatter.c:
	* plugins/save-attachments/save-attachments.c:
	Use the new MailMsg API for messages.


svn path=/trunk/; revision=34730

2 files changed, 19 insertions, 21 deletions

diff --git a/plugins/itip-formatter/ChangeLog b/plugins/itip-formatter/ChangeLog
index d4132aeb8f..cc9f039bfd 100644
--- a/plugins/itip-formatter/ChangeLog
+++ b/plugins/itip-formatter/ChangeLog
@@ -1,3 +1,10 @@
+2007-12-20  Matthew Barnes  <mbarnes@redhat.com>
+
+	** Fixes part of bug #362638
+
+	* itip-formatter.c:
+	Use the new MailMsg API for messages.
+
 2007-12-04  Milan Crha  <mcrha@redhat.com>
 
 	** Fix for bug #220846
diff --git a/plugins/itip-formatter/itip-formatter.c b/plugins/itip-formatter/itip-formatter.c
index a3bb6c2e80..bd62118d40 100644
--- a/plugins/itip-formatter/itip-formatter.c
+++ b/plugins/itip-formatter/itip-formatter.c
@@ -1551,47 +1551,38 @@ extract_itip_data (FormatItipPObject *pitip, GtkContainer *container)
 }
 
 struct _opencal_msg {
-	struct _mail_msg msg;
+	MailMsg base;
 
 	char *command; /* command line to run */
 };
 
 static char *
-open_calendar_desc (struct _mail_msg *mm, int done)
+open_calendar__desc (struct _opencal_msg *m, gint complete)
 {
 	return g_strdup (_("Opening calendar"));
 }
 
 static void
-open_calendar_do (struct _mail_msg *mm)
+open_calendar__exec (struct _opencal_msg *m)
 {
-	struct _opencal_msg *m = (struct _opencal_msg *)mm;
-
 	if (!g_spawn_command_line_async (m->command, NULL)) {
 		g_warning ("Could not launch %s", m->command);
 	}
 }
 
 static void
-open_calendar_done (struct _mail_msg *mm)
-{
-	/*struct _opencal_msg *m = (struct _opencal_msg *)mm;*/
-}
-
-static void
-open_calendar_free (struct _mail_msg *mm)
+open_calendar__free (struct _opencal_msg *m)
 {
-	struct _opencal_msg *m = (struct _opencal_msg *)mm;
-
 	g_free (m->command);
 	m->command = NULL;
 }
 
-static struct _mail_msg_op open_calendar_op = {
-	open_calendar_desc,
-	open_calendar_do,
-	open_calendar_done,
-	open_calendar_free,
+static MailMsgInfo open_calendar_info = {
+	sizeof (struct _opencal_msg),
+	(MailMsgDescFunc) open_calendar__desc,
+	(MailMsgExecFunc) open_calendar__exec,
+	(MailMsgDoneFunc) NULL,
+	(MailMsgFreeFunc) open_calendar__free,
 };
 
 static gboolean
@@ -1600,10 +1591,10 @@ idle_open_cb (gpointer data)
 	FormatItipPObject *pitip = data;
 	struct _opencal_msg *m;
 
-	m = mail_msg_new (&open_calendar_op, NULL, sizeof (*m));
+	m = mail_msg_new (&open_calendar_info);
 	m->command = g_strdup_printf ("evolution \"calendar://?startdate=%s&enddate=%s\"",
 				   isodate_from_time_t (pitip->start_time), isodate_from_time_t (pitip->end_time));
-	e_thread_put (mail_thread_queued_slow, (EMsg *)m);
+	mail_msg_slow_ordered_push (m);
 
 	return FALSE;
 }