# HG changeset patch
# User Elliott Sales de Andrade <qulogic@pidgin.im>
# Date 1227845732 0
# Node ID 9a3ed3b7ac4baee26bc0fda913776250eeae250a
# Parent  7a5f6ddd11b1e66b6fab5c4b3b3026aafbd9a6ed
Check for NULL group_id before calling msn_group_new since that function will
return if you give it a NULL id. This avoids an extra debug message.

diff -r 7a5f6ddd11b1 -r 9a3ed3b7ac4b libpurple/protocols/msn/contact.c
--- a/libpurple/protocols/msn/contact.c	Mon Nov 24 06:05:01 2008 +0000
+++ b/libpurple/protocols/msn/contact.c	Fri Nov 28 04:15:32 2008 +0000
@@ -445,16 +445,16 @@
 		if ((groupInfo = xmlnode_get_child(group, "groupInfo")) && (groupname = xmlnode_get_child(groupInfo, "name")))
 			group_name = xmlnode_get_data(groupname);
 
-		msn_group_new(session->userlist, group_id, group_name);
-
-		if (group_id == NULL){
+		if (group_id == NULL) {
 			/* Group of ungroupped buddies */
 			g_free(group_name);
 			continue;
 		}
 
+		msn_group_new(session->userlist, group_id, group_name);
+
 		purple_debug_info("msn", "AB group_id: %s, name: %s\n", group_id, group_name ? group_name : "(null)");
-		if ((purple_find_group(group_name)) == NULL){
+		if ((purple_find_group(group_name)) == NULL) {
 			PurpleGroup *g = purple_group_new(group_name);
 			purple_blist_add_group(g, NULL);
 		}