[Bug 29531] high-level API for text channels

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Oct 28 16:35:47 CEST 2010


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

--- Comment #7 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> 2010-10-28 07:35:47 PDT ---
(In reply to comment #5)
> - How should we deal with delivery reporting/failure. See the FIXME in
> send_message_cb. Have a monitor object or something ?

We could have something like that:

gboolean tp_text_channel_send_message_finish (TpTextChannel *self,
    GAsyncResult *result,
    TpMessageDeliveryMonitor **monitor,
    GError **error);

TpMessageDeliveryMonitor::delivery-report-received(TpSignalledMessage *)

But should we then expose delivery report in TpTextChannel::message-received?
I guess we should, except if the message-sent signal create a
TpMessageDeliveryMonitor as well so non-sender process can track the delivery.

But if the delivery report is never send we'll end up with monitor objects
staying around for all the lifetime of the channel.

Actually I'm wondering if that's not overkill and we should just give the
token in tp_text_channel_send_message_finish() and let the user do the
delivery-report mapping itself if it cares.

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