D21132: Properly cleanup in notificationmanagertest

David Faure noreply at phabricator.kde.org
Fri May 10 16:42:32 BST 2019


dfaure created this revision.
dfaure added a reviewer: dvratil.
dfaure requested review of this revision.

REVISION SUMMARY
  The call to "quit" is symmetrical with
  
    QMetaObject::invokeMethod(&manager, "init", Qt::DirectConnection);
  
  on line 56 of the test (due to NoThread mode).
  
  But for the call to quit() to work, the base class has to not assert.

TEST PLAN
  bin/notificationmanagertest in an ASAN build

BRANCH
  Applications/19.04

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

AFFECTED FILES
  autotests/server/notificationmanagertest.cpp
  src/server/akthread.cpp

To: dfaure, dvratil
Cc: kde-pim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190510/bc7a1749/attachment.html>


More information about the kde-pim mailing list