Mercurial > pidgin.yaz
diff libpurple/protocols/msn/userlist.c @ 23566:a0e957b7b923
Take care of more cleanup in the MSN code.
No more C++ style comments.
msn_convert_iso8601 was replaced with purple_str_to_time everywhere, so
remove it.
REM and SYN were removed in MSNP13.
REA was removed in MSNP11.
PRP and BLP were added already, so remove the TODO.
Contact information is now taken from the contact server, so remove all
that commented code from the ADL command.
author | Elliott Sales de Andrade <qulogic@pidgin.im> |
---|---|
date | Sat, 12 Jul 2008 06:44:19 +0000 |
parents | 7bceac816e19 |
children | 16734635febf 3f4b6162995f |
line wrap: on
line diff
--- a/libpurple/protocols/msn/userlist.c Sat Jul 12 04:50:47 2008 +0000 +++ b/libpurple/protocols/msn/userlist.c Sat Jul 12 06:44:19 2008 +0000 @@ -138,36 +138,6 @@ return FALSE; } -#if 0 -static const char* -get_friendly_name(MsnUser *user) -{ - const char *friendly_name; - - g_return_val_if_fail(user != NULL, NULL); - - friendly_name = msn_user_get_friendly_name(user); - - if (friendly_name != NULL) - friendly_name = purple_url_encode(friendly_name); - else - friendly_name = msn_user_get_passport(user); - - /* this might be a bit of a hack, but it should prevent notification server - * disconnections for people who have buddies with insane friendly names - * who added you to their buddy list from being disconnected. Stu. */ - /* Shx: What? Isn't the store_name obtained from the server, and hence it's - * below the BUDDY_ALIAS_MAXLEN ? */ - /* Stu: yeah, that's why it's a bit of a hack, as you pointed out, we're - * probably decoding the incoming store_name wrong, or something. bleh. */ - - if (strlen(friendly_name) > BUDDY_ALIAS_MAXLEN) - friendly_name = msn_user_get_passport(user); - - return friendly_name; -} -#endif - /************************************************************************** * Server functions **************************************************************************/ @@ -256,7 +226,7 @@ * looked at this. Maybe we should use the store * name instead? --KingAnt */ -// got_new_entry(gc, passport, friendly); +/* got_new_entry(gc, passport, friendly); */ } } @@ -386,7 +356,7 @@ if (!(list_op & (MSN_LIST_AL_OP | MSN_LIST_BL_OP))) { -// got_new_entry(gc, passport, store); +/* got_new_entry(gc, passport, store); */ } } @@ -779,10 +749,6 @@ return; } - //friendly_name = (user != NULL) ? get_friendly_name(user) : who; - - //purple_debug_info("msn", "store_name = %s\n", store_name); - /* XXX: see XXX above, this should really be done when we get the response from the server */