[Bug 38248] Get handle identifiers for contacts related to a channel

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Sep 29 15:55:10 CEST 2011


--- Comment #22 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> 2011-09-29 06:55:09 PDT ---

Not mergeable. We should do a spec release and sync tp-glib.

tp_svc_channel_interface_group_emit_handle_owners_changed_detailed (
    obj, empty_hash_table, arr_owners_removed, empty_hash_table);
According to the spec we MAY include the identifier of the removed handles;
any reason to not to?

dup_handle_array(): can't we use g_array_ref? Doesn't dbus-glib play nice with
GArray now?

dup_owners_table(): you don't check if dup_contact returned NULL.

contacts_queue_item_set_contacts(): add a "g_assert (item->contacts == NULL);"
to make sure we don't leak contacts?

* - if @self does not have flags that include
 *   result is undefined;

          "The Connection Manager does not implement the required telepathy "
          "specification to prepare TP_CHANNEL_FEATURE_CONTACTS");
Include the spec version number?

+   * TpChannel::group-contacts-changed:
+   * @added: (type GLib.PtrArray) (element-type TelepathyGLib.Contact):
+   *  a #GList of #TpContact containing the full members added
It's not a GList.
ditto for the other args.
Say that those contacts have been prepared if possible?


+  g_print ("contact added: %s\n", tp_contact_get_identifier (g_ptr_array_index
(added, 0)));
should be removed.

Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.

More information about the telepathy-bugs mailing list