Mercurial > pidgin

changeset 13692:ea77ce998dc7

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
[gaim-migrate @ 16095] Part of SF Patch #1437836 from Sadrul "1. gaim_privacy_[permit|deny]_remove checks whether the buddy in the list. So it is not necessary to do that before calling these functions. (first block in the patch)" Additionally, it does a gaim_normalize()d UTF-8 compare, which should be better than strcmp(). committer: Tailor Script <tailor@pidgin.im>
author Richard Laager <rlaager@wiktel.com>
date Tue, 25 Apr 2006 17:27:17 +0000
parents 87b027bb1f15
children 510b7b7dd645
files src/protocols/msn/userlist.c
diffstat 1 files changed, 2 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/src/protocols/msn/userlist.c	Tue Apr 25 17:19:45 2006 +0000
+++ b/src/protocols/msn/userlist.c	Tue Apr 25 17:27:17 2006 +0000
@@ -399,26 +399,14 @@
 	if (list_op & MSN_LIST_AL_OP)
 	{
 		/* These are users who are allowed to see our status. */
-
-		if (g_slist_find_custom(account->deny, passport,
-								(GCompareFunc)strcmp))
-		{
-			gaim_privacy_deny_remove(gc->account, passport, TRUE);
-		}
-
+		gaim_privacy_deny_remove(account, passport, TRUE);
 		gaim_privacy_permit_add(account, passport, TRUE);
 	}
 
 	if (list_op & MSN_LIST_BL_OP)
 	{
 		/* These are users who are not allowed to see our status. */
-
-		if (g_slist_find_custom(account->permit, passport,
-								(GCompareFunc)strcmp))
-		{
-			gaim_privacy_permit_remove(gc->account, passport, TRUE);
-		}
-
+		gaim_privacy_permit_remove(account, passport, TRUE);
 		gaim_privacy_deny_add(account, passport, TRUE);
 	}