cpw.darkrain42.roster, im.pidgin.pidgin: ab9fcf28: Clarify a comment a little.

darkrain42 at pidgin.im darkrain42 at pidgin.im
Sun Jul 12 01:57:23 EDT 2009


-----------------------------------------------------------------
Revision: ab9fcf2868687b3a72855ccb1496fca58035dda4
Ancestor: 14e9968d26dbac46295dfaaf180c849241aed847
Author: darkrain42 at pidgin.im
Date: 2009-07-12T03:44:30
Branch: im.pidgin.cpw.darkrain42.roster
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/ab9fcf2868687b3a72855ccb1496fca58035dda4

Modified files:
        libpurple/protocols/jabber/roster.c

ChangeLog: 

Clarify a comment a little.

-------------- next part --------------
============================================================
--- libpurple/protocols/jabber/roster.c	e45883f7205b2f439f2c1856b97256fb6a8a6d0f
+++ libpurple/protocols/jabber/roster.c	b3f625f5ea45285c4250bebe38a6bbee2ac1bbee
@@ -94,8 +94,9 @@ static void add_purple_buddy_to_groups(J
 
 		/* XMPP groups are case-sensitive, but libpurple groups are
 		 * case-insensitive. We treat a buddy in both "Friends" and "friends"
-		 * as only being in one group, so if we push changes about the buddy
+		 * as only being in one group, but if we push changes about the buddy
 		 * to the server, the buddy will be dropped from one of the groups.
+		 * Not optimal, but better than the alternative, I think.
 		 */
 		if((l = g_slist_find_custom(groups, purple_group_get_name(g), (GCompareFunc)purple_utf8_strcasecmp))) {
 			/* The buddy is already on the local list. Update info. */


More information about the Commits mailing list