[Bug 29531] high-level API for text channels

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Dec 16 16:05:21 CET 2010


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

--- Comment #33 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-12-16 07:05:21 PST ---
(In reply to comment #29)
> (In reply to comment #25)
> > The other things you didn't address with code changes (not removing
> > pending-message-id; comparison with tp-qt4) both seem fine.
> 
> humm sorry, I failed to parse this sentence. :)

Rephrasing: you convinced me that you don't need to do anything about those two
things.

(In reply to comment #32)
> > The handling of PendingMessagesRemoved is a bit unfortunate. If we're waiting
> > for messages' senders to be resolved, and the message is removed while we wait,
> > then we'll ignore the removal, and it'll sit there forever...
> > 
> > The solution would be to have a counter for the number of "pending pending
> > messages", and defer processing of removed messages until it reaches 0, I
> > suppose. I look forward to Telepathy 1.0 making this unnecessary.
> 
> Humm can't we just say that if you really care about this issue then you
> should fix your CM?

Yeah, I suppose so. If we decide to do the "pending pending messages" version
in tp-glib later, it won't be an API break, so we have an escape route if it
turns out to be a practical problem.

OK, ship it!

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