Review Request 129177: [KNotifications] Don't use KPassivePopup without QApplication

[Kai Uwe Broulik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129177/
-----------------------------------------------------------

(Updated Oct. 14, 2016, 7:31 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit 11341f1a024edd907b90f34c267e107e930aebf8 by Kai Uwe Broulik to branch master.


Repository: knotifications


Description
-------

When using KNotification from a QGuiApplication and there is no notification service present, the application will assert.

BUG: 370667


Diffs
-----

  src/notifybypopup.cpp 7d69a36 

Diff: https://git.reviewboard.kde.org/r/129177/diff/


Testing
-------

PowerDevil no longer crashes when a notification is emitted while no notificiation service is running.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161014/b71b65ed/attachment.html>