Bug 25659 - ObserveChannels implementation might actually return immediately
Summary: ObserveChannels implementation might actually return immediately
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-qt (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-15 11:38 UTC by Will Thompson
Modified: 2010-02-22 07:24 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Will Thompson 2009-12-15 11:38:48 UTC
While learning more about how to implement services with QtDBus, I noticed that TpQt4 doesn't call setDelayedReply(true) on the DBusMessage; from my reading of the documentation, this means that ObserveChannels() is returning immediately, rather than waiting for the library user's implementation to signal that it's ready to return from ObserveChannels(). We should check if this actually works.

(Thiago Macieira informs us that having a DBusMessage argument in the adaptor's method implementation is deprecated—in some nebulous fashion which does not involve marking it as such in the documentation :-)—in favour of the exported object (not adaptor) deriving from DBusContext and calling magic methods to get the current connection and message, etc. But possibly switching to that is orthogonal to checking this potential bug out.)
Comment 1 Andre Moreira Magalhaes 2010-02-22 07:24:28 UTC
Fixed. It will be in release 0.2.2


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.