]> git.0d.be Git - empathy.git/commit
Fixed some of Gillaume's review comments from Bug #571642
authorJonathon Jongsma <jonathon.jongsma@collabora.co.uk>
Fri, 13 Feb 2009 17:35:42 +0000 (11:35 -0600)
committerGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Wed, 18 Nov 2009 19:09:34 +0000 (19:09 +0000)
commitea9f4b06393425e7496a3494e084520da3de7618
tree0e44ddd8d2648f4ee79f7a57ba583ead70519942
parent500093ef89387aead8be8086e33c05fa250446e9
Fixed some of Gillaume's review comments from Bug #571642

The following comments should be fixed in this commit:
- tp_group_update_members: add "old" and "new" in comments so we known which
  contact we are talking about.
- Always use TpHandle instead of guint when storing handles in variables.
- contact_list_store_member_renamed_cb: would be good to share the common code
  with contact_list_store_members_changed_cb in a common function

One comment about the last one: I split out common code into two new functions:
contact_list_store_add_contact_and_connect(), and the matching
contact_list_store_remove_contact_and_disconnect().  I wondered whether the
signal connection/disconnection should just be added to the _add_contact() and
_remove_contact() functions directly, but they do seem to be used in other
places without connecting to signals, so i thought it would be safer to simply
add some wrapper functions.
libempathy-gtk/empathy-contact-list-store.c