D15623: Finish the notification before removing it from the hash

[Jaime Torres Amate]

jtamate abandoned this revision.
jtamate added a comment.


  I didn't realize the notifications are not owned by m_notifications :-(. 
  Just looking at the backtrace of the crash, it was the only suspicious thing.

REPOSITORY
  R289 KNotifications

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

To: jtamate, #frameworks, sitter
Cc: anthonyfieroni, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180921/f44fc3bd/attachment-0001.html>