D9821: Use QSignalSpy::wait instead of QTest::wait where possible

David Faure noreply at phabricator.kde.org
Mon Jan 15 08:45:23 UTC 2018


dfaure added inline comments.

INLINE COMMENTS

> kdirwatch_unittest.cpp:256
>      // Give time for KDirWatch to notify us
>      while (spyDirty.count() < expected) {
>          if (++numTries > s_maxTries) {

but then the whole while loop could be removed too, it served the same purpose as what QSignalSpy::wait does.

REPOSITORY
  R244 KCoreAddons

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

To: mwolff, dfaure
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180115/548f738a/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list