# HG changeset patch
# User Mark Doliner <mark@kingant.net>
# Date 1231469018 0
# Node ID 6f1f809d6fa21cfb37123078641ee9e71299070c
# Parent  ded121a7fbb6d508ef9c9204742dec6c7efc65f7
Use purple_buddy_get_group() instead of aim_ssi_itemlist_exists() to check
if the buddy is in our list or not.  Should be much more efficient.

diff -r ded121a7fbb6 -r 6f1f809d6fa2 libpurple/protocols/oscar/oscar.c
--- a/libpurple/protocols/oscar/oscar.c	Fri Jan 09 01:45:45 2009 +0000
+++ b/libpurple/protocols/oscar/oscar.c	Fri Jan 09 02:43:38 2009 +0000
@@ -6200,8 +6200,9 @@
 		menu = g_list_prepend(menu, act);
 	}
 
-	if (aim_ssi_itemlist_exists(od->ssi.local, buddy->name) != NULL)
+	if (purple_buddy_get_group(buddy) != NULL)
 	{
+		/* We only do this if the user is in our buddy list */
 		act = purple_menu_action_new(_("Edit Buddy Comment"),
 		                           PURPLE_CALLBACK(oscar_buddycb_edit_comment),
 		                           NULL, NULL);
@@ -6240,7 +6241,7 @@
 #endif
 	}
 
-	if (od->ssi.received_data)
+	if (od->ssi.received_data && purple_buddy_get_group(buddy) != NULL)
 	{
 		char *gname;
 		gname = aim_ssi_itemlist_findparentname(od->ssi.local, buddy->name);