Switch to using g_file_get_contents() instead of our own code. rlaager.gaim_migration

Tue, 24 Apr 2007 19:58:14 +0000

author
Richard Laager <rlaager@pidgin.im>
date
Tue, 24 Apr 2007 19:58:14 +0000
branch
rlaager.gaim_migration
changeset 16532
070f046b1316
parent 16531
98363f6cdc6c
child 16533
91e3e84b05f2

Switch to using g_file_get_contents() instead of our own code.

libpurple/buddyicon.c file | annotate | diff | comparison | revisions
--- a/libpurple/buddyicon.c	Tue Apr 24 19:12:48 2007 +0000
+++ b/libpurple/buddyicon.c	Tue Apr 24 19:58:14 2007 +0000
@@ -491,33 +491,18 @@
 static gboolean
 read_icon_file(const char *path, guchar **data, size_t *len)
 {
-	struct stat st;
+	GError *err = NULL;
 
-	if (!g_stat(path, &st))
+	if (!g_file_get_contents(path, data, len, &err))
 	{
-		FILE *f = g_fopen(path, "rb");
-		if (f)
-		{
-			*data = g_malloc(st.st_size);
-			if (!fread(*data, st.st_size, 1, f))
-			{
-				purple_debug_error("buddyicon", "Error reading %s: %s\n",
-				                   path, strerror(errno));
-				g_free(*data);
-				return FALSE;
-			}
-			fclose(f);
+		purple_debug_error("buddyicon", "Error reading %s: %s\n",
+		                   path, err->message);
+		g_error_free(err);
 
-			*len = st.st_size;
-			return TRUE;
-		}
-		else
-		{
-			purple_debug_error("buddyicon", "Unable to open file %s for reading: %s\n",
-			                   path, strerror(errno));
-		}
+		return FALSE;
 	}
-	return FALSE;
+
+	return TRUE;
 }
 
 PurpleBuddyIcon *

mercurial