cpw.darkrain42.roster: 42b26858: Remove this; it's been commented out for...

darkrain42 at pidgin.im darkrain42 at pidgin.im
Sat Jun 6 20:20:30 EDT 2009


-----------------------------------------------------------------
Revision: 42b26858f379ef210c6754854bfdc0969ab9ebfd
Ancestor: eed256df62cff36f40e4398c900667b920f46f82
Author: darkrain42 at pidgin.im
Date: 2009-06-06T22:47:49
Branch: im.pidgin.cpw.darkrain42.roster
URL: http://d.pidgin.im/viewmtn/revision/info/42b26858f379ef210c6754854bfdc0969ab9ebfd

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

ChangeLog: 

Remove this; it's been commented out for about 3.5 years.

-------------- next part --------------
============================================================
--- libpurple/protocols/jabber/roster.c	e981c0685dccd483b8c0e347b3c22f2fa929d3b2
+++ libpurple/protocols/jabber/roster.c	ae789d5498723948729f1fe1de8e66b72d21f4ba
@@ -194,20 +194,9 @@ void jabber_roster_parse(JabberStream *j
 				jb->subscription = JABBER_SUB_BOTH;
 			else if(!strcmp(subscription, "remove"))
 				jb->subscription = JABBER_SUB_REMOVE;
-			/* XXX: if subscription is now "from" or "none" we need to
-			 * fake a signoff, since we won't get any presence from them
-			 * anymore */
-			/* YYY: I was going to use this, but I'm not sure it's necessary
-			 * anymore, but it's here in case it is. */
-			/*
-			if ((jb->subscription & JABBER_SUB_FROM) ||
-					(jb->subscription & JABBER_SUB_NONE)) {
-				purple_prpl_got_user_status(js->gc->account, jid, "offline", NULL);
-			}
-			*/
 		}
 
-		if(ask && !strcmp(ask, "subscribe"))
+		if(purple_strequal(ask, "subscribe"))
 			jb->subscription |= JABBER_SUB_PENDING;
 		else
 			jb->subscription &= ~JABBER_SUB_PENDING;


More information about the Commits mailing list