# HG changeset patch # User Daniel Atallah # Date 1217357580 0 # Node ID f4001a13076529c7ed3a74913a9cede5e7be1590 # Parent c1e23b9d9cf5d8eb7df8d283279e3ecc2c6c1535 Minor cleanup that's been sitting on my machine for a while. diff -r c1e23b9d9cf5 -r f4001a130765 libpurple/protocols/msn/user.c --- 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);