pidgin: e9fdc42f: Improve these comments? I was having a ...

markdoliner at pidgin.im markdoliner at pidgin.im
Thu Aug 12 13:10:26 EDT 2010


----------------------------------------------------------------------
Revision: e9fdc42f15dcc8d398fe439c63f6fce243549303
Parent:   4cd1560a0773ff2b69e5121c201e5dbfd33cabc0
Author:   markdoliner at pidgin.im
Date:     08/12/10 13:05:07
Branch:   im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/e9fdc42f15dcc8d398fe439c63f6fce243549303

Changelog: 

Improve these comments?  I was having a hard time trying to figure out what
they meant.  Also, do we even use the MSN_NON_IM_GROUP??  It looks like
we don't.

Changes against parent 4cd1560a0773ff2b69e5121c201e5dbfd33cabc0

  patched  libpurple/protocols/msn/contact.c

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/contact.c	d8fafb300e6376dfdc1111b798f1c3b830e0bebb
+++ libpurple/protocols/msn/contact.c	92383913d29f9934d5915a75d155d7f16d89ff4d
@@ -885,7 +885,7 @@ msn_parse_addressbook(MsnSession *sessio
 		msn_parse_addressbook_groups(session, groups);
 	}
 
-	/*add a default No group to set up the no group Membership*/
+	/* Add an "Other Contacts" group for buddies who aren't in a group */
 	msn_group_new(session->userlist, MSN_INDIVIDUALS_GROUP_ID,
 				  MSN_INDIVIDUALS_GROUP_NAME);
 	purple_debug_misc("msn", "AB group_id:%s name:%s\n",
@@ -895,7 +895,7 @@ msn_parse_addressbook(MsnSession *sessio
 		purple_blist_add_group(g, NULL);
 	}
 
-	/*add a default No group to set up the no group Membership*/
+	/* Add a "Non-IM Contacts" group */
 	msn_group_new(session->userlist, MSN_NON_IM_GROUP_ID, MSN_NON_IM_GROUP_NAME);
 	purple_debug_misc("msn", "AB group_id:%s name:%s\n", MSN_NON_IM_GROUP_ID, MSN_NON_IM_GROUP_NAME);
 	if ((purple_find_group(MSN_NON_IM_GROUP_NAME)) == NULL) {


More information about the Commits mailing list