empathy r1697 - trunk/libempathy-gtk

[xclaesse svn gnome org]

Author: xclaesse
Date: Tue Nov 11 15:27:24 2008
New Revision: 1697
URL: http://svn.gnome.org/viewvc/empathy?rev=1697&view=rev

Log:
Make can_satisfy_mime_type_requirements accept NULL out params.

Modified:
   trunk/libempathy-gtk/empathy-avatar-chooser.c

Modified: trunk/libempathy-gtk/empathy-avatar-chooser.c
==============================================================================
--- trunk/libempathy-gtk/empathy-avatar-chooser.c	(original)
+++ trunk/libempathy-gtk/empathy-avatar-chooser.c	Tue Nov 11 15:27:24 2008
@@ -309,9 +309,11 @@
 				    gchar **satisfactory_format_name,
 				    gchar **satisfactory_mime_type)
 {
-	GSList *formats;
-	GSList *l;
-	gchar **strv;
+	GSList  *formats;
+	GSList  *l;
+	gchar  **strv;
+	gchar   *name = NULL,
+	        *type = NULL;
 	gboolean done = FALSE;
 
 	if (accepted_mime_types == NULL || *accepted_mime_types == NULL) {
@@ -334,8 +336,8 @@
 		format_mime_types = gdk_pixbuf_format_get_mime_types (format);
 		for (strv = format_mime_types; *strv != NULL; strv++) {
 			if (str_in_strv (*strv, accepted_mime_types)) {
-				*satisfactory_format_name = gdk_pixbuf_format_get_name (format);
-				*satisfactory_mime_type = g_strdup (*strv);
+				name = gdk_pixbuf_format_get_name (format);
+				type = g_strdup (*strv);
 				done = TRUE;
 				break;
 			}
@@ -345,7 +347,23 @@
 
 	g_slist_free (formats);
 
-	return done;
+	if (done) {
+		if (satisfactory_format_name != NULL) {
+			*satisfactory_format_name = name;
+		} else {
+			g_free (name);
+		}
+		if (satisfactory_mime_type != NULL) {
+			*satisfactory_mime_type = type;
+		} else {
+			g_free (type);
+		}
+		return TRUE;
+	} else {
+		/* check we're not leaking. */
+		g_assert (name == NULL && type == NULL);
+		return FALSE;
+	}
 }
 
 static EmpathyAvatar *