[Bug 28200] TpBaseContactList

[bugzilla-daemon at freedesktop.org]

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

Simon McVittie <simon.mcvittie at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://git.collabora.co.uk/ |http://git.collabora.co.uk/
                   |?p=user/smcv/telepathy-glib |?p=user/smcv/telepathy-glib
                   |-smcv.git;a=shortlog;h=refs |-smcv.git;a=shortlog;h=refs
                   |/heads/contact-list-manager |/heads/contact-list-fixes

--- Comment #23 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-06-16 11:01:33 PDT ---
Testing with Gabble reveals that TpBaseContactList didn't deal with channel
requests correctly; see branch smcv/contact-list-fixes.

Further thoughts on the design based on Gabble: we either need to:

* add async success/error-reporting to all of TpBaseContactList's virtual
methods; or

* change Gabble and all other CMs to have an overlay of requested state
changes, pretend those changes had taken place for signalling purposes, and
revert them if they turn out to fail; or

* change the spec draft to say that the methods "succeed" instantly, changes
are signalled later, and failure isn't explicitly signalled.

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