# HG changeset patch # User Paul Aurich # Date 1244328469 0 # Node ID 65b5bc5432142bda43cea9611885623a98164988 # Parent ff099a916d6b7ae4a01f906ddab6d41c5809dfef Remove this; it's been commented out for about 3.5 years. diff -r ff099a916d6b -r 65b5bc543214 libpurple/protocols/jabber/roster.c --- a/libpurple/protocols/jabber/roster.c Sat Jun 06 22:38:15 2009 +0000 +++ b/libpurple/protocols/jabber/roster.c Sat Jun 06 22:47:49 2009 +0000 @@ -194,20 +194,9 @@ 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;