Mercurial > pidgin.yaz
diff src/protocols/oscar/ssi.c @ 4344:d6f1086d52a6
[gaim-migrate @ 4609]
A few small changes...
The "orphans" groups should stop reappearing every time you sign
online. Twas a case of too much error checking.
Buddies, permits, and denies with empty names will all be deleted
automatically by the cleaning function. This should never really
happen, but somehow one of my accounts had some empty-named permit
buddies, so I threw it in here.
I also took out some BR tags before some HR tags. I really really
really think it looks better this way. Feel free to disagree.
committer: Tailor Script <tailor@pidgin.im>
author | Mark Doliner <mark@kingant.net> |
---|---|
date | Sun, 19 Jan 2003 07:35:18 +0000 |
parents | 051265d35a36 |
children | 2903c656875a |
line wrap: on
line diff
--- a/src/protocols/oscar/ssi.c Sun Jan 19 03:29:00 2003 +0000 +++ b/src/protocols/oscar/ssi.c Sun Jan 19 07:35:18 2003 +0000 @@ -643,6 +643,21 @@ { struct aim_ssi_item *cur, *next; + /* Delete any buddies, permits, or denies with empty names */ + cur = sess->ssi.local; + while (cur) { + next = cur->next; + if (!cur->name) { + if (cur->type == AIM_SSI_TYPE_BUDDY) + aim_ssi_delbuddy(sess, conn, cur->name, NULL); + else if (cur->type == AIM_SSI_TYPE_PERMIT) + aim_ssi_delpermit(sess, conn, cur->name); + else if (cur->type == AIM_SSI_TYPE_DENY) + aim_ssi_deldeny(sess, conn, cur->name); + } + cur = next; + } + /* If there are any buddies directly in the master group, put them in a real group */ /* This will kind of mess up if you hit the item limit, but this function isn't too critical */ for (cur=sess->ssi.local; cur; cur=cur->next) @@ -794,7 +809,7 @@ { struct aim_ssi_item *del; - if (!sess || !conn || !name || !group) + if (!sess || !conn || !name) return -EINVAL; /* Find the buddy */ @@ -838,7 +853,7 @@ { struct aim_ssi_item *del; - if (!sess || !conn || !name) + if (!sess || !conn) return -EINVAL; /* Find the item */ @@ -866,7 +881,7 @@ { struct aim_ssi_item *del; - if (!sess || !conn || !name) + if (!sess || !conn) return -EINVAL; /* Find the item */