[Bug 26205] High-level API for ContactLists

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=26205

--- Comment #24 from Olli Salli <ollisal at gmail.com> 2011-06-07 05:00:54 PDT ---
(In reply to comment #22)
> For the refcycle problem I see 2 possibilities:
> 

> TpContactList *tp_connection_dup_contact_list(TpConnection*self);
> 
> void tp_contact_list_prepare_async (TpContactList *self, guint n_features,
> TpContactFeature *contact_features);
> 

If going this route, I'd rather have a tp_connection_dup_contact_list_async (or
tp_connection_request_contact_list or whatever naming you use) which takes the
features and a callback, and only gives you the contact list once successfully
retrieved.

> 
> 2) Stop TpContact doing a strong-ref on its TpConnection, and make TpConnection
> strong-ref the roster TpContact.
>
> ...
>
> If this goes together with the breaks for the "telepathy 6.0" plans, I would
> personally vote for solution 2. If we want to merge this sooner, then solution
> 1 is the only choice we have.
> 
> Suggestions?

I vote for this as the long term solution as well.

-- 
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.