comparison libpurple/protocols/jabber/roster.c @ 22817:b77d000b738b

Add a comment. Could someone familiar with the Jabber protocol or our Jabber PRPL comment on this (or change my comment to an explanatory one?) Etan? Nathan? Thanks!

22816:04e0d8677b9a

	if(!(who = jabber_get_bare_jid(buddy->name)))
		return;

	jb = jabber_buddy_find(js, buddy->name, FALSE);

	if(!jb || !(jb->subscription & JABBER_SUB_TO)) {
		groups = g_slist_append(groups, group->name);
	}

	jabber_roster_update(js, who, groups);

22817:b77d000b738b

	if(!(who = jabber_get_bare_jid(buddy->name)))
		return;

	jb = jabber_buddy_find(js, buddy->name, FALSE);

	/*
	 * For some reason if we're waiting for our subscription request
	 * to be approved and we try to add the buddy to another group
	 * then we remove the buddy from the old group.  I don't understand
	 * the rationale for this, can someone please explain it?  It seems
	 * like we should pass NULL as the groups parameter to
	 * jabber_roster_update().
	 */
	if(!jb || !(jb->subscription & JABBER_SUB_TO)) {
		groups = g_slist_append(groups, group->name);
	}

	jabber_roster_update(js, who, groups);