D5321: [Notifications Applet] Don't crash when containment() disappears

Anthony Fieroni noreply at phabricator.kde.org
Tue Apr 11 08:22:20 UTC 2017


anthonyfieroni added inline comments.

INLINE COMMENTS

> notificationsapplet.cpp:54
>      Q_ASSERT(containment()->corona());
>      connect(containment()->corona(), &Plasma::Corona::availableScreenRectChanged, this, &NotificationsApplet::onScreenChanges);
>  

I think, corona (witch is parent of containment) is still alive and that's why slot is invoked.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, davidedmundson
Cc: anthonyfieroni, davidedmundson, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170411/8f2e2a97/attachment.html>


More information about the Plasma-devel mailing list