pidgin: 37706b74: Now that there's no speed difference bet...

[markdoliner at pidgin.im]

----------------------------------------------------------------------
Revision: 37706b74bb67b5e0abf08dd08bcd0d509075cc42
Parent:   e269e95a12c064ea6c89788b5766de059fbb54ec
Author:   markdoliner at pidgin.im
Date:     08/22/11 01:39:00
Branch:   im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/37706b74bb67b5e0abf08dd08bcd0d509075cc42

Changelog: 

Now that there's no speed difference between
purple_notify_user_info_add_pair_html and
purple_notify_user_info_prepend_pair, use the former to that we don't
add items in reverse order.  This is hopefully a little less error
prone?  It also means we won't have to create separate prepend_pair_html
and prepend_pair_plaintext functions

Changes against parent e269e95a12c064ea6c89788b5766de059fbb54ec

  patched  libpurple/protocols/jabber/buddy.c

-------------- next part --------------
============================================================
--- libpurple/protocols/jabber/buddy.c	a27963457bbff0bb49ebb52b0ecef3a23278110a
+++ libpurple/protocols/jabber/buddy.c	b1d903b0b7d334b0ac34c4c3834335bbb62ba4fe
@@ -726,16 +726,39 @@ add_jbr_info(JabberBuddyInfo *jbi, const
 	jbir = g_hash_table_lookup(jbi->resources, resource);
 	user_info = jbi->user_info;
 
-	if (jbr && jbr->client.name) {
-		char *tmp =
-			g_strdup_printf("%s%s%s", jbr->client.name,
-		                    (jbr->client.version ? " " : ""),
-		                    (jbr->client.version ? jbr->client.version : ""));
-		purple_notify_user_info_prepend_pair(user_info, _("Client"), tmp);
+	if (jbr) {
+		char *purdy = NULL;
+		char *tmp;
+		char priority[12];
+		const char *status_name = jabber_buddy_state_get_name(jbr->state);
+
+		g_snprintf(priority, sizeof(priority), "%d", jbr->priority);
+		purple_notify_user_info_add_pair_html(user_info, _("Priority"), priority);
+
+		if (jbr->status) {
+			tmp = purple_markup_escape_text(jbr->status, -1);
+			purdy = purple_strdup_withhtml(tmp);
+			g_free(tmp);
+
+			if (purple_strequal(status_name, purdy))
+				status_name = NULL;
+		}
+
+		tmp = g_strdup_printf("%s%s%s", (status_name ? status_name : ""),
+						((status_name && purdy) ? ": " : ""),
+						(purdy ? purdy : ""));
+		purple_notify_user_info_add_pair_html(user_info, _("Status"), tmp);
+
 		g_free(tmp);
+		g_free(purdy);
+	} else {
+		purple_notify_user_info_add_pair_html(user_info, _("Status"), _("Unknown"));
+	}
 
-		if (jbr->client.os)
-			purple_notify_user_info_prepend_pair(user_info, _("Operating System"), jbr->client.os);
+	if (jbir && jbir->idle_seconds > 0) {
+		char *idle = purple_str_seconds_to_string(jbir->idle_seconds);
+		purple_notify_user_info_add_pair_html(user_info, _("Idle"), idle);
+		g_free(idle);
 	}
 
 	if (jbr && jbr->tz_off != PURPLE_NO_TZ_OFF) {
@@ -751,43 +774,21 @@ add_jbr_info(JabberBuddyInfo *jbi, const
 		                    jbr->tz_off < 0 ? '-' : '+',
 		                    abs(jbr->tz_off / (60*60)),
 		                    abs((jbr->tz_off % (60*60)) / 60));
-		purple_notify_user_info_prepend_pair(user_info, _("Local Time"), timestamp);
+		purple_notify_user_info_add_pair_html(user_info, _("Local Time"), timestamp);
 		g_free(timestamp);
 	}
 
-	if (jbir && jbir->idle_seconds > 0) {
-		char *idle = purple_str_seconds_to_string(jbir->idle_seconds);
-		purple_notify_user_info_prepend_pair(user_info, _("Idle"), idle);
-		g_free(idle);
-	}
-
-	if (jbr) {
-		char *purdy = NULL;
+	if (jbr && jbr->client.name) {
 		char *tmp;
-		char priority[12];
-		const char *status_name = jabber_buddy_state_get_name(jbr->state);
 
-		if (jbr->status) {
-			tmp = purple_markup_escape_text(jbr->status, -1);
-			purdy = purple_strdup_withhtml(tmp);
-			g_free(tmp);
+		if (jbr->client.os)
+			purple_notify_user_info_add_pair_html(user_info, _("Operating System"), jbr->client.os);
 
-			if (purple_strequal(status_name, purdy))
-				status_name = NULL;
-		}
-
-		tmp = g_strdup_printf("%s%s%s", (status_name ? status_name : ""),
-						((status_name && purdy) ? ": " : ""),
-						(purdy ? purdy : ""));
-		purple_notify_user_info_prepend_pair(user_info, _("Status"), tmp);
-
-		g_snprintf(priority, sizeof(priority), "%d", jbr->priority);
-		purple_notify_user_info_prepend_pair(user_info, _("Priority"), priority);
-
+		tmp = g_strdup_printf("%s%s%s", jbr->client.name,
+		                      (jbr->client.version ? " " : ""),
+		                      (jbr->client.version ? jbr->client.version : ""));
+		purple_notify_user_info_add_pair_html(user_info, _("Client"), tmp);
 		g_free(tmp);
-		g_free(purdy);
-	} else {
-		purple_notify_user_info_prepend_pair(user_info, _("Status"), _("Unknown"));
 	}
 }