dfaure requested changes to this revision.
dfaure added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> ksambasharetest.cpp:26
> +
> +#include <QtTest>
> +

This includes all of QtTest *and* all of QtCore. Please use <QTest> instead.

> ksambasharetest.h:21
> +
> +#ifndef KSAMBASAHRETEST_H
> +#define KSAMBASAHRETEST_H

typo: s/AH/HA/

REPOSITORY
  R241 KIO

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

To: broulik, dfaure, fvogt
Cc: ngraham, bruns, kde-frameworks-devel, michaelh

Reply via email to