Bug 14184

Summary:	TpDynamicHandleRepo: listen for name owner changes more efficiently
Product:	Telepathy	Reporter:	Simon McVittie <smcv>
Component:	tp-glib	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	enhancement
Priority:	medium	CC:	mikhail.zabaluev
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Bug Depends on:	14183
Bug Blocks:

Description Simon McVittie 2008-01-21 11:14:27 UTC

If dbus-glib didn't indiscriminately connect to NameOwnerChanged, we could avoid waking up every time a process connects to or disconnects from the session bus, by using a less brute-force approach to monitoring the death of client processes in TpDynamicHandleRepo. Until #14183 is fixed, there's no point, though.

Comment 1 Mikhail Zabaluev 2008-12-30 10:20:23 UTC

Note that installing a name-specific watch upon learning about the client connection has a race condition with the client falling off before the watch is effective.
You may want to Ping the client after installing the watch, but that brings another bit of extra traffic in between useful messages. Sending the ping from an idle handler could be an optimization.

Comment 2 Mikhail Zabaluev 2009-01-12 05:19:05 UTC

(In reply to comment #1)
> You may want to Ping the client after installing the watch, but that brings
> another bit of extra traffic in between useful messages. Sending the ping from
> an idle handler could be an optimization.

And of course, there is a method to get the current name owner.

Comment 3 Simon McVittie 2009-09-09 10:43:12 UTC

Fixed in 0.7.35

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.