[Bug 27835] Avoid being notified about Connection features nobody cares about

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon May 3 18:11:59 CEST 2010


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

--- Comment #3 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-05-03 09:11:59 PDT ---
(In reply to comment #0)
> Proposed telepathy-glib API:
> 
> * implement ACI, RCI unconditionally
> * have an API in TpBaseConnectionClass to declare which interfaces need to
> track "interest counts"
> * have an API for Location to call: void tp_base_connection_add_client_interest
> (TpBaseConnection *, const gchar *unique_name, const gchar *interface);
> * emit signals TpBaseConnection::clients-interested::$INTERFACE,
> TpBaseConnection::clients-uninterested::$INTERFACE on transitions between 0 and
> 1 total interest (we'd have to check that all D-Bus interface name characters
> are allowed in a signal detail)

I implemented all of this in Bug #27948, except for the API for Location to
call into.

In Bug #27948, if you try to Remove an interest you didn't have in the first
place, no error is raised. I don't think this is a problem, because in
practice, what are you going to do about it? - actually, I think Add and Remove
should both be called with no reply (fire-and-forget).

I now wonder whether the Add D-Bus method should become singular (rationale: in
practice, nobody's going to subscribe to multiple things at once).

The Remove method should probably stay plural, so a hypothetical future
TpConnection can release all of its subscriptions at once.

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



More information about the telepathy-bugs mailing list