pidgin: e4758002: Fix the _add_pair_html/plaintext calls.

qulogic at pidgin.im qulogic at pidgin.im
Tue Aug 23 23:35:58 EDT 2011


----------------------------------------------------------------------
Revision: e4758002eaeecbc33a4c70f5d66aec13bf13abce
Parent:   951efa92ea29f24db3d8d12f604ca9e149e94c96
Author:   qulogic at pidgin.im
Date:     08/23/11 23:30:47
Branch:   im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/e4758002eaeecbc33a4c70f5d66aec13bf13abce

Changelog: 

Fix the _add_pair_html/plaintext calls.

Changes against parent 951efa92ea29f24db3d8d12f604ca9e149e94c96

  patched  libpurple/protocols/msn/msn.c

-------------- next part --------------
============================================================
--- libpurple/protocols/msn/msn.c	8ba865c149b31ccdbcc97a49bf8f6bf24dfc40ba
+++ libpurple/protocols/msn/msn.c	90d1a419a46e9141a2df047fd7828bd59eeae49b
@@ -1101,8 +1101,6 @@ msn_tooltip_text(PurpleBuddy *buddy, Pur
 		}
 
 		if (currentmedia) {
-			/* TODO: Check whether it's correct to call add_pair_html,
-			         or if we should be using add_pair_plaintext */
 			purple_notify_user_info_add_pair_html(user_info, mediatype, currentmedia);
 			g_free(currentmedia);
 		}
@@ -1129,23 +1127,17 @@ msn_tooltip_text(PurpleBuddy *buddy, Pur
 
 		phone = msn_user_get_home_phone(user);
 		if (phone != NULL) {
-			/* TODO: Check whether it's correct to call add_pair_html,
-			         or if we should be using add_pair_plaintext */
-			purple_notify_user_info_add_pair_html(user_info, _("Home Phone Number"), phone);
+			purple_notify_user_info_add_pair_plaintext(user_info, _("Home Phone Number"), phone);
 		}
 
 		phone = msn_user_get_work_phone(user);
 		if (phone != NULL) {
-			/* TODO: Check whether it's correct to call add_pair_html,
-			         or if we should be using add_pair_plaintext */
-			purple_notify_user_info_add_pair_html(user_info, _("Work Phone Number"), phone);
+			purple_notify_user_info_add_pair_plaintext(user_info, _("Work Phone Number"), phone);
 		}
 
 		phone = msn_user_get_mobile_phone(user);
 		if (phone != NULL) {
-			/* TODO: Check whether it's correct to call add_pair_html,
-			         or if we should be using add_pair_plaintext */
-			purple_notify_user_info_add_pair_html(user_info, _("Mobile Phone Number"), phone);
+			purple_notify_user_info_add_pair_plaintext(user_info, _("Mobile Phone Number"), phone);
 		}
 	}
 }


More information about the Commits mailing list