[Bug 27872] TpBaseClient Handler support

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue May 25 10:30:47 CEST 2010


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

--- Comment #11 from Guillaume Desmottes <guillaume.desmottes at collabora.co.uk> 2010-05-25 01:30:46 PDT ---
(In reply to comment #10)
> > (In reply to comment #8)
> > > +++ b/tests/dbus/base-client.c
> > > > +#include <telepathy-glib/handle-channels-context-internal.h>
> > > 
> > > Why? If possible I'd prefer it if regression tests didn't use internal API.
> > 
> > We use it to easily access to the ctx semi-private attributes (conn, account,
> > channels, etc).
> > Note that we do that for the Observer and Approver as well.
> 
> That's an acceptable use, but I'd prefer it commented.

I added a comment.

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



More information about the telepathy-bugs mailing list