Mercurial > pidgin

changeset 23615:f4001a130765

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Minor cleanup that's been sitting on my machine for a while.
author Daniel Atallah <daniel.atallah@gmail.com>
date Tue, 29 Jul 2008 18:53:00 +0000
parents c1e23b9d9cf5
children 6659ba7b3941
files libpurple/protocols/msn/user.c
diffstat 1 files changed, 4 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/protocols/msn/user.c	Tue Jul 29 07:02:18 2008 +0000
+++ b/libpurple/protocols/msn/user.c	Tue Jul 29 18:53:00 2008 +0000
@@ -235,21 +235,19 @@
 
 /*add group id to User object*/
 void
-msn_user_add_group_id(MsnUser *user, const char* id)
+msn_user_add_group_id(MsnUser *user, const char* group_id)
 {
 	MsnUserList *userlist;
 	PurpleAccount *account;
 	PurpleBuddy *b;
 	PurpleGroup *g;
 	const char *passport;
-	char *group_id;
 	const char *group_name;
 
 	g_return_if_fail(user != NULL);
-	g_return_if_fail(id != NULL);
+	g_return_if_fail(group_id != NULL);
 
-	group_id = g_strdup(id);
-	user->group_ids = g_list_append(user->group_ids, group_id);
+	user->group_ids = g_list_append(user->group_ids, g_strdup(group_id));
 
 	userlist = user->userlist;
 	account = userlist->session->account;
@@ -261,7 +259,7 @@
 
 	g = purple_find_group(group_name);
 
-	if ((id == NULL) && (g == NULL))
+	if ((group_id == NULL) && (g == NULL))
 	{
 		g = purple_group_new(group_name);
 		purple_blist_add_group(g, NULL);