Mercurial > pidgin
diff src/protocols/yahoo/yahoo.c @ 13968:6fc412e59214
[gaim-migrate @ 16525]
A bunch of little things
* Use GAIM_CONNECTION_IS_VALID(gc) in a lot of places where
we were doing g_list_find(gaim_connections_get_all(), gc)
* Get rid of a lot of places where we were doing
g_list_find(gaim_connections_get_all(), gc). The handle used
by the request API ensures that the ok and cancel callback
functions won't be called if the gc is destroyed. However,
GAIM_CONNECTION_IS_VALID(gc) is still very important for
callback functions where we can't cancel the request.
For example, gaim_proxy_connect() callback functions.
* "Added" a function to Yahoo! that should help us notice
when our buddies change their buddy icon/display picture
* Some comments in a few places
* Changed GAIM_CONNECTION_IS_VALID(gc) to only look through
the list of "all" connections and not the list of
"connecting" connections. Some time ago we changed how
this was done so that the list of "all" connections now
includes the "connection" connections.
committer: Tailor Script <tailor@pidgin.im>
author | Mark Doliner <mark@kingant.net> |
---|---|
date | Thu, 20 Jul 2006 08:11:54 +0000 |
parents | b5ff22440a5b |
children | eb1a7d176149 |
line wrap: on
line diff
--- a/src/protocols/yahoo/yahoo.c Thu Jul 20 07:31:15 2006 +0000 +++ b/src/protocols/yahoo/yahoo.c Thu Jul 20 08:11:54 2006 +0000 @@ -810,6 +810,7 @@ if (!msg || !g_utf8_validate(msg, -1, NULL)) return; + /* TODO: Does this really need to be escaped? It seems like it doesn't. */ escmsg = g_markup_escape_text(msg, -1); prim = g_strdup_printf(_("Yahoo! system message for %s:"), @@ -2132,6 +2133,9 @@ case YAHOO_SERVICE_PICTURE_UPLOAD: yahoo_process_picture_upload(gc, pkt); break; + case YAHOO_SERVICE_AVATAR_UPDATE: + yahoo_process_avatar_update(gc, pkt); + break; case YAHOO_SERVICE_AUDIBLE: yahoo_process_audible(gc, pkt); break; @@ -2234,7 +2238,7 @@ struct yahoo_data *yd; struct yahoo_packet *pkt; - if (!g_list_find(gaim_connections_get_all(), gc)) { + if (!GAIM_CONNECTION_IS_VALID(gc)) { close(source); return; } @@ -2261,7 +2265,7 @@ struct yahoo_data *yd; struct yahoo_packet *pkt; - if (!g_list_find(gaim_connections_get_all(), gc)) { + if (!GAIM_CONNECTION_IS_VALID(gc)) { close(source); return; }