dfaure added inline comments. INLINE COMMENTS
> kfileplacesviewtest.cpp:30 > + > +#include <QtTest> > +#include <QSignalSpy> Please don't include the full module (which also contains all QtCore) #include <QTest> is OK, #include <QtTest> is not. Confusing, I know. REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D8332 To: renatoo, #frameworks, #dolphin, #kde_applications, dvratil, #vdg, ngraham, ervin, mlaurent Cc: dfaure, ervin, usta, mlaurent, dvratil, ngraham, #frameworks