[Galago-commits] r2438 - in trunk/libnotify: . libnotify

galago-commits at freedesktop.org galago-commits at freedesktop.org
Thu Jan 19 21:55:34 PST 2006


Author: chipx86
Date: 2006-01-19 21:55:30 -0800 (Thu, 19 Jan 2006)
New Revision: 2438

Modified:
   trunk/libnotify/ChangeLog
   trunk/libnotify/libnotify/dbus-compat.h
   trunk/libnotify/libnotify/notify.c
   trunk/libnotify/libnotify/notify.h
   trunk/libnotify/libnotify/notifycommon.h
   trunk/libnotify/libnotify/notifynotification.c
   trunk/libnotify/libnotify/notifynotification.h
Log:
Fixed the coding style.


Modified: trunk/libnotify/ChangeLog
===================================================================
--- trunk/libnotify/ChangeLog	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/ChangeLog	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,3 +1,13 @@
+Thu Jan 19 21:55:08 PST 2006  Christian Hammond <chipx86 at chipx86.com>
+
+	* libnotify/dbus-compat.h:
+	* libnotify/notify.c:
+	* libnotify/notifycommon.h:
+	* libnotify/notify.h:
+	* libnotify/notifynotification.c:
+	* libnotify/notifynotification.h:
+	  - Fixed the coding style.
+
 Thu Jan 19 01:34:57 PST 2006  Christian Hammond <chipx86 at chipx86.com>
 
 	* tools/notify-send.c:

Modified: trunk/libnotify/libnotify/dbus-compat.h
===================================================================
--- trunk/libnotify/libnotify/dbus-compat.h	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/dbus-compat.h	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,3 +1,4 @@
+
 /**
  * @file libnotify/dbus-compat.h Private D-BUS Compatibility API
  *

Modified: trunk/libnotify/libnotify/notify.c
===================================================================
--- trunk/libnotify/libnotify/notify.c	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/notify.c	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,9 +1,8 @@
-/* -*- mode: c-mode; tab-width: 4; indent-tabs-mode: t; -*- */
 /**
  * @file libnotify/notify.c Notifications library
  *
- * @Copyright (C) 2004 Christian Hammond <chipx86 at chipx86.com>
- * @Copyright (C) 2004 Mike Hearn <mike at navi.cx>
+ * @Copyright (C) 2004-2006 Christian Hammond <chipx86 at chipx86.com>
+ * @Copyright (C) 2004-2006 Mike Hearn <mike at navi.cx>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -50,65 +49,65 @@
 
 #if 0
 static void format_func
-print_error (char *message, ...)
+print_error(char *message, ...)
 {
-  char buf[1024];
-  va_list args;
+	char buf[1024];
+	va_list args;
 
-  va_start (args, message);
-  vsnprintf (buf, sizeof (buf), message, args);
-  va_end (args);
+	va_start(args, message);
+	vsnprintf(buf, sizeof(buf), message, args);
+	va_end(args);
 
-  fprintf (stderr, "%s(%d): libnotify: %s",
-	   (getenv ("_") ? getenv ("_") : ""), getpid (), buf);
+	fprintf(stderr, "%s(%d): libnotify: %s",
+			(getenv("_") ? getenv("_") : ""), getpid(), buf);
 }
 #endif
 
 gboolean
-notify_init (const char *app_name)
+notify_init(const char *app_name)
 {
-  g_return_val_if_fail (app_name != NULL, FALSE);
-  g_return_val_if_fail (*app_name != '\0', FALSE);
+	g_return_val_if_fail(app_name != NULL, FALSE);
+	g_return_val_if_fail(*app_name != '\0', FALSE);
 
-  if (_initted)
-    return TRUE;
+	if (_initted)
+		return TRUE;
 
-  _app_name = g_strdup (app_name);
+	_app_name = g_strdup(app_name);
 
-  g_type_init ();
+	g_type_init();
 
 #ifdef HAVE_ATEXIT
-  atexit (notify_uninit);
+	atexit(notify_uninit);
 #endif /* HAVE_ATEXIT */
 
-  _initted = TRUE;
+	_initted = TRUE;
 
-  return TRUE;
+	return TRUE;
 }
 
 const gchar *
-notify_get_app_name (void)
+notify_get_app_name(void)
 {
-  return _app_name;
+	return _app_name;
 }
 
 void
-notify_uninit (void)
+notify_uninit(void)
 {
 
-  if (_app_name != NULL)
-    {
-      g_free (_app_name);
-      _app_name = NULL;
-    }
+	if (_app_name != NULL)
+	{
+		g_free(_app_name);
+		_app_name = NULL;
+	}
 
-  /* TODO: keep track of all notifications and destroy them here? */
+	/* TODO: keep track of all notifications and destroy them here? */
 }
 
 gboolean
-notify_is_initted (void)
+notify_is_initted(void)
 {
-  return _initted;
+	return _initted;
 }
 
 static DBusGProxy *

Modified: trunk/libnotify/libnotify/notify.h
===================================================================
--- trunk/libnotify/libnotify/notify.h	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/notify.h	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,8 +1,7 @@
-/* -*- mode: c-mode; tab-width: 4; indent-tabs-mode: t; -*- */
 /**
  * @file libnotify/notify.h Notifications library
  *
- * @Copyright (C) 2004 Christian Hammond
+ * @Copyright (C) 2004-2006 Christian Hammond
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
@@ -19,7 +18,6 @@
  * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA  02111-1307, USA.
  */
-
 #ifndef _LIBNOTIFY_NOTIFY_H_
 #define _LIBNOTIFY_NOTIFY_H_
 
@@ -58,6 +56,11 @@
  */
 gboolean notify_is_initted(void);
 
+/**
+ * Returns the name of the application set when notify_init() was called.
+ *
+ * @return The name of the application.
+ */
 const gchar *notify_get_app_name(void);
 
 /**
@@ -79,8 +82,10 @@
  *
  * @return TRUE if the call succeeded, or FALSE if there were errors.
  */
-gboolean notify_get_server_info(char **ret_name, char **ret_vendor,
-								char **ret_version, char **ret_spec_version);
+gboolean notify_get_server_info(char **ret_name,
+								char **ret_vendor,
+								char **ret_version,
+								char **ret_spec_version);
 
 /*@}*/
 

Modified: trunk/libnotify/libnotify/notifycommon.h
===================================================================
--- trunk/libnotify/libnotify/notifycommon.h	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/notifycommon.h	2006-01-20 05:55:30 UTC (rev 2438)
@@ -3,7 +3,7 @@
 #include <glib.h>
 
 #define NOTIFY_TIMEOUT_DEFAULT -1
-#define NOTIFY_TIMEOUT_NEVER 0
+#define NOTIFY_TIMEOUT_NEVER    0
 
 #define NOTIFY_DBUS_NAME           "org.freedesktop.Notifications"
 #define NOTIFY_DBUS_CORE_INTERFACE "org.freedesktop.Notifications"
@@ -14,8 +14,8 @@
  */
 typedef enum
 {
-	NOTIFY_URGENCY_LOW,       /**< Low urgency.      */
-	NOTIFY_URGENCY_NORMAL,    /**< Normal urgency.   */
+	NOTIFY_URGENCY_LOW,		  /**< Low urgency.      */
+	NOTIFY_URGENCY_NORMAL,	  /**< Normal urgency.   */
 	NOTIFY_URGENCY_CRITICAL,  /**< Critical urgency. */
 
 } NotifyUrgency;

Modified: trunk/libnotify/libnotify/notifynotification.c
===================================================================
--- trunk/libnotify/libnotify/notifynotification.c	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/notifynotification.c	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,19 +1,23 @@
-/*
- * This program is free software; you can redistribute it and/or
+/**
+ * @file libnotify/notifynotification.c Notification object
+ *
+ * @Copyright (C) 2006 Christian Hammond
+ * @Copyright (C) 2006 John Palmieri
+ *
+ * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2.1 of the License, or (at your option) any later version.
- * 
- * This program is distributed in the hope that it will be useful,
+ *
+ * This library 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
  * Lesser General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU Lesser General Public
- * License along with main.c; if not, write to:
- *            The Free Software Foundation, Inc.,
- *            59 Temple Place - Suite 330,
- *            Boston,  MA  02111-1307, USA.
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA  02111-1307, USA.
  */
 #include "config.h"
 #include <dbus/dbus.h>
@@ -23,254 +27,235 @@
 #include "notifynotification.h"
 #include "notify-marshal.h"
 
-static void notify_notification_class_init (NotifyNotificationClass * klass);
-static void notify_notification_init (NotifyNotification * sp);
-static void notify_notification_finalize (GObject * object);
-static void _close_signal_handler (DBusGProxy *proxy,
-                                   guint32 id, 
-                                   NotifyNotification *notification); 
+static void notify_notification_class_init(NotifyNotificationClass *klass);
+static void notify_notification_init(NotifyNotification *sp);
+static void notify_notification_finalize(GObject *object);
+static void _close_signal_handler(DBusGProxy *proxy, guint32 id,
+								  NotifyNotification *notification);
 
-static void _action_signal_handler (DBusGProxy *proxy,
-                                    guint32 id,
-                                    gchar *action, 
-                                    NotifyNotification *notification); 
+static void _action_signal_handler(DBusGProxy *proxy, guint32 id,
+								   gchar *action,
+								   NotifyNotification *notification);
 
-struct NotifyNotificationPrivate
+struct _NotifyNotificationPrivate
 {
-  guint32 id;
-  gchar *summary;
-  gchar *message;
+	guint32 id;
+	gchar *summary;
+	gchar *message;
 
-  /*NULL to use icon data
-     anything else to have server lookup icon */
-  gchar *icon_name;
+	/* NULL to use icon data anything else to have server lookup icon */
+	gchar *icon_name;
 
-    /*-1 = use server default
-       0 = never timeout
-      >0 = Number of milliseconds before we timeout
+	/*
+	 * -1   = use server default
+	 *  0   = never timeout
+	 *  > 0 = Number of milliseconds before we timeout
     */
-  gint timeout;
+	gint timeout;
 
-  GSList *actions;
-  GHashTable *action_map;
-  GHashTable *hints;
+	GSList *actions;
+	GHashTable *action_map;
+	GHashTable *hints;
 
-  GtkWidget *attached_widget;
-  gint widget_old_x;
-  gint widget_old_y;
+	GtkWidget *attached_widget;
+	gint widget_old_x;
+	gint widget_old_y;
 
-  gpointer user_data;
-  GDestroyNotify user_data_free_func;
+	gpointer user_data;
+	GDestroyNotify user_data_free_func;
 
-  gboolean updates_pending;
+	gboolean updates_pending;
 
-  DBusGProxy *proxy;
+	DBusGProxy *proxy;
 };
 
 typedef enum
 {
-  SIGNAL_TYPE_CLOSED, 
-  LAST_SIGNAL
+	SIGNAL_TYPE_CLOSED,
+	LAST_SIGNAL
+
 } NotifyNotificationSignalType;
 
 typedef struct
 {
-  NotifyNotification *object;
+	NotifyNotification *object;
+
 } NotifyNotificationSignal;
 
-static guint notify_notification_signals[LAST_SIGNAL] = { 0 }; 
+static guint notify_notification_signals[LAST_SIGNAL] = { 0 };
 static GObjectClass *parent_class = NULL;
 
 GType
-notify_notification_get_type ()
+notify_notification_get_type()
 {
-  static GType type = 0;
+	static GType type = 0;
 
-  if (type == 0)
-    {
-      static const GTypeInfo our_info = {
-	sizeof (NotifyNotificationClass),
-	NULL,
-	NULL,
-	(GClassInitFunc) notify_notification_class_init,
-	NULL,
-	NULL,
-	sizeof (NotifyNotification),
-	0,
-	(GInstanceInitFunc) notify_notification_init,
-      };
+	if (type == 0)
+	{
+		static const GTypeInfo our_info = {
+			sizeof(NotifyNotificationClass),
+			NULL,
+			NULL,
+			(GClassInitFunc) notify_notification_class_init,
+			NULL,
+			NULL,
+			sizeof(NotifyNotification),
+			0,
+			(GInstanceInitFunc) notify_notification_init,
+		};
 
-      type = g_type_register_static (G_TYPE_OBJECT,
-				     "NotifyNotification", &our_info, 0);
-    }
+		type = g_type_register_static(G_TYPE_OBJECT,
+									  "NotifyNotification", &our_info, 0);
+	}
 
-  return type;
+	return type;
 }
 
 static void
-notify_notification_class_init (NotifyNotificationClass * klass)
+notify_notification_class_init(NotifyNotificationClass * klass)
 {
-  GObjectClass *object_class = G_OBJECT_CLASS (klass);
+	GObjectClass *object_class = G_OBJECT_CLASS(klass);
 
-  parent_class = g_type_class_peek_parent (klass);
-  object_class->finalize = notify_notification_finalize;
+	parent_class = g_type_class_peek_parent(klass);
+	object_class->finalize = notify_notification_finalize;
 
-  /* Create signals here: */
-     notify_notification_signals[SIGNAL_TYPE_CLOSED] = 
-       g_signal_new ("closed",
-                     G_TYPE_FROM_CLASS (object_class),
-                     G_SIGNAL_RUN_FIRST,
-                     G_STRUCT_OFFSET (NotifyNotificationClass, closed),
-                     NULL, NULL, 
-                     g_cclosure_marshal_VOID__VOID,
-                     G_TYPE_NONE,
-                     0);
+	/* Create signals here: */
+	notify_notification_signals[SIGNAL_TYPE_CLOSED] =
+		g_signal_new("closed",
+					 G_TYPE_FROM_CLASS(object_class),
+					 G_SIGNAL_RUN_FIRST,
+					 G_STRUCT_OFFSET(NotifyNotificationClass, closed),
+					 NULL, NULL,
+					 g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
 
-  dbus_g_object_register_marshaller (notify_marshal_VOID__UINT_STRING,
-                                     G_TYPE_NONE, 
-                                     G_TYPE_UINT,
-                                     G_TYPE_STRING, 
-                                     G_TYPE_INVALID);
-
+	dbus_g_object_register_marshaller(notify_marshal_VOID__UINT_STRING,
+									  G_TYPE_NONE,
+									  G_TYPE_UINT,
+									  G_TYPE_STRING, G_TYPE_INVALID);
 }
 
 static void
-_g_value_free (GValue * value)
+_g_value_free(GValue *value)
 {
-  g_value_unset (value);
-  g_free (value);
+	g_value_unset(value);
+	g_free(value);
 }
 
 static void
-notify_notification_init (NotifyNotification * obj)
+notify_notification_init(NotifyNotification *obj)
 {
-  obj->priv = g_new0 (NotifyNotificationPrivate, 1);
+	obj->priv = g_new0(NotifyNotificationPrivate, 1);
 
-  obj->priv->id = 0;
+	obj->priv->id = 0;
+	obj->priv->summary = NULL;
+	obj->priv->message = NULL;
+	obj->priv->icon_name = NULL;
+	obj->priv->timeout = NOTIFY_TIMEOUT_DEFAULT;
+	obj->priv->actions = NULL;
+	obj->priv->hints = g_hash_table_new_full(g_str_hash, g_str_equal,
+											 g_free,
+											 (GFreeFunc)_g_value_free);
 
-  obj->priv->summary = NULL;
-  obj->priv->message = NULL;
-  obj->priv->icon_name = NULL;
-  obj->priv->timeout = NOTIFY_TIMEOUT_DEFAULT;
+	obj->priv->action_map = g_hash_table_new_full(g_str_hash, g_str_equal,
+												  g_free, NULL);
 
-  obj->priv->actions = NULL;
-  obj->priv->hints = g_hash_table_new_full (g_str_hash,
-					    g_str_equal,
-					    g_free,
-					    (GDestroyNotify) _g_value_free);
+	obj->priv->attached_widget = NULL;
+	obj->priv->user_data = NULL;
+	obj->priv->user_data_free_func = NULL;
 
-  
-  obj->priv->action_map = g_hash_table_new_full (g_str_hash,
-                                                 g_str_equal,
-                                                 g_free,
-                                                 NULL);
+	obj->priv->updates_pending = FALSE;
 
-  obj->priv->attached_widget = NULL;
-  obj->priv->user_data = NULL;
-  obj->priv->user_data_free_func = NULL;
+	obj->priv->widget_old_x = 0;
+	obj->priv->widget_old_y = 0;
 
-  obj->priv->updates_pending = FALSE;
-
-  obj->priv->widget_old_x = 0;
-  obj->priv->widget_old_y = 0;
-
-  obj->priv->proxy = NULL;
-
+	obj->priv->proxy = NULL;
 }
 
 static void
-notify_notification_finalize (GObject * object)
+notify_notification_finalize(GObject *object)
 {
-  NotifyNotification *obj;
-  NotifyNotificationPrivate *priv;
+	NotifyNotification *obj = NOTIFY_NOTIFICATION(object);
+	NotifyNotificationPrivate *priv = obj->priv;
 
-  obj = NOTIFY_NOTIFICATION (object);
-  priv = obj->priv;
+	g_free(priv->summary);
+	g_free(priv->message);
+	g_free(priv->icon_name);
 
-  g_free (priv->summary);
-  g_free (priv->message);
-  g_free (priv->icon_name);
+	if (priv->actions != NULL)
+	{
+		g_slist_foreach(priv->actions, (GFunc)g_free, NULL);
+		g_slist_free(priv->actions);
+	}
 
+	if (priv->action_map != NULL)
+		g_hash_table_destroy(priv->action_map);
 
-  if (priv->actions != NULL)
-    {
-      g_slist_foreach (priv->actions, (GFunc) g_free, NULL);
-      g_slist_free (priv->actions);
-    }
+	if (priv->hints != NULL)
+		g_hash_table_destroy(priv->hints);
 
-  if (priv->action_map != NULL)
-    g_hash_table_destroy (priv->action_map);
+	if (priv->attached_widget != NULL)
+		g_object_unref(G_OBJECT(priv->attached_widget));
 
-  if (priv->hints != NULL)
-    g_hash_table_destroy (priv->hints);
+	if (priv->user_data_free_func != NULL)
+		priv->user_data_free_func(priv->user_data);
 
-  if (priv->attached_widget != NULL)
-    gtk_widget_unref (priv->attached_widget);
+	dbus_g_proxy_disconnect_signal(priv->proxy, "NotificationClosed",
+								   G_CALLBACK(_close_signal_handler),
+								   object);
 
-  if (priv->user_data_free_func != NULL)
-    priv->user_data_free_func (priv->user_data);
+	dbus_g_proxy_disconnect_signal(priv->proxy, "ActionInvoked",
+								   G_CALLBACK(_action_signal_handler),
+								   object);
 
-  dbus_g_proxy_disconnect_signal (priv->proxy, "NotificationClosed", 
-                                  (GCallback) _close_signal_handler, 
-                                  object);
-
-  dbus_g_proxy_disconnect_signal (priv->proxy, "ActionInvoked", 
-                                  (GCallback) _action_signal_handler, 
-                                  object);
-  
-  g_free (obj->priv);
-  G_OBJECT_CLASS (parent_class)->finalize (object);
+	g_free(obj->priv);
+	G_OBJECT_CLASS(parent_class)->finalize(object);
 }
 
 static gboolean
-_notify_notification_update_applet_hints (NotifyNotification * n)
+_notify_notification_update_applet_hints(NotifyNotification *n)
 {
-  NotifyNotificationPrivate *priv;
-  gboolean update_pending;
+	NotifyNotificationPrivate *priv = n->priv;
+	gboolean update_pending = FALSE;
 
-  update_pending = FALSE;
-  priv = n->priv;
+	if (priv->attached_widget != NULL)
+	{
+		gint x, y, h, w;
+		GtkWidget *widget = priv->attached_widget;
+		GtkRequisition requisition;
 
-  if (priv->attached_widget != NULL)
-    {
-      gint x, y, h, w;
-      GtkWidget *widget;
-      GtkRequisition requisition;
+		gtk_widget_size_request(widget, &requisition);
+		w = requisition.width;
+		h = requisition.height;
 
-      widget = priv->attached_widget;
+		gdk_window_get_origin(widget->window, &x, &y);
 
-      gtk_widget_size_request (widget, &requisition);
-      w = requisition.width;
-      h = requisition.height;
+		if (GTK_WIDGET_NO_WINDOW(widget))
+		{
+			x += widget->allocation.x;
+			y += widget->allocation.y;
+		}
 
-      gdk_window_get_origin (widget->window, &x, &y);
-      if (GTK_WIDGET_NO_WINDOW (widget))
-	{
-	  x += widget->allocation.x;
-	  y += widget->allocation.y;
-	}
+		x += widget->allocation.width / 2;
+		y += widget->allocation.height / 2;
 
-      x += widget->allocation.width / 2;
-      y += widget->allocation.height / 2;
+		if (x != priv->widget_old_x)
+		{
+			notify_notification_set_hint_int32(n, "x", x);
+			priv->widget_old_x = x;
 
-      if (x != priv->widget_old_x)
-	{
-	  notify_notification_set_hint_int32 (n, "x", x);
-	  priv->widget_old_x = x;
+			update_pending = TRUE;
+		}
 
-	  update_pending = TRUE;
-	}
+		if (y != priv->widget_old_y)
+		{
+			notify_notification_set_hint_int32(n, "y", y);
+			priv->widget_old_y = y;
 
-      if (y != priv->widget_old_y)
-	{
-	  notify_notification_set_hint_int32 (n, "y", y);
-	  priv->widget_old_y = y;
-
-	  update_pending = TRUE;
+			update_pending = TRUE;
+		}
 	}
-    }
 
-  return update_pending;
+	return update_pending;
 }
 
 #if 0
@@ -282,663 +267,611 @@
    leave this alone for now.
 */
 static gboolean
-_idle_check_updates (void *user_data)
+_idle_check_updates(void *user_data)
 {
-  NotifyNotification *n;
-  NotifyNotificationPrivate *priv;
+	NotifyNotification *n = NOTIFY_NOTIFICATION(user_data);
+	NotifyNotificationPrivate *priv = n->priv;
 
-  n = NOTIFY_NOTIFICATION (user_data);
-  priv = n->priv;
+	if (priv->is_visible)
+	{
+		priv->updates_pending = _notify_notification_update_applet_hints(n);
 
-  if (priv->is_visible)
-    {
-      priv->updates_pending = _notify_notification_update_applet_hints (n);
-      if (priv->updates_pending)
+		if (priv->updates_pending)
+		{
+			/* Try again if we fail on next idle */
+			priv->updates_pending = !notify_notification_show(n, NULL);
+		}
+	}
+	else
 	{
-	  /* Try again if we fail on next idle */
-	  priv->updates_pending = !notify_notification_show (n, NULL);
+		priv->updates_pending = FALSE;
 	}
-    }
-  else
-    {
-      priv->updates_pending = FALSE;
-    }
 
-  n = NOTIFY_NOTIFICATION (user_data);
-  priv = n->priv;
-
-  return TRUE;
+	return TRUE;
 }
 #endif
 
 GdkFilterReturn
-_catch (GdkXEvent * xevent, GdkEvent * event, gpointer data)
+_catch(GdkXEvent *xevent, GdkEvent *event, gpointer data)
 {
-  static int i = 1;
-  printf ("here, %i\n", i);
-  i++;
-  return GDK_FILTER_CONTINUE;
+	static int i = 1;
+
+	printf("here, %i\n", i);
+	i++;
+	return GDK_FILTER_CONTINUE;
 }
 
 NotifyNotification *
-notify_notification_new (const gchar * summary,
-			 const gchar * message,
-			 const gchar * icon, GtkWidget * attach)
+notify_notification_new(const gchar *summary, const gchar *message,
+						const gchar *icon, GtkWidget *attach)
 {
-  NotifyNotification *obj;
+	NotifyNotification *obj;
 
-  g_assert (summary != NULL);
-  g_assert (message != NULL);
+	g_assert(summary != NULL);
+	g_assert(message != NULL);
 
-  obj = NOTIFY_NOTIFICATION (g_object_new (NOTIFY_TYPE_NOTIFICATION, NULL));
+	obj = NOTIFY_NOTIFICATION(g_object_new(NOTIFY_TYPE_NOTIFICATION, NULL));
 
-  obj->priv->summary = g_strdup (summary);
-  obj->priv->message = g_strdup (message);
-  obj->priv->icon_name = g_strdup (icon);
+	obj->priv->summary = g_strdup(summary);
+	obj->priv->message = g_strdup(message);
+	obj->priv->icon_name = g_strdup(icon);
 
-  if (attach != NULL)
-    {
-      gtk_widget_ref (attach);
-      obj->priv->attached_widget = attach;
-    }
+	if (attach != NULL)
+	{
+		g_object_ref(G_OBJECT(attach));
+		obj->priv->attached_widget = attach;
+	}
 
-  return obj;
+	return obj;
 }
 
 gboolean
-notify_notification_update (NotifyNotification * notification,
-			    const gchar * summary,
-			    const gchar * message, const gchar * icon)
+notify_notification_update(NotifyNotification *notification,
+						   const gchar *summary,
+						   const gchar *message, const gchar *icon)
 {
-  NotifyNotificationPrivate *priv;
+	NotifyNotificationPrivate *priv = notification->priv;
 
-  priv = notification->priv;
-  g_free (priv->summary);
-  g_free (priv->message);
-  g_free (priv->icon_name);
+	g_free(priv->summary);
+	g_free(priv->message);
+	g_free(priv->icon_name);
 
-  priv->summary = g_strdup (summary);
-  priv->message = g_strdup (message);
-  priv->icon_name = g_strdup (icon);
+	priv->summary = g_strdup(summary);
+	priv->message = g_strdup(message);
+	priv->icon_name = g_strdup(icon);
 
-  priv->updates_pending = TRUE;
+	priv->updates_pending = TRUE;
 
-  /*TODO: return false on OOM */
-  return TRUE;
+	/*TODO: return false on OOM */
+	return TRUE;
 }
 
 void
-notify_notification_attach_to_widget (NotifyNotification * notification,
-				      GtkWidget * attach)
+notify_notification_attach_to_widget(NotifyNotification *notification,
+									 GtkWidget *attach)
 {
-  NotifyNotificationPrivate *priv;
+	NotifyNotificationPrivate *priv = notification->priv;
 
-  priv = notification->priv;
+	if (priv->attached_widget != NULL)
+		g_object_unref(priv->attached_widget);
 
-  if (priv->attached_widget != NULL)
-    gtk_widget_unref (priv->attached_widget);
-
-  if (attach != NULL)
-    priv->attached_widget = gtk_widget_ref (attach);
-  else
-    priv->attached_widget = NULL;
-
+	priv->attached_widget = (attach != NULL ? g_object_ref(attach) : NULL);
 }
 
 gboolean
-notify_notification_set_user_data (NotifyNotification * notification,
-				   void *user_data, GFreeFunc free_func)
+notify_notification_set_user_data(NotifyNotification *notification,
+								  void *user_data, GFreeFunc free_func)
 {
-  NotifyNotificationPrivate *priv;
+	NotifyNotificationPrivate *priv = notification->priv;
 
-  priv = notification->priv;
+	if (priv->user_data)
+	{
+		if (priv->user_data_free_func)
+			priv->user_data_free_func(priv->user_data);
+	}
 
-  if (priv->user_data)
-    if (priv->user_data_free_func)
-      priv->user_data_free_func (priv->user_data);
+	priv->user_data = user_data;
+	priv->user_data_free_func = free_func;
 
-  priv->user_data = user_data;
-  priv->user_data_free_func = free_func;
-
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
 gpointer
-notify_notification_get_user_data (NotifyNotification * notification)
+notify_notification_get_user_data(NotifyNotification *notification)
 {
-  NotifyNotificationPrivate *priv;
-
-  priv = notification->priv;
-
-  return priv->user_data;
+	return notification->priv->user_data;
 }
 
-static void 
-_close_signal_handler (DBusGProxy *proxy, 
-                       guint32 id, 
-                       NotifyNotification *notification) 
+static void
+_close_signal_handler(DBusGProxy *proxy, guint32 id,
+					  NotifyNotification *notification)
 {
-  printf ("Got the NotificationClosed signal (id = %i, notification->id = %i)\n"
-, id, notification->priv->id);
+	printf("Got the NotificationClosed signal (id = %i, "
+		   "notification->id = %i)\n",
+		   id, notification->priv->id);
 
-  if (id == notification->priv->id)
-      g_signal_emit (notification, 
-                     notify_notification_signals[SIGNAL_TYPE_CLOSED], 
-                     0);
+	if (id == notification->priv->id)
+	{
+		g_signal_emit(notification,
+					  notify_notification_signals[SIGNAL_TYPE_CLOSED], 0);
+	}
 }
 
-static void 
-_action_signal_handler (DBusGProxy *proxy, 
-                        guint32 id, 
-                        gchar *action,
-                        NotifyNotification *notification) 
+static void
+_action_signal_handler(DBusGProxy *proxy,
+					   guint32 id,
+					   gchar *action, NotifyNotification *notification)
 {
-  g_assert (NOTIFY_IS_NOTIFICATION (notification));
+	g_assert(NOTIFY_IS_NOTIFICATION(notification));
 
-  if (id == notification->priv->id)
-    {
-      NotifyActionCallback callback;
+	if (id == notification->priv->id)
+	{
+		NotifyActionCallback callback;
 
-      callback = (NotifyActionCallback) g_hash_table_lookup (notification->priv->action_map,
-                                                             action);
+		callback = (NotifyActionCallback)g_hash_table_lookup(
+			notification->priv->action_map, action);
 
-      if (callback == NULL)
-        g_warning ("Recieved unknown action %s", action);
-      else
-        callback (notification, action);
-
-    }
+		if (callback == NULL)
+			g_warning("Recieved unknown action %s", action);
+		else
+			callback(notification, action);
+	}
 }
 
 static gchar **
-_gslist_to_string_array (GSList *list)
+_gslist_to_string_array(GSList *list)
 {
-  GSList *element;
-  GArray *a;
-  gsize len;
-  gchar **result;
+	GSList *l;
+	GArray *a;
 
-  len = g_slist_length (list);
-  
-  a = g_array_sized_new (TRUE, FALSE, sizeof (gchar *), len);
+	a = g_array_sized_new(TRUE, FALSE, sizeof(gchar *),
+						  g_slist_length(list));
 
-  element = list;
-  while (element != NULL)
-    {
-      g_array_append_val (a, element->data); 
+	for (l = list; l != NULL; l = l->next)
+		g_array_append_val(a, l->data);
 
-      element = g_slist_next (element);
-    }
-
-    result = (gchar **)g_array_free (a, FALSE);
-
-    return result;
+	return (gchar **)g_array_free(a, FALSE);
 }
 
 static gboolean
-_notify_notification_show_internal (NotifyNotification *notification, 
-                                    GError **error,
-                                    gboolean ignore_reply)
+_notify_notification_show_internal(NotifyNotification *notification,
+								   GError **error, gboolean ignore_reply)
 {
-  NotifyNotificationPrivate *priv;
-  GError *tmp_error;
-  gchar **action_array;
+	NotifyNotificationPrivate *priv = notification->priv;
+	GError *tmp_error = NULL;
+	gchar **action_array;
 
-  g_return_val_if_fail (error == NULL || *error == NULL, FALSE);
+	g_return_val_if_fail(error == NULL || *error == NULL, FALSE);
 
-  priv = notification->priv;
-
-  tmp_error = NULL;
-
-  if (priv->proxy == NULL)
-    {
-      DBusGConnection *bus;
-      bus = dbus_g_bus_get (DBUS_BUS_SESSION, &tmp_error);
-      if (tmp_error != NULL)
+	if (priv->proxy == NULL)
 	{
-	  g_propagate_error (error, tmp_error);
-	  return FALSE;
-	}
+		DBusGConnection *bus;
 
-      priv->proxy = dbus_g_proxy_new_for_name (bus,
-					       NOTIFY_DBUS_NAME,
-					       NOTIFY_DBUS_CORE_OBJECT,
-					       NOTIFY_DBUS_CORE_INTERFACE);
+		bus = dbus_g_bus_get(DBUS_BUS_SESSION, &tmp_error);
 
-      dbus_g_proxy_add_signal (priv->proxy, "NotificationClosed",
-                               G_TYPE_UINT, G_TYPE_INVALID);
-      dbus_g_proxy_connect_signal (priv->proxy, "NotificationClosed", 
-                               (GCallback) _close_signal_handler, 
-                               notification, NULL);
+		if (tmp_error != NULL)
+		{
+			g_propagate_error(error, tmp_error);
+			return FALSE;
+		}
 
-      dbus_g_proxy_add_signal (priv->proxy, "ActionInvoked",
-                               G_TYPE_UINT, G_TYPE_STRING, G_TYPE_INVALID);
-      dbus_g_proxy_connect_signal (priv->proxy, "ActionInvoked", 
-                               (GCallback) _action_signal_handler, 
-                               notification, NULL);
+		priv->proxy = dbus_g_proxy_new_for_name(bus,
+												NOTIFY_DBUS_NAME,
+												NOTIFY_DBUS_CORE_OBJECT,
+												NOTIFY_DBUS_CORE_INTERFACE);
 
+		dbus_g_proxy_add_signal(priv->proxy, "NotificationClosed",
+								G_TYPE_UINT, G_TYPE_INVALID);
+		dbus_g_proxy_connect_signal(priv->proxy, "NotificationClosed",
+									G_CALLBACK(_close_signal_handler),
+									notification, NULL);
 
-      dbus_g_connection_unref (bus);
-    }
+		dbus_g_proxy_add_signal(priv->proxy, "ActionInvoked",
+								G_TYPE_UINT, G_TYPE_STRING,
+								G_TYPE_INVALID);
+		dbus_g_proxy_connect_signal(priv->proxy, "ActionInvoked",
+									G_CALLBACK(_action_signal_handler),
+									notification, NULL);
 
-  /*if attached to a widget modify x and y in hints */
-  _notify_notification_update_applet_hints (notification);
+		dbus_g_connection_unref(bus);
+	}
 
-  action_array = _gslist_to_string_array (priv->actions);
+	/* If attached to a widget, modify x and y in hints */
+	_notify_notification_update_applet_hints(notification);
 
-  /*TODO: make this nonblocking */
-  if (!ignore_reply)
-    dbus_g_proxy_call (priv->proxy, "Notify", &tmp_error,
-		     G_TYPE_STRING, notify_get_app_name (),
-		     G_TYPE_STRING,
-		     (priv->icon_name != NULL) ? priv->icon_name : "",
-		     G_TYPE_UINT, priv->id, G_TYPE_STRING, priv->summary,
-		     G_TYPE_STRING, priv->message,
-		     G_TYPE_STRV,
-		     action_array, dbus_g_type_get_map ("GHashTable",
-							 G_TYPE_STRING,
-							 G_TYPE_VALUE),
-		     priv->hints, G_TYPE_INT, priv->timeout, G_TYPE_INVALID,
-		     G_TYPE_UINT, &priv->id, G_TYPE_INVALID);
-  else
-    dbus_g_proxy_call_no_reply (priv->proxy, "Notify",
-		     G_TYPE_STRING, notify_get_app_name (),
-		     G_TYPE_STRING,
-		     (priv->icon_name != NULL) ? priv->icon_name : "",
-		     G_TYPE_UINT, priv->id, G_TYPE_STRING, priv->summary,
-		     G_TYPE_STRING, priv->message,
-		     G_TYPE_STRV,
-		     action_array, dbus_g_type_get_map ("GHashTable",
-							 G_TYPE_STRING,
-							 G_TYPE_VALUE),
-		     priv->hints, G_TYPE_INT, priv->timeout, G_TYPE_INVALID);
+	action_array = _gslist_to_string_array(priv->actions);
 
-  
+	/* TODO: make this nonblocking */
 
-  /*don't free the elements because they are owned by priv->actions */
-  g_free (action_array);
+	if (!ignore_reply)
+	{
+		dbus_g_proxy_call(priv->proxy, "Notify", &tmp_error,
+						  G_TYPE_STRING, notify_get_app_name(),
+						  G_TYPE_STRING,
+						  (priv->icon_name != NULL) ? priv->icon_name : "",
+						  G_TYPE_UINT, priv->id, G_TYPE_STRING,
+						  priv->summary, G_TYPE_STRING, priv->message,
+						  G_TYPE_STRV, action_array,
+						  dbus_g_type_get_map("GHashTable", G_TYPE_STRING,
+											  G_TYPE_VALUE), priv->hints,
+						  G_TYPE_INT, priv->timeout, G_TYPE_INVALID,
+						  G_TYPE_UINT, &priv->id, G_TYPE_INVALID);
+	}
+	else
+	{
+		dbus_g_proxy_call_no_reply(priv->proxy, "Notify",
+								   G_TYPE_STRING, notify_get_app_name(),
+								   G_TYPE_STRING,
+								   (priv->icon_name !=
+									NULL) ? priv->icon_name : "",
+								   G_TYPE_UINT, priv->id, G_TYPE_STRING,
+								   priv->summary, G_TYPE_STRING,
+								   priv->message, G_TYPE_STRV,
+								   action_array,
+								   dbus_g_type_get_map("GHashTable",
+													   G_TYPE_STRING,
+													   G_TYPE_VALUE),
+								   priv->hints, G_TYPE_INT, priv->timeout,
+								   G_TYPE_INVALID);
+	}
 
-  if (tmp_error != NULL)
-    {
-      g_propagate_error (error, tmp_error);
-      return FALSE;
-    }
+	/* Don't free the elements because they are owned by priv->actions */
+	g_free(action_array);
 
-  return TRUE;
+	if (tmp_error != NULL)
+	{
+		g_propagate_error(error, tmp_error);
+		return FALSE;
+	}
+
+	return TRUE;
 }
 
-
-
 gboolean
-notify_notification_show (NotifyNotification *notification, GError **error)
+notify_notification_show(NotifyNotification *notification, GError **error)
 {
-  return _notify_notification_show_internal (notification, error, FALSE);
+	return _notify_notification_show_internal(notification, error, FALSE);
 }
 
 gboolean
-notify_notification_show_and_forget (NotifyNotification *notification, GError **error)
+notify_notification_show_and_forget(NotifyNotification *notification,
+									GError **error)
 {
-  gboolean result;
+	gboolean result =
+		_notify_notification_show_internal(notification, error, TRUE);
 
-  result =  _notify_notification_show_internal (notification, error, TRUE);
-  g_object_unref (G_OBJECT (notification));
+	g_object_unref(G_OBJECT(notification));
 
-  return result;
+	return result;
 }
 
 void
-notify_notification_set_timeout (NotifyNotification * notification,
-				 gint timeout)
+notify_notification_set_timeout(NotifyNotification *notification,
+								gint timeout)
 {
-  notification->priv->timeout = timeout;
+	notification->priv->timeout = timeout;
 }
 
 gboolean
-notify_notification_set_category (NotifyNotification * notification,
-				  const char *category)
+notify_notification_set_category(NotifyNotification *notification,
+								 const char *category)
 {
-  return notify_notification_set_hint_string (notification,
-					      "category", category);
+	return notify_notification_set_hint_string(notification,
+											   "category", category);
 }
 
 gboolean
-notify_notification_set_urgency (NotifyNotification * notification,
-				 NotifyUrgency l)
+notify_notification_set_urgency(NotifyNotification *notification,
+								NotifyUrgency l)
 {
-  return notify_notification_set_hint_byte (notification,
-					    "urgency", (guchar) l);
+	return notify_notification_set_hint_byte(notification,
+											 "urgency", (guchar)l);
 }
 
 static gboolean
-_gvalue_array_append_int (GValueArray * array, gint i)
+_gvalue_array_append_int(GValueArray *array, gint i)
 {
-  GValue *value;
+	GValue *value = g_new0(GValue, 1);
 
-  value = g_new0 (GValue, 1);
-  if (!value)
-    return FALSE;
+	if (value == NULL)
+		return FALSE;
 
-  g_value_init (value, G_TYPE_INT);
-  g_value_set_int (value, i);
+	g_value_init(value, G_TYPE_INT);
+	g_value_set_int(value, i);
+	g_value_array_append(array, value);
 
-  g_value_array_append (array, value);
-
-  return TRUE;
+	return TRUE;
 }
 
 static gboolean
-_gvalue_array_append_bool (GValueArray * array, gboolean b)
+_gvalue_array_append_bool(GValueArray *array, gboolean b)
 {
-  GValue *value;
+	GValue *value = g_new0(GValue, 1);
 
-  value = g_new0 (GValue, 1);
-  if (!value)
-    return FALSE;
+	if (value == NULL)
+		return FALSE;
 
-  g_value_init (value, G_TYPE_BOOLEAN);
-  g_value_set_boolean (value, b);
+	g_value_init(value, G_TYPE_BOOLEAN);
+	g_value_set_boolean(value, b);
+	g_value_array_append(array, value);
 
-  g_value_array_append (array, value);
-
-  return TRUE;
+	return TRUE;
 }
 
 static gboolean
-_gvalue_array_append_byte_array (GValueArray * array,
-				 guchar * bytes, gsize len)
+_gvalue_array_append_byte_array(GValueArray *array, guchar *bytes, gsize len)
 {
-  GArray *byte_array;
-  GValue *value;
+	GArray *byte_array;
+	GValue *value;
 
-  byte_array = g_array_sized_new (FALSE, FALSE, sizeof (guchar), len);
-  if (!byte_array)
-    return FALSE;
+	byte_array = g_array_sized_new(FALSE, FALSE, sizeof(guchar), len);
 
-  byte_array = g_array_append_vals (byte_array, bytes, len);
+	if (byte_array == NULL)
+		return FALSE;
 
-  value = g_new0 (GValue, 1);
-  if (!value)
-    {
-      g_array_free (byte_array, TRUE);
-      return FALSE;
-    }
+	byte_array = g_array_append_vals(byte_array, bytes, len);
 
-  g_value_init (value, dbus_g_type_get_collection ("GArray", G_TYPE_UCHAR));
-  g_value_set_boxed_take_ownership (value, byte_array);
+	if ((value = g_new0(GValue, 1)) == NULL)
+	{
+		g_array_free(byte_array, TRUE);
+		return FALSE;
+	}
 
-  g_value_array_append (array, value);
+	g_value_init(value, dbus_g_type_get_collection("GArray", G_TYPE_UCHAR));
+	g_value_set_boxed_take_ownership(value, byte_array);
+	g_value_array_append(array, value);
 
-  return TRUE;
+	return TRUE;
 }
 
-
-
 gboolean
-notify_notification_set_icon_data_from_pixbuf (NotifyNotification *
-					       notification, GdkPixbuf * icon)
+notify_notification_set_icon_data_from_pixbuf(
+	NotifyNotification *notification, GdkPixbuf *icon)
 {
-  gint width;
-  gint height;
-  gint rowstride;
-  gboolean alpha;
-  gint bits_per_sample;
-  gint n_channels;
-  guchar *image;
-  gsize image_len;
-  gchar *key_dup;
+	gint width;
+	gint height;
+	gint rowstride;
+	gboolean alpha;
+	gint bits_per_sample;
+	gint n_channels;
+	guchar *image;
+	gsize image_len;
+	GValueArray *image_struct;
+	GValue *value;
+	NotifyNotificationPrivate *priv;
 
-  GValueArray *image_struct;
-  GValue *value;
-  NotifyNotificationPrivate *priv;
+	priv = notification->priv;
 
-  priv = notification->priv;
+	width           = gdk_pixbuf_get_width(icon);
+	height          = gdk_pixbuf_get_height(icon);
+	rowstride       = gdk_pixbuf_get_rowstride(icon);
+	n_channels      = gdk_pixbuf_get_n_channels(icon);
+	bits_per_sample = gdk_pixbuf_get_bits_per_sample(icon);
+	alpha           = gdk_pixbuf_get_has_alpha(icon);
+	image_len       = (height - 1) * rowstride + width *
+	                  ((n_channels * bits_per_sample + 7) / 8);
 
-  width = gdk_pixbuf_get_width (icon);
-  height = gdk_pixbuf_get_height (icon);
-  rowstride = gdk_pixbuf_get_rowstride (icon);
-  n_channels = gdk_pixbuf_get_n_channels (icon);
-  bits_per_sample = gdk_pixbuf_get_bits_per_sample (icon);
-  alpha = gdk_pixbuf_get_has_alpha (icon);
-  image_len =
-    (height - 1) * rowstride +
-    width * ((n_channels * bits_per_sample + 7) / 8);
+	image = gdk_pixbuf_get_pixels(icon);
 
-  image = gdk_pixbuf_get_pixels (icon);
+	image_struct = g_value_array_new(8);
 
-  image_struct = g_value_array_new (8);
+	if (image_struct == NULL)
+		goto fail;
 
-  if (!image_struct)
-    goto fail;
+	_gvalue_array_append_int(image_struct, width);
+	_gvalue_array_append_int(image_struct, height);
+	_gvalue_array_append_int(image_struct, rowstride);
+	_gvalue_array_append_bool(image_struct, alpha);
+	_gvalue_array_append_int(image_struct, bits_per_sample);
+	_gvalue_array_append_int(image_struct, n_channels);
+	_gvalue_array_append_byte_array(image_struct, image, image_len);
 
-  _gvalue_array_append_int (image_struct, width);
-  _gvalue_array_append_int (image_struct, height);
-  _gvalue_array_append_int (image_struct, rowstride);
-  _gvalue_array_append_bool (image_struct, alpha);
-  _gvalue_array_append_int (image_struct, bits_per_sample);
-  _gvalue_array_append_int (image_struct, n_channels);
-  _gvalue_array_append_byte_array (image_struct, image, image_len);
+	value = g_new0(GValue, 1);
 
-  value = g_new0 (GValue, 1);
-  if (!value)
-    goto fail;
+	if (value == NULL)
+		goto fail;
 
-  g_value_init (value, G_TYPE_VALUE_ARRAY);
-  g_value_set_boxed (value, image_struct);
+	g_value_init(value, G_TYPE_VALUE_ARRAY);
+	g_value_set_boxed(value, image_struct);
 
-  key_dup = g_strdup ("icon_data");
-  if (!key_dup)
-    goto fail;
+	g_hash_table_insert(priv->hints, g_strdup("icon_data"), value);
 
-  g_hash_table_insert (priv->hints, key_dup, value);
+	return TRUE;
 
-  return TRUE;
-
 fail:
-  if (image_struct != NULL)
-    g_value_array_free (image_struct);
-  return FALSE;
+	if (image_struct != NULL)
+		g_value_array_free(image_struct);
+
+	return FALSE;
 }
 
 gboolean
-notify_notification_set_hint_int32 (NotifyNotification * notification,
-				    const gchar * key, gint value)
+notify_notification_set_hint_int32(NotifyNotification *notification,
+								   const gchar *key, gint value)
 {
-  NotifyNotificationPrivate *priv;
-  GValue *hint_value;
-  gchar *key_dup;
+	NotifyNotificationPrivate *priv;
+	GValue *hint_value;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  hint_value = g_new0 (GValue, 1);
-  g_value_init (hint_value, G_TYPE_INT);
-  g_value_set_int (hint_value, value);
+	hint_value = g_new0(GValue, 1);
+	g_value_init(hint_value, G_TYPE_INT);
+	g_value_set_int(hint_value, value);
 
-  key_dup = g_strdup (key);
+	g_hash_table_insert(priv->hints, g_strdup(key), hint_value);
 
-  g_hash_table_insert (priv->hints, key_dup, hint_value);
-
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
 gboolean
-notify_notification_set_hint_double (NotifyNotification * notification,
-				     const gchar * key, gdouble value)
+notify_notification_set_hint_double(NotifyNotification *notification,
+									const gchar *key, gdouble value)
 {
-  NotifyNotificationPrivate *priv;
-  GValue *hint_value;
-  gchar *key_dup;
+	NotifyNotificationPrivate *priv;
+	GValue *hint_value;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  hint_value = g_new0 (GValue, 1);
-  g_value_init (hint_value, G_TYPE_FLOAT);
-  g_value_set_float (hint_value, value);
+	hint_value = g_new0(GValue, 1);
+	g_value_init(hint_value, G_TYPE_FLOAT);
+	g_value_set_float(hint_value, value);
 
-  key_dup = g_strdup (key);
+	g_hash_table_insert(priv->hints, g_strdup(key), hint_value);
 
-  g_hash_table_insert (priv->hints, key_dup, hint_value);
-
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
 gboolean
-notify_notification_set_hint_byte (NotifyNotification * notification,
-				   const gchar * key, guchar value)
+notify_notification_set_hint_byte(NotifyNotification *notification,
+								  const gchar *key, guchar value)
 {
-  NotifyNotificationPrivate *priv;
-  GValue *hint_value;
-  gchar *key_dup;
+	NotifyNotificationPrivate *priv;
+	GValue *hint_value;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  hint_value = g_new0 (GValue, 1);
-  g_value_init (hint_value, G_TYPE_UCHAR);
-  g_value_set_uchar (hint_value, value);
+	hint_value = g_new0(GValue, 1);
+	g_value_init(hint_value, G_TYPE_UCHAR);
+	g_value_set_uchar(hint_value, value);
 
-  key_dup = g_strdup (key);
+	g_hash_table_insert(priv->hints, g_strdup(key), hint_value);
 
-  g_hash_table_insert (priv->hints, key_dup, hint_value);
-
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
 gboolean
-notify_notification_set_hint_byte_array (NotifyNotification * notification,
-					 const gchar * key,
-					 const guchar * value, gsize len)
+notify_notification_set_hint_byte_array(NotifyNotification *notification,
+										const gchar *key,
+										const guchar *value, gsize len)
 {
-  NotifyNotificationPrivate *priv;
-  GValue *hint_value;
-  gchar *key_dup;
-  GArray *byte_array;
+	NotifyNotificationPrivate *priv;
+	GValue *hint_value;
+	GArray *byte_array;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  byte_array = g_array_sized_new (FALSE, FALSE, sizeof (guchar), len);
-  byte_array = g_array_append_vals (byte_array, value, len);
+	byte_array = g_array_sized_new(FALSE, FALSE, sizeof(guchar), len);
+	byte_array = g_array_append_vals(byte_array, value, len);
 
-  hint_value = g_new0 (GValue, 1);
-  g_value_init (hint_value,
-		dbus_g_type_get_collection ("GArray", G_TYPE_UCHAR));
-  g_value_set_boxed_take_ownership (hint_value, byte_array);
-  key_dup = g_strdup (key);
+	hint_value = g_new0(GValue, 1);
+	g_value_init(hint_value, dbus_g_type_get_collection("GArray",
+														G_TYPE_UCHAR));
+	g_value_set_boxed_take_ownership(hint_value, byte_array);
 
-  g_hash_table_insert (priv->hints, key_dup, hint_value);
+	g_hash_table_insert(priv->hints, g_strdup(key), hint_value);
 
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
-
 gboolean
-notify_notification_set_hint_string (NotifyNotification * notification,
-				     const gchar * key, const gchar * value)
+notify_notification_set_hint_string(NotifyNotification *notification,
+									const gchar *key, const gchar *value)
 {
-  NotifyNotificationPrivate *priv;
-  GValue *hint_value;
-  gchar *key_dup;
+	NotifyNotificationPrivate *priv;
+	GValue *hint_value;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  hint_value = g_new0 (GValue, 1);
-  g_value_init (hint_value, G_TYPE_STRING);
-  g_value_set_string (hint_value, value);
+	hint_value = g_new0(GValue, 1);
+	g_value_init(hint_value, G_TYPE_STRING);
+	g_value_set_string(hint_value, value);
 
-  key_dup = g_strdup (key);
+	g_hash_table_insert(priv->hints, g_strdup(key), hint_value);
 
-  g_hash_table_insert (priv->hints, key_dup, hint_value);
-
-  /* TODO: return FALSE on OOM */
-  return TRUE;
+	/* TODO: return FALSE on OOM */
+	return TRUE;
 }
 
 static gboolean
-_remove_all (void)
+_remove_all(void)
 {
-  return TRUE;
+	return TRUE;
 }
 
-void 
-notify_notification_clear_hints (NotifyNotification *notification)
+void
+notify_notification_clear_hints(NotifyNotification *notification)
 {
-  g_hash_table_foreach_remove (notification->priv->hints,
-                               (GHRFunc) _remove_all, NULL);
+	g_hash_table_foreach_remove(notification->priv->hints,
+								(GHRFunc)_remove_all, NULL);
 }
 
 void
-notify_notification_clear_actions (NotifyNotification *notification)
+notify_notification_clear_actions(NotifyNotification *notification)
 {
-  g_hash_table_foreach_remove (notification->priv->action_map, (GHRFunc) _remove_all, NULL);
-   
-  if (notification->priv->actions != NULL)
-    {
-      g_slist_foreach (notification->priv->actions, (GFunc) g_free, NULL);
-      g_slist_free (notification->priv->actions);
-    }
+	g_hash_table_foreach_remove(notification->priv->action_map,
+								(GHRFunc)_remove_all, NULL);
 
-  notification->priv->actions = NULL;
+	if (notification->priv->actions != NULL)
+	{
+		g_slist_foreach(notification->priv->actions, (GFunc)g_free, NULL);
+		g_slist_free(notification->priv->actions);
+	}
+
+	notification->priv->actions = NULL;
 }
 
 gboolean
-notify_notification_add_action (NotifyNotification *notification,
-				const char *action,
-				const char *label,
-				NotifyActionCallback callback)
+notify_notification_add_action(NotifyNotification *notification,
+							   const char *action,
+							   const char *label,
+							   NotifyActionCallback callback)
 {
-  NotifyNotificationPrivate *priv;
+	NotifyNotificationPrivate *priv;
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  priv->actions = g_slist_append (priv->actions, g_strdup (action));
-  priv->actions = g_slist_append (priv->actions, g_strdup (label));
+	priv->actions = g_slist_append(priv->actions, g_strdup(action));
+	priv->actions = g_slist_append(priv->actions, g_strdup(label));
 
-  g_hash_table_insert (priv->action_map, g_strdup (action), callback);
-  
-  return FALSE;
+	g_hash_table_insert(priv->action_map, g_strdup(action), callback);
+
+	return FALSE;
 }
 
 gboolean
-notify_notification_close (NotifyNotification * notification, GError ** error)
+notify_notification_close(NotifyNotification *notification,
+						  GError **error)
 {
-  NotifyNotificationPrivate *priv;
-  GError *tmp_error;
+	NotifyNotificationPrivate *priv;
+	GError *tmp_error;
 
-  g_return_val_if_fail (error == NULL || *error == NULL, FALSE);
+	g_return_val_if_fail(error == NULL || *error == NULL, FALSE);
 
-  priv = notification->priv;
+	priv = notification->priv;
 
-  tmp_error = NULL;
+	tmp_error = NULL;
 
-  if (priv->proxy == NULL)
-    {
-      DBusGConnection *bus;
-
-      bus = dbus_g_bus_get (DBUS_BUS_SESSION, &tmp_error);
-      if (tmp_error != NULL)
+	if (priv->proxy == NULL)
 	{
-	  g_propagate_error (error, tmp_error);
-	  return FALSE;
-	}
+		DBusGConnection *bus = dbus_g_bus_get(DBUS_BUS_SESSION, &tmp_error);
 
-      priv->proxy = dbus_g_proxy_new_for_name (bus,
-					       NOTIFY_DBUS_NAME,
-					       NOTIFY_DBUS_CORE_OBJECT,
-					       NOTIFY_DBUS_CORE_INTERFACE);
-      dbus_g_connection_unref (bus);
-    }
+		if (tmp_error != NULL)
+		{
+			g_propagate_error(error, tmp_error);
+			return FALSE;
+		}
 
-  dbus_g_proxy_call (priv->proxy, "CloseNotification", &tmp_error,
-		     G_TYPE_UINT, priv->id, G_TYPE_INVALID, G_TYPE_INVALID);
+		priv->proxy = dbus_g_proxy_new_for_name(bus,
+												NOTIFY_DBUS_NAME,
+												NOTIFY_DBUS_CORE_OBJECT,
+												NOTIFY_DBUS_CORE_INTERFACE);
+		dbus_g_connection_unref(bus);
+	}
 
-  if (tmp_error != NULL)
-    {
-      g_propagate_error (error, tmp_error);
-      return FALSE;
-    }
+	dbus_g_proxy_call(priv->proxy, "CloseNotification", &tmp_error,
+					  G_TYPE_UINT, priv->id, G_TYPE_INVALID,
+					  G_TYPE_INVALID);
 
-  return TRUE;
+	if (tmp_error != NULL)
+	{
+		g_propagate_error(error, tmp_error);
+		return FALSE;
+	}
+
+	return TRUE;
 }

Modified: trunk/libnotify/libnotify/notifynotification.h
===================================================================
--- trunk/libnotify/libnotify/notifynotification.h	2006-01-19 18:18:23 UTC (rev 2437)
+++ trunk/libnotify/libnotify/notifynotification.h	2006-01-20 05:55:30 UTC (rev 2438)
@@ -1,24 +1,27 @@
-/*
- * This program is free software; you can redistribute it and/or
+/**
+ * @file libnotify/notifynotification.h Notification object
+ *
+ * @Copyright (C) 2006 Christian Hammond
+ * @Copyright (C) 2006 John Palmieri
+ *
+ * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2.1 of the License, or (at your option) any later version.
- * 
- * This program is distributed in the hope that it will be useful,
+ *
+ * This library 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
  * Lesser General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU Lesser General Public
- * License along with main.c; if not, write to:
- *            The Free Software Foundation, Inc.,
- *            59 Temple Place - Suite 330,
- *            Boston,  MA  02111-1307, USA.
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA  02111-1307, USA.
  */
+#ifndef _NOTIFY_NOTIFICATION_H_
+#define _NOTIFY_NOTIFICATION_H_
 
-#ifndef NOTIFY_NOTIFICATION_H
-#define NOTIFY_NOTIFICATION_H
-
 #include <glib.h>
 #include <glib-object.h>
 #include <gtk/gtk.h>
@@ -27,100 +30,111 @@
 
 G_BEGIN_DECLS
 
-#define NOTIFY_TYPE_NOTIFICATION         (notify_notification_get_type ())
-#define NOTIFY_NOTIFICATION(o)           (G_TYPE_CHECK_INSTANCE_CAST ((o), NOTIFY_TYPE_NOTIFICATION, NotifyNotification))
-#define NOTIFY_NOTIFICATION_CLASS(k)     (G_TYPE_CHECK_CLASS_CAST((k), NOTIFY_TYPE_NOTIFICATION, NotifyNotificationClass))
-#define NOTIFY_IS_NOTIFICATION(o)        (G_TYPE_CHECK_INSTANCE_TYPE ((o), NOTIFY_TYPE_NOTIFICATION))
-#define NOTIFY_IS_NOTIFICATION_CLASS(k)  (G_TYPE_CHECK_CLASS_TYPE ((k), NOTIFY_TYPE_NOTIFICATION))
-#define NOTIFY_NOTIFICATION_GET_CLASS(o) (G_TYPE_INSTANCE_GET_CLASS ((o), NOTIFY_TYPE_NOTIFICATION, NotifyNotificationClass))
+#define NOTIFY_TYPE_NOTIFICATION (notify_notification_get_type ())
+#define NOTIFY_NOTIFICATION(o) \
+	(G_TYPE_CHECK_INSTANCE_CAST ((o), NOTIFY_TYPE_NOTIFICATION, \
+								 NotifyNotification))
+#define NOTIFY_NOTIFICATION_CLASS(k) \
+	(G_TYPE_CHECK_CLASS_CAST((k), NOTIFY_TYPE_NOTIFICATION, \
+							 NotifyNotificationClass))
+#define NOTIFY_IS_NOTIFICATION(o) \
+	(G_TYPE_CHECK_INSTANCE_TYPE ((o), NOTIFY_TYPE_NOTIFICATION))
+#define NOTIFY_IS_NOTIFICATION_CLASS(k) \
+	(G_TYPE_CHECK_CLASS_TYPE ((k), NOTIFY_TYPE_NOTIFICATION))
+#define NOTIFY_NOTIFICATION_GET_CLASS(o) \
+	(G_TYPE_INSTANCE_GET_CLASS ((o), NOTIFY_TYPE_NOTIFICATION, \
+								NotifyNotificationClass))
 
-typedef struct NotifyNotificationPrivate NotifyNotificationPrivate;
+typedef struct _NotifyNotification        NotifyNotification;
+typedef struct _NotifyNotificationClass   NotifyNotificationClass;
+typedef struct _NotifyNotificationPrivate NotifyNotificationPrivate;
 
-typedef struct {
+struct _NotifyNotification
+{
 	GObject parent;
 	NotifyNotificationPrivate *priv;
-} NotifyNotification;
+};
 
-typedef struct {
+struct _NotifyNotificationClass
+{
 	GObjectClass parent_class;
-	/* Add Signal Functions Here */
-	void (*closed) (void);
-} NotifyNotificationClass;
 
-typedef void (*NotifyActionCallback )(NotifyNotification *, gchar *);
+	/* Signals */
+	void (*closed)(NotifyNotification *notification);
+};
 
+typedef void (*NotifyActionCallback)(NotifyNotification *, gchar *);
+
 GType notify_notification_get_type();
-NotifyNotification *notify_notification_new (const gchar *summary,
-                                             const gchar *message,
-                                             const gchar *icon,
-                                             GtkWidget *attach);
 
-gboolean notify_notification_update (NotifyNotification *notification,
-                                     const gchar *summary,
-                                     const gchar *message,
-                                     const gchar *icon);
+NotifyNotification *notify_notification_new(const gchar *summary,
+											const gchar *message,
+											const gchar *icon,
+											GtkWidget *attach);
 
-void notify_notification_attach_to_widget (NotifyNotification *notification,
-                                           GtkWidget *attach);
+gboolean notify_notification_update(NotifyNotification *notification,
+									const gchar *summary,
+									const gchar *message,
+									const gchar *icon);
 
-gboolean notify_notification_set_user_data (NotifyNotification *notification,
-                                            void *user_data,
-                                            GFreeFunc free_func);
+void notify_notification_attach_to_widget(NotifyNotification* notification,
+										  GtkWidget *attach);
 
-gpointer notify_notification_get_user_data (NotifyNotification *notification);
+gboolean notify_notification_set_user_data(NotifyNotification *notification,
+										   void *user_data,
+										   GFreeFunc free_func);
 
-gboolean notify_notification_show (NotifyNotification *notification, 
-                                   GError **error);
+gpointer notify_notification_get_user_data(NotifyNotification *notification);
 
-gboolean notify_notification_show_and_forget (NotifyNotification *notification, 
-                                              GError **error);
+gboolean notify_notification_show(NotifyNotification *notification,
+								  GError **error);
 
-void notify_notification_set_timeout (NotifyNotification *notification,
-                                      gint timeout);
+gboolean notify_notification_show_and_forget(NotifyNotification *notification,
+											 GError **error);
 
-gboolean notify_notification_set_category (NotifyNotification *notification,
-                                           const char *category);
+void notify_notification_set_timeout(NotifyNotification *notification,
+									 gint timeout);
 
-gboolean notify_notification_set_urgency (NotifyNotification *notification,
-                                          NotifyUrgency l);
+gboolean notify_notification_set_category(NotifyNotification *notification,
+										  const char *category);
 
-gboolean notify_notification_set_icon_data_from_pixbuf (NotifyNotification *notification,
-                                                        GdkPixbuf *icon);
+gboolean notify_notification_set_urgency(NotifyNotification *notification,
+										 NotifyUrgency l);
 
-                                          
-gboolean notify_notification_set_hint_int32 (NotifyNotification *notification,
-                                             const gchar *key,
-                                             gint value);
+gboolean notify_notification_set_icon_data_from_pixbuf(
+	NotifyNotification *notification, GdkPixbuf *icon);
 
-gboolean notify_notification_set_hint_double (NotifyNotification *notification,
-                                              const gchar *key,
-                                              gdouble value);
- 
-gboolean notify_notification_set_hint_string (NotifyNotification *notification,
-                                              const gchar *key,
-                                              const gchar *value);
-                                              
-gboolean notify_notification_set_hint_byte (NotifyNotification *notification,
-                                            const gchar *key,
-                                            guchar value);
+gboolean notify_notification_set_hint_int32(NotifyNotification *notification,
+											const gchar *key, gint value);
 
-gboolean notify_notification_set_hint_byte_array (
-                                         NotifyNotification *notification,
-                                         const gchar *key,
-                                         const guchar *value,
-                                         gsize len);
+gboolean notify_notification_set_hint_double(NotifyNotification *notification,
+											 const gchar *key,
+											 gdouble value);
 
-void notify_notification_clear_hints (NotifyNotification *notification);
+gboolean notify_notification_set_hint_string(NotifyNotification *notification,
+											 const gchar *key,
+											 const gchar *value);
 
-gboolean notify_notification_add_action (NotifyNotification *notification,
-                                         const char *action,
-                                         const char *label,
-                                         NotifyActionCallback callback);
+gboolean notify_notification_set_hint_byte(NotifyNotification *notification,
+										   const gchar *key,
+										   guchar value);
 
-void notify_notification_clear_actions (NotifyNotification *notification);                                
-gboolean notify_notification_close  (NotifyNotification *notification, 
-                                     GError **error); 
-                                         
-NotifyNotification *notify_notification_ref (NotifyNotification *notification);
-void notify_notification_unref (NotifyNotification *notification);
+gboolean notify_notification_set_hint_byte_array(
+	NotifyNotification *notification, const gchar *key,
+	const guchar *value, gsize len);
+
+void notify_notification_clear_hints(NotifyNotification *notification);
+
+gboolean notify_notification_add_action(NotifyNotification *notification,
+										const char *action,
+										const char *label,
+										NotifyActionCallback callback);
+
+void notify_notification_clear_actions(NotifyNotification *notification);
+gboolean notify_notification_close(NotifyNotification *notification,
+								   GError **error);
+
+NotifyNotification *notify_notification_ref(NotifyNotification *notification);
+void notify_notification_unref(NotifyNotification *notification);
+
 #endif /* NOTIFY_NOTIFICATION_H */



More information about the galago-commits mailing list