Mercurial > pidgin.yaz
diff src/buddyicon.c @ 11423:202a3b3c5a88
[gaim-migrate @ 13660]
Jason LeBrun asked for this on gaim-devel. gaim_buddy_icon_new() now leaves a reference for the caller.
committer: Tailor Script <tailor@pidgin.im>
| author | Richard Laager <rlaager@wiktel.com> |
|---|---|
| date | Fri, 02 Sep 2005 06:57:54 +0000 |
| parents | 17142948653e |
| children | d0662345799b |
line wrap: on
line diff
--- a/src/buddyicon.c Fri Sep 02 05:46:28 2005 +0000 +++ b/src/buddyicon.c Fri Sep 02 06:57:54 2005 +0000 @@ -75,11 +75,13 @@ gaim_buddy_icon_ref(icon); gaim_buddy_icon_set_data(icon, icon_data, icon_len); - gaim_buddy_icon_unref(icon); - /* We don't take a reference here. gaim_buddy_icon_set_data() makes blist.c or - conversation.c, or both, do that for us. + /* gaim_buddy_icon_set_data() makes blist.c or + * conversation.c, or both, take a reference. + * + * Plus, we leave one for the caller of this function. */ + return icon; } @@ -412,7 +414,8 @@ } else { - gaim_buddy_icon_new(account, username, icon_data, icon_len); + GaimBuddyIcon *icon = gaim_buddy_icon_new(account, username, icon_data, icon_len); + gaim_buddy_icon_unref(icon); } }