D18672: Properly clean up scopes when deregistering a subscriber.

David Faure noreply at phabricator.kde.org
Sat Feb 2 15:18:57 GMT 2019


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

REVISION SUMMARY
  It was done "by hand", for only a few setters; use apply() to do this
  consistently for all of them.
  
  New unittest notificationmanagertest
  This requires adding a special mode for unittests in AkThread, to be
  able to call methods in NotificationManager from the main thread.

TEST PLAN
  bin/notificationmanagertest

BRANCH
  2019_01_aggregatedfetchscope_test

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

AFFECTED FILES
  autotests/server/CMakeLists.txt
  autotests/server/notificationmanagertest.cpp
  src/server/akthread.cpp
  src/server/akthread.h
  src/server/notificationmanager.cpp
  src/server/notificationmanager.h
  src/server/notificationsubscriber.cpp

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


More information about the kde-pim mailing list