aacid added inline comments.

INLINE COMMENTS

> HelperProxy.cpp:25
> +
> +#include <QDBusUnixFileDescriptor>
> +#include <QDBusArgument>

i don't think we should have dbus stuff in HelperProxy, dbus stuff should be in 
DBusHelperProxy, no?

REPOSITORY
  R283 KAuth

REVISION DETAIL
  https://phabricator.kde.org/D24245

To: volkov, fvogt, chinmoyr, cfeck, #frameworks, security-team
Cc: aacid, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to