[Bug 21594] Implement being a Handler

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=21594





--- Comment #10 from Will Thompson <will.thompson at collabora.co.uk>  2009-05-19 06:20:24 PST ---
(In reply to comment #9)
> (In reply to comment #6)
> > MethodInvocationContext only supports returning <= 8 values? This seems like an
> > unfortunate arbitrary limitation. Can it be raised without an ABI break?
> This is a chosen limitation. QDBusPendingReply already supports only <=8
> values, so we chose to use the same.

Oh, okay.

> > Why is Q_DISABLE_COPY(MethodInvocationContext) in the private: section? Other
> > classes have it as the first thing inside class ____ {
> As there are some typedefs there, I prefer to move the declaration to the
> private section, it looked better to me, but I can change that if you prefer.

Nope, it's fine as is; I didn't know the relevant background.

I think this is all my review comments covered!


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