annotate PLUGIN_HOWTO @ 22401:d9105ead88dc

When purple_buddy_icons_set_account_icon() is called, it unrefs the old PurpleStoredImage and refs the new one. Previously, it notified the prpl of the change in the buddy icon before updating pointer_icon_cache, which meant that if the prpl then called purple_buddy_icons_find_account_icon() it would get the old PurpleStoredImage (which is at this point not only old but also a pointer to invalid memory if unref'ing it caused it to be destroyed). This happens in jabber_set_info() as of 2.4.0, causing a crash when setting no-buddy-icon for an account after it has previously had an icon. I think this also means that XMPP accounts in 2.4.0 will also always set serverside the *last* icon set, not the current one, when changing icons, but I didn't test that. The solution is simple: Update pointer_icon_cache earlier, setting the new img if there is one or removing the account entirely from the hash table if there isn't one now. This fixes both problems described above, making purple_buddy_icons_find_account_icon() return the new, current icon (or NULL) at all times.
author Evan Schoenberg <evan.s@dreskin.net>
date Tue, 04 Mar 2008 00:11:22 +0000
parents 83ec0b408926
children db5a58aabe38
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
15937
cc3c50816cc9 More gaim->whatever isms
Mark Doliner <mark@kingant.net>
parents: 14505
diff changeset
1 For information on writing a plugin for Purple, Pidgin or Finch, go
16176
83ec0b408926 Beat some older documentation into submission.
Richard Laager <rlaager@wiktel.com>
parents: 15937
diff changeset
2 http://developer.pidgin.im and click on API. From there, see the HOWTOs in the
83ec0b408926 Beat some older documentation into submission.
Richard Laager <rlaager@wiktel.com>
parents: 15937
diff changeset
3 "Related Pages" section.
14505
7acebc9d043f [gaim-migrate @ 17225]
Mark Doliner <mark@kingant.net>
parents:
diff changeset
4
7acebc9d043f [gaim-migrate @ 17225]
Mark Doliner <mark@kingant.net>
parents:
diff changeset
5 You can also generate this documentation locally by installing
15937
cc3c50816cc9 More gaim->whatever isms
Mark Doliner <mark@kingant.net>
parents: 14505
diff changeset
6 doxygen and graphviz dot, then running "make docs" in the
14505
7acebc9d043f [gaim-migrate @ 17225]
Mark Doliner <mark@kingant.net>
parents:
diff changeset
7 source tree. The documentation will be in the docs/html directory.