pidgin.2.5.7: e5ef0741: *** Plucked rev b932d4f94f9c771877b6a8a6...
khc at pidgin.im
khc at pidgin.im
Sat Jun 20 14:30:43 EDT 2009
-----------------------------------------------------------------
Revision: e5ef0741927c783e7a34ed871c430f42c61b8175
Ancestor: 6fe6e9e68841c1c84987b2663e6893ed3dadacbf
Author: khc at pidgin.im
Date: 2009-06-20T18:27:49
Branch: im.pidgin.pidgin.2.5.7
URL: http://d.pidgin.im/viewmtn/revision/info/e5ef0741927c783e7a34ed871c430f42c61b8175
Modified files:
libpurple/protocols/msn/msn.c
libpurple/protocols/msn/notification.c
libpurple/protocols/msn/notification.h
libpurple/protocols/msn/userlist.c
libpurple/protocols/msn/userlist.h
ChangeLog:
*** Plucked rev b932d4f94f9c771877b6a8a6a1d7dd3ea9c08005 (qulogic at pidgin.im):
Treat FQY responses with no network ID as on the Passport network.
Fixes #9240.
*** Plucked rev 76f13d0fc7fbff369d01187fcc1b8f517854e119 (qulogic at pidgin.im):
These strcmp checks are for equality, meaning a return of 0.
*** Plucked rev a7bd74cbe46498f1390a4d33cce115bdd123a68d (qulogic at pidgin.im):
Add a data parameter for FQY callbacks and remove the pending users list
that was previously used instead.
*** Plucked rev 84f6765b71f2ad2d6c7cddd461114ab5fc758b72 (qulogic at pidgin.im):
We can't get an ADD error if we never send an ADD command.
*** Plucked rev 6d3a0b29b22baeedf393b76044dde689caef3b45 (qulogic at pidgin.im):
Add a msn_notification_post_rml analog.
*** Plucked rev 0e2af7d2799397628273f534fac31e6dd7c6fe14 (qulogic at pidgin.im):
Remove some left over code.
*** Plucked rev 0841af13da028245cc279ce8f927ff05c0367ae1 (qulogic at pidgin.im):
Check network type, and send an FQY if necessary, when modifying the allow
and block lists. Before, users who were not added by you (i.e. spammers)
would have no network type and cause disconnects. This should fix it.
Fixes #8977.
*** Plucked rev a1463341c3bf211c429282870babdb4785c7d817 (qulogic at pidgin.im):
Remove notification-server group commands ADG, REG, RMG as groups are now
handled in the AddressBook with SOAP calls.
-------------- next part --------------
============================================================
--- libpurple/protocols/msn/msn.c c555a864b54dca93e4e52d35ef054ebe2d01ac46
+++ libpurple/protocols/msn/msn.c b280031c83a8a8ba7dcbfa6ef35e2812b933eb8b
@@ -1374,56 +1374,36 @@ msn_set_idle(PurpleConnection *gc, int i
msn_change_status(session);
}
-#if 0
+/*
+ * Actually adds a buddy once we have the response from FQY
+ */
static void
-fake_userlist_add_buddy(MsnUserList *userlist,
- const char *who, int list_id,
- const char *group_name)
+add_pending_buddy(MsnSession *session,
+ const char *who,
+ MsnNetwork network,
+ MsnUser *user)
{
- MsnUser *user;
- static int group_id_c = 1;
- int group_id;
+ MsnUserList *userlist = session->userlist;
+ MsnUser *user2;
+ char *group;
- group_id = -1;
+ g_return_if_fail(user != NULL);
- if (group_name != NULL)
- {
- MsnGroup *group;
- group = msn_group_new(userlist, group_id_c, group_name);
- group_id = group_id_c++;
- }
+ group = msn_user_remove_pending_group(user);
- user = msn_userlist_find_user(userlist, who);
-
- if (user == NULL)
- {
- user = msn_user_new(userlist, who, NULL);
+ user2 = msn_userlist_find_user(userlist, who);
+ if (user2 != NULL) {
+ /* User already in userlist, so just update it. */
+ msn_user_destroy(user);
+ user = user2;
+ } else {
msn_userlist_add_user(userlist, user);
}
- else
- if (user->list_op & (1 << list_id))
- {
- if (list_id == MSN_LIST_FL)
- {
- if (group_id >= 0)
- if (g_list_find(user->group_ids,
- GINT_TO_POINTER(group_id)))
- return;
- }
- else
- return;
- }
- if (group_id >= 0)
- {
- /* This is wrong... user->group_ids contains g_strdup()'d data now */
- user->group_ids = g_list_append(user->group_ids,
- GINT_TO_POINTER(group_id));
- }
-
- user->list_op |= (1 << list_id);
+ msn_user_set_network(user, network);
+ msn_userlist_add_buddy(userlist, who, group);
+ g_free(group);
}
-#endif
static void
msn_add_buddy(PurpleConnection *gc, PurpleBuddy *buddy, PurpleGroup *group)
@@ -1440,23 +1420,11 @@ msn_add_buddy(PurpleConnection *gc, Purp
purple_debug_info("msn", "Add user:%s to group:%s\n", who, (group && group->name) ? group->name : "(null)");
if (!session->logged_in)
{
-#if 0
- fake_userlist_add_buddy(session->sync_userlist, who, MSN_LIST_FL,
- group ? group->name : NULL);
-#else
purple_debug_error("msn", "msn_add_buddy called before connected\n");
-#endif
return;
}
-#if 0
- if (group != NULL && group->name != NULL)
- purple_debug_info("msn", "msn_add_buddy: %s, %s\n", who, group->name);
- else
- purple_debug_info("msn", "msn_add_buddy: %s\n", who);
-#endif
-
/* XXX - Would group ever be NULL here? I don't think so...
* shx: Yes it should; MSN handles non-grouped buddies, and this is only
* internal. */
@@ -1469,13 +1437,15 @@ msn_add_buddy(PurpleConnection *gc, Purp
char **tokens;
char *fqy;
/* We need to check the network for this buddy first */
- msn_userlist_save_pending_buddy(userlist, who, group ? group->name : NULL);
+ user = msn_user_new(userlist, who, NULL);
+ msn_user_set_pending_group(user, group ? group->name : NULL);
+ msn_user_set_network(user, MSN_NETWORK_UNKNOWN);
tokens = g_strsplit(who, "@", 2);
fqy = g_strdup_printf("<ml><d n=\"%s\"><c n=\"%s\"/></d></ml>",
tokens[1],
tokens[0]);
msn_notification_send_fqy(session, fqy, strlen(fqy),
- (MsnFqyCb)msn_userlist_add_pending_buddy);
+ (MsnFqyCb)add_pending_buddy, user);
g_free(fqy);
g_strfreev(tokens);
}
============================================================
--- libpurple/protocols/msn/notification.c 4b5b8c5d060df9d6b666315f8384d9cf2224005d
+++ libpurple/protocols/msn/notification.c b607c6534f3d2130478788bd42272edf7e655f67
@@ -149,48 +149,6 @@ msn_notification_disconnect(MsnNotificat
}
/**************************************************************************
- * Util
- **************************************************************************/
-
-static void
-group_error_helper(MsnSession *session, const char *msg, const char *group_id, int error)
-{
- PurpleAccount *account;
- PurpleConnection *gc;
- char *reason = NULL;
- char *title = NULL;
-
- account = session->account;
- gc = purple_account_get_connection(account);
-
- if (error == 224)
- {
- if (group_id == 0)
- {
- return;
- }
- else
- {
- const char *group_name;
- group_name = msn_userlist_find_group_name(session->userlist,group_id);
- reason = g_strdup_printf(_("%s is not a valid group."),
- group_name ? group_name : "");
- }
- }
- else
- {
- reason = g_strdup(_("Unknown error."));
- }
-
- title = g_strdup_printf(_("%s on %s (%s)"), msg,
- purple_account_get_username(account),
- purple_account_get_protocol_name(account));
- purple_notify_error(gc, NULL, title, reason);
- g_free(title);
- g_free(reason);
-}
-
-/**************************************************************************
* Login
**************************************************************************/
@@ -521,6 +479,12 @@ chl_cmd(MsnCmdProc *cmdproc, MsnCommand
/**************************************************************************
* Buddy Lists
**************************************************************************/
+
+typedef struct MsnFqyCbData {
+ MsnFqyCb cb;
+ gpointer data;
+} MsnFqyCbData;
+
/* add contact to xmlnode */
static void
msn_add_contact_xml(MsnSession *session, xmlnode *mlNode,const char *passport, MsnListOp list_op, MsnNetwork networkId)
@@ -593,24 +557,40 @@ msn_notification_post_adl(MsnCmdProc *cm
msn_cmdproc_send_trans(cmdproc, trans);
}
+static void
+msn_notification_post_rml(MsnCmdProc *cmdproc, const char *payload, int payload_len)
+{
+ MsnTransaction *trans;
+ purple_debug_info("msn", "Sending RML with payload: %s\n", payload);
+ trans = msn_transaction_new(cmdproc, "RML", "%i", payload_len);
+ msn_transaction_set_payload(trans, payload, payload_len);
+ msn_cmdproc_send_trans(cmdproc, trans);
+}
+
void
msn_notification_send_fqy(MsnSession *session,
const char *payload, int payload_len,
- MsnFqyCb cb)
+ MsnFqyCb cb,
+ gpointer cb_data)
{
MsnTransaction *trans;
MsnCmdProc *cmdproc;
+ MsnFqyCbData *data;
cmdproc = session->notification->cmdproc;
+ data = g_new(MsnFqyCbData, 1);
+ data->cb = cb;
+ data->data = cb_data;
+
trans = msn_transaction_new(cmdproc, "FQY", "%d", payload_len);
msn_transaction_set_payload(trans, payload, payload_len);
- msn_transaction_set_data(trans, cb);
+ msn_transaction_set_data(trans, data);
msn_cmdproc_send_trans(cmdproc, trans);
}
static void
-update_contact_network(MsnSession *session, const char *passport, MsnNetwork network)
+update_contact_network(MsnSession *session, const char *passport, MsnNetwork network, gpointer unused)
{
MsnUser *user;
@@ -726,7 +706,7 @@ msn_notification_dump_contact(MsnSession
payload = xmlnode_to_str(fqy_node, &payload_len);
msn_notification_send_fqy(session, payload, payload_len,
- update_contact_network);
+ update_contact_network, NULL);
g_free(payload);
xmlnode_free(fqy_node);
@@ -751,7 +731,7 @@ msn_notification_dump_contact(MsnSession
payload = xmlnode_to_str(fqy_node, &payload_len);
msn_notification_send_fqy(session, payload, payload_len,
- update_contact_network);
+ update_contact_network, NULL);
g_free(payload);
}
@@ -961,15 +941,20 @@ fqy_cmd_post(MsnCmdProc *cmdproc, MsnCom
passport = g_strdup_printf("%s@%s", local, domain);
- if (!g_ascii_isdigit(cmd->command[0]) && type != NULL)
- network = (MsnNetwork)strtoul(type, NULL, 10);
- else
+ if (g_ascii_isdigit(cmd->command[0]))
network = MSN_NETWORK_UNKNOWN;
+ else if (type != NULL)
+ network = (MsnNetwork)strtoul(type, NULL, 10);
purple_debug_info("msn", "FQY response says %s is from network %d\n",
passport, network);
- if (cmd->trans->data)
- ((MsnFqyCb)cmd->trans->data)(session, passport, network);
+ if (cmd->trans->data) {
+ MsnFqyCbData *fqy_data = cmd->trans->data;
+ fqy_data->cb(session, passport, network, fqy_data->data);
+ /* TODO: This leaks, but the server responds to FQY multiple times, so we
+ can't free it yet. We need to figure out somewhere else to do so.
+ g_free(fqy_data); */
+ }
g_free(passport);
}
@@ -994,7 +979,7 @@ fqy_error(MsnCmdProc *cmdproc, MsnTransa
FQY error, it's probably going to disconnect us. So it isn't necessary
to tell the handler about it. */
else if (trans->data)
- ((MsnFqyCb)trans->data)(session, NULL, MSN_NETWORK_UNKNOWN); */
+ ((MsnFqyCb)trans->data)(session, NULL, MSN_NETWORK_UNKNOWN, NULL);
#endif
}
@@ -1023,117 +1008,6 @@ static void
}
static void
-add_error(MsnCmdProc *cmdproc, MsnTransaction *trans, int error)
-{
- MsnSession *session;
- PurpleAccount *account;
- PurpleConnection *gc;
- const char *list, *passport;
- char *reason = NULL;
- char *msg = NULL;
- char **params;
-
- session = cmdproc->session;
- account = session->account;
- gc = purple_account_get_connection(account);
- params = g_strsplit(trans->params, " ", 0);
-
- list = params[0];
- passport = params[1];
-
- if (!strcmp(list, "FL"))
- msg = g_strdup_printf(_("Unable to add user on %s (%s)"),
- purple_account_get_username(account),
- purple_account_get_protocol_name(account));
- else if (!strcmp(list, "BL"))
- msg = g_strdup_printf(_("Unable to block user on %s (%s)"),
- purple_account_get_username(account),
- purple_account_get_protocol_name(account));
- else if (!strcmp(list, "AL"))
- msg = g_strdup_printf(_("Unable to permit user on %s (%s)"),
- purple_account_get_username(account),
- purple_account_get_protocol_name(account));
-
- if (!strcmp(list, "FL"))
- {
- if (error == 210)
- {
- reason = g_strdup_printf(_("%s could not be added because "
- "your buddy list is full."), passport);
- }
- }
-
- if (reason == NULL)
- {
- if (error == 208)
- {
- reason = g_strdup_printf(_("%s is not a valid passport account."),
- passport);
- }
- else if (error == 500)
- {
- reason = g_strdup(_("Service Temporarily Unavailable."));
- }
- else
- {
- reason = g_strdup(_("Unknown error."));
- }
- }
-
- if (msg != NULL)
- {
- purple_notify_error(gc, NULL, msg, reason);
- g_free(msg);
- }
-
- if (!strcmp(list, "FL"))
- {
- PurpleBuddy *buddy;
-
- buddy = purple_find_buddy(account, passport);
-
- if (buddy != NULL)
- purple_blist_remove_buddy(buddy);
- }
-
- g_free(reason);
-
- g_strfreev(params);
-}
-
-static void
-adg_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
-{
- MsnSession *session;
- gint group_id;
- const char *group_name;
-
- session = cmdproc->session;
-
- group_id = atoi(cmd->params[3]);
-
- group_name = purple_url_decode(cmd->params[2]);
-
- msn_group_new(session->userlist, cmd->params[3], group_name);
-
- /* There is a user that must be moved to this group */
- if (cmd->trans->data)
- {
- /* msn_userlist_move_buddy(); */
- MsnUserList *userlist = cmdproc->session->userlist;
- MsnCallbackState *data = cmd->trans->data;
-
- if (data->old_group_name != NULL)
- {
- msn_userlist_move_buddy(userlist, data->who, data->old_group_name, group_name);
- g_free(data->old_group_name);
- } else {
- /* msn_add_contact_to_group(userlist, data, data->who, group_name); */
- }
- }
-}
-
-static void
qng_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
{
/* TODO: Call PNG after the timeout specified. */
@@ -1468,61 +1342,6 @@ prp_cmd(MsnCmdProc *cmdproc, MsnCommand
}
}
-static void
-reg_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
-{
- MsnSession *session;
- const char *group_id, *group_name;
-
- session = cmdproc->session;
- group_id = cmd->params[2];
- group_name = purple_url_decode(cmd->params[3]);
-
- msn_userlist_rename_group_id(session->userlist, group_id, group_name);
-}
-
-static void
-reg_error(MsnCmdProc *cmdproc, MsnTransaction *trans, int error)
-{
- const char * group_id;
- char **params;
-
- params = g_strsplit(trans->params, " ", 0);
-
- group_id = params[0];
-
- group_error_helper(cmdproc->session, _("Unable to rename group"), group_id, error);
-
- g_strfreev(params);
-}
-
-static void
-rmg_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
-{
- MsnSession *session;
- const char *group_id;
-
- session = cmdproc->session;
- group_id = cmd->params[2];
-
- msn_userlist_remove_group_id(session->userlist, group_id);
-}
-
-static void
-rmg_error(MsnCmdProc *cmdproc, MsnTransaction *trans, int error)
-{
- const char *group_id;
- char **params;
-
- params = g_strsplit(trans->params, " ", 0);
-
- group_id = params[0];
-
- group_error_helper(cmdproc->session, _("Unable to delete group"), group_id, error);
-
- g_strfreev(params);
-}
-
/**************************************************************************
* Misc commands
**************************************************************************/
@@ -2075,10 +1894,54 @@ system_msg(MsnCmdProc *cmdproc, MsnMessa
g_hash_table_destroy(table);
}
+/**************************************************************************
+ * Dispatch server list management
+ **************************************************************************/
+typedef struct MsnAddRemoveListData {
+ MsnCmdProc *cmdproc;
+ MsnUser *user;
+ MsnListOp list_op;
+ gboolean add;
+} MsnAddRemoveListData;
+
+static void
+modify_unknown_buddy_on_list(MsnSession *session, const char *passport,
+ MsnNetwork network, gpointer data)
+{
+ MsnAddRemoveListData *addrem = data;
+ MsnCmdProc *cmdproc;
+ xmlnode *node;
+ char *payload;
+ int payload_len;
+
+ cmdproc = addrem->cmdproc;
+
+ /* Update user first */
+ msn_user_set_network(addrem->user, network);
+
+ node = xmlnode_new("ml");
+ node->child = NULL;
+
+ msn_add_contact_xml(session, node, passport,
+ addrem->list_op, network);
+
+ payload = xmlnode_to_str(node, &payload_len);
+ xmlnode_free(node);
+
+ if (addrem->add)
+ msn_notification_post_adl(cmdproc, payload, payload_len);
+ else
+ msn_notification_post_rml(cmdproc, payload, payload_len);
+
+ g_free(payload);
+ g_free(addrem);
+}
+
void
msn_notification_add_buddy_to_list(MsnNotification *notification, MsnListId list_id,
MsnUser *user)
{
+ MsnAddRemoveListData *addrem;
MsnCmdProc *cmdproc;
MsnListOp list_op = 1 << list_id;
xmlnode *adl_node;
@@ -2093,11 +1956,23 @@ msn_notification_add_buddy_to_list(MsnNo
msn_add_contact_xml(notification->session, adl_node, user->passport,
list_op, user->networkid);
- payload = xmlnode_to_str(adl_node,&payload_len);
+ payload = xmlnode_to_str(adl_node, &payload_len);
xmlnode_free(adl_node);
- msn_notification_post_adl(notification->servconn->cmdproc,
- payload,payload_len);
+ if (user->networkid != MSN_NETWORK_UNKNOWN) {
+ msn_notification_post_adl(cmdproc, payload, payload_len);
+
+ } else {
+ addrem = g_new(MsnAddRemoveListData, 1);
+ addrem->cmdproc = cmdproc;
+ addrem->user = user;
+ addrem->list_op = list_op;
+ addrem->add = TRUE;
+
+ msn_notification_send_fqy(notification->session, payload, payload_len,
+ modify_unknown_buddy_on_list, addrem);
+ }
+
g_free(payload);
}
@@ -2105,8 +1980,8 @@ msn_notification_rem_buddy_from_list(Msn
msn_notification_rem_buddy_from_list(MsnNotification *notification, MsnListId list_id,
MsnUser *user)
{
+ MsnAddRemoveListData *addrem;
MsnCmdProc *cmdproc;
- MsnTransaction *trans;
MsnListOp list_op = 1 << list_id;
xmlnode *rml_node;
char *payload;
@@ -2123,10 +1998,20 @@ msn_notification_rem_buddy_from_list(Msn
payload = xmlnode_to_str(rml_node, &payload_len);
xmlnode_free(rml_node);
- purple_debug_info("msn", "Send RML with payload:\n%s\n", payload);
- trans = msn_transaction_new(cmdproc, "RML","%" G_GSIZE_FORMAT, strlen(payload));
- msn_transaction_set_payload(trans, payload, strlen(payload));
- msn_cmdproc_send_trans(cmdproc, trans);
+ if (user->networkid != MSN_NETWORK_UNKNOWN) {
+ msn_notification_post_rml(cmdproc, payload, payload_len);
+
+ } else {
+ addrem = g_new(MsnAddRemoveListData, 1);
+ addrem->cmdproc = cmdproc;
+ addrem->user = user;
+ addrem->list_op = list_op;
+ addrem->add = FALSE;
+
+ msn_notification_send_fqy(notification->session, payload, payload_len,
+ modify_unknown_buddy_on_list, addrem);
+ }
+
g_free(payload);
}
@@ -2149,9 +2034,6 @@ msn_notification_init(void)
msn_table_add_cmd(cbs_table, "VER", "VER", ver_cmd);
msn_table_add_cmd(cbs_table, "PRP", "PRP", prp_cmd);
msn_table_add_cmd(cbs_table, "BLP", "BLP", blp_cmd);
- msn_table_add_cmd(cbs_table, "REG", "REG", reg_cmd);
- msn_table_add_cmd(cbs_table, "ADG", "ADG", adg_cmd);
- msn_table_add_cmd(cbs_table, "RMG", "RMG", rmg_cmd);
msn_table_add_cmd(cbs_table, "XFR", "XFR", xfr_cmd);
/* Asynchronous */
@@ -2184,11 +2066,8 @@ msn_notification_init(void)
msn_table_add_cmd(cbs_table, NULL, "241", adl_241_error_cmd);
- msn_table_add_error(cbs_table, "ADD", add_error);
msn_table_add_error(cbs_table, "ADL", adl_error);
msn_table_add_error(cbs_table, "FQY", fqy_error);
- msn_table_add_error(cbs_table, "REG", reg_error);
- msn_table_add_error(cbs_table, "RMG", rmg_error);
msn_table_add_error(cbs_table, "USR", usr_error);
msn_table_add_msg_type(cbs_table,
============================================================
--- libpurple/protocols/msn/notification.h c5fda71615516de493ca7e188a4573602e0e67fc
+++ libpurple/protocols/msn/notification.h efe0cdabd251a4338c91706bf251c3129e0bba70
@@ -58,7 +58,7 @@ struct _MsnNotification
gboolean in_use;
};
-typedef void (*MsnFqyCb)(MsnSession *session, const char *passport, MsnNetwork network);
+typedef void (*MsnFqyCb)(MsnSession *session, const char *passport, MsnNetwork network, gpointer data);
#include "state.h"
void uum_send_msg(MsnSession *session,MsnMessage *msg);
@@ -73,7 +73,7 @@ void msn_notification_send_fqy(MsnSessio
void msn_notification_send_fqy(MsnSession *session,
const char *payload, int payload_len,
- MsnFqyCb cb);
+ MsnFqyCb cb, gpointer cb_data);
MsnNotification *msn_notification_new(MsnSession *session);
void msn_notification_destroy(MsnNotification *notification);
============================================================
--- libpurple/protocols/msn/userlist.c 8b8355df7ccdba233c4785c22421e39277458651
+++ libpurple/protocols/msn/userlist.c 7d7abf28e603c296f0bfc16eccea7968d9138a8c
@@ -748,70 +748,7 @@ msn_userlist_add_buddy(MsnUserList *user
msn_add_contact_to_group(userlist->session, state, who, group_id);
}
-/*
- * Save a buddy address/group until we get back response from FQY
- */
void
-msn_userlist_save_pending_buddy(MsnUserList *userlist,
- const char *who,
- const char *group_name)
-{
- MsnUser *user;
-
- g_return_if_fail(userlist != NULL);
-
- user = msn_user_new(userlist, who, NULL);
- msn_user_set_pending_group(user, group_name);
- msn_user_set_network(user, MSN_NETWORK_UNKNOWN);
- userlist->pending = g_list_prepend(userlist->pending, user);
-}
-
-/*
- * Actually adds a buddy once we have the response from FQY
- */
-void
-msn_userlist_add_pending_buddy(MsnSession *session,
- const char *who,
- /*MsnNetwork*/ int network)
-{
- MsnUserList *userlist = session->userlist;
- MsnUser *user = NULL;
- MsnUser *user2;
- GList *l;
- char *group;
-
- for (l = userlist->pending; l != NULL; l = l->next)
- {
- user = (MsnUser *)l->data;
-
- if (!g_strcasecmp(who, user->passport)) {
- userlist->pending = g_list_delete_link(userlist->pending, l);
- break;
- }
- }
-
- if (user == NULL) {
- purple_debug_error("msn", "Attempting to add a pending user that does not exist.\n");
- return;
- }
-
- group = msn_user_remove_pending_group(user);
-
- user2 = msn_userlist_find_user(userlist, who);
- if (user2 != NULL) {
- /* User already in userlist, so just update it. */
- msn_user_destroy(user);
- user = user2;
- } else {
- msn_userlist_add_user(userlist, user);
- }
-
- msn_user_set_network(user, network);
- msn_userlist_add_buddy(userlist, who, group);
- g_free(group);
-}
-
-void
msn_userlist_add_buddy_to_list(MsnUserList *userlist, const char *who,
MsnListId list_id)
{
============================================================
--- libpurple/protocols/msn/userlist.h 236de1e3340d8437f04814cb4b1ff27caca260d2
+++ libpurple/protocols/msn/userlist.h 271de166efca23d605bf8a7c00f6c723584c4e6d
@@ -47,7 +47,6 @@ struct _MsnUserList
GList *users; /* Contains MsnUsers */
GList *groups; /* Contains MsnGroups */
- GList *pending; /* MsnUsers pending addition (waiting for FQY response) */
GQueue *buddy_icon_requests;
int buddy_icon_window;
@@ -92,12 +91,6 @@ void msn_userlist_add_buddy(MsnUserList
void msn_userlist_rem_buddy(MsnUserList *userlist, const char *who);
void msn_userlist_add_buddy(MsnUserList *userlist,
const char *who, const char *group_name);
-void msn_userlist_save_pending_buddy(MsnUserList *userlist,
- const char *who,
- const char *group_name);
-void msn_userlist_add_pending_buddy(MsnSession *session,
- const char *who,
- /*MsnNetwork*/ int network);
void msn_userlist_move_buddy(MsnUserList *userlist, const char *who,
const char *old_group_name,
const char *new_group_name);
More information about the Commits
mailing list