# HG changeset patch
# User Nathan Walp <nwalp@pidgin.im>
# Date 1064447768 0
# Node ID 13da9fe5f213e5c854c2155f4c28f2aa5c6e9d1d
# Parent  ccc0d9654098b28d41b5288280b0230cca7c637f
[gaim-migrate @ 7508]
ok, I think I'm done optimizing for now

committer: Tailor Script <tailor@pidgin.im>

diff -r ccc0d9654098 -r 13da9fe5f213 src/blist.c
--- a/src/blist.c	Wed Sep 24 23:41:35 2003 +0000
+++ b/src/blist.c	Wed Sep 24 23:56:08 2003 +0000
@@ -238,7 +238,6 @@
 	}
 
 	buddy->timer = 0;
-	gaim_contact_compute_priority_buddy(gaim_buddy_get_contact(buddy));
 
 	if (ops)
 		ops->update(gaimbuddylist, (GaimBlistNode*)buddy);
@@ -255,13 +254,13 @@
 
 void gaim_blist_update_buddy_presence(GaimBuddy *buddy, int presence) {
 	struct gaim_blist_ui_ops *ops = gaimbuddylist->ui_ops;
-	gboolean do_timer = FALSE;
+	gboolean do_something = FALSE;
 
 	if (!GAIM_BUDDY_IS_ONLINE(buddy) && presence) {
 		int old_present = buddy->present;
 		buddy->present = GAIM_BUDDY_SIGNING_ON;
 		gaim_signal_emit(gaim_blist_get_handle(), "buddy-signed-on", buddy);
-		do_timer = TRUE;
+		do_something = TRUE;
 
 		if(old_present != GAIM_BUDDY_SIGNING_OFF) {
 			((GaimContact*)((GaimBlistNode*)buddy)->parent)->online++;
@@ -271,24 +270,27 @@
 	} else if(GAIM_BUDDY_IS_ONLINE(buddy) && !presence) {
 		buddy->present = GAIM_BUDDY_SIGNING_OFF;
 		gaim_signal_emit(gaim_blist_get_handle(), "buddy-signed-off", buddy);
-		do_timer = TRUE;
+		do_something = TRUE;
 	}
 
-	if(do_timer) {
+	if(do_something) {
 		if(buddy->timer > 0)
 			g_source_remove(buddy->timer);
 		buddy->timer = g_timeout_add(10000, (GSourceFunc)presence_update_timeout_cb, buddy);
+
+		gaim_contact_compute_priority_buddy(gaim_buddy_get_contact(buddy));
+		if (ops)
+			ops->update(gaimbuddylist, (GaimBlistNode*)buddy);
 	}
-
-	gaim_contact_compute_priority_buddy(gaim_buddy_get_contact(buddy));
-	if (ops)
-		ops->update(gaimbuddylist, (GaimBlistNode*)buddy);
 }
 
 
 void  gaim_blist_update_buddy_idle (GaimBuddy *buddy, int idle)
 {
 	struct gaim_blist_ui_ops *ops = gaimbuddylist->ui_ops;
+	if(buddy->idle == idle)
+		return;
+
 	buddy->idle = idle;
 	gaim_contact_compute_priority_buddy(gaim_buddy_get_contact(buddy));
 	if (ops)
@@ -298,6 +300,9 @@
 void  gaim_blist_update_buddy_evil (GaimBuddy *buddy, int warning)
 {
 	struct gaim_blist_ui_ops *ops = gaimbuddylist->ui_ops;
+	if(buddy->evil == warning)
+		return;
+
 	buddy->evil = warning;
 	if (ops)
 		ops->update(gaimbuddylist,(GaimBlistNode*)buddy);
@@ -1411,11 +1416,8 @@
 							ops->remove(gaimbuddylist, bnode);
 					}
 				}
-				if(recompute) {
+				if(recompute)
 					gaim_contact_compute_priority_buddy((GaimContact*)cnode);
-						if(ops)
-							ops->remove(gaimbuddylist, bnode);
-				}
 			} else if(GAIM_BLIST_NODE_IS_CHAT(cnode) &&
 					((GaimBlistChat*)cnode)->account == account) {
 				((GaimGroup*)gnode)->currentsize--;