cfeck added inline comments.
INLINE COMMENTS
> kmessagewidgetautotest.cpp:24
> #include <QtTest/QtTest>
> +#include <QSignalSpy>
> #include <kmessagewidget.h>
Isn't QSignalSpy part of QtTest? In other words, isn't it included by above
#include line?
> kmessagewidgetautotest.cpp:125
> + w.animatedShow();
> + if (delay) QTest::qWait(overlappingWaitingTime);
> + w.animatedHide();
if (delay) {
QTest::bla;
}
> kmessagewidgetautotest.cpp:158
> + w.animatedHide();
> + if (delay) QTest::qWait(overlappingWaitingTime);
> + w.animatedShow();
s.a.
> kmessagewidgetautotest.cpp:189
> + w.animatedShow();
> + if (delay) QTest::qWait(overlappingWaitingTime);
> + w.animatedShow();
s.a.
> kmessagewidgetautotest.cpp:218
> + w.animatedHide();
> + if (delay) QTest::qWait(overlappingWaitingTime);
> + w.animatedHide();
s.a.
REPOSITORY
R236 KWidgetsAddons
REVISION DETAIL
https://phabricator.kde.org/D4329
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: kossebau, #frameworks, dhaumann
Cc: cfeck
