tristan/kdeconnect-kde
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Updated comment

Browse source
This commit is contained in:
Albert Vaca 2015-06-14 19:02:47 -07:00
parent d4074b42fa
commit c041026317
1 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
plugins/notifications/notificationsplugin.cpp
View file

@ -46,9 +46,8 @@ NotificationsPlugin::~NotificationsPlugin()
{ {
//FIXME: Qt dbus does not allow to remove an adaptor! (it causes a crash in //FIXME: Qt dbus does not allow to remove an adaptor! (it causes a crash in
// the next dbus access to its parent). The implication of not deleting this // the next dbus access to its parent). The implication of not deleting this
// is that disabling the plugin does not remove the interface (that will // is that disabling the plugin leaks the interface. As a mitigation we are
// return outdated values) and that enabling it again instantiates a second // cleaning up the notifications inside the adaptor here.
// adaptor.
//notificationsDbusInterface->deleteLater(); //notificationsDbusInterface->deleteLater();
notificationsDbusInterface->clearNotifications(); notificationsDbusInterface->clearNotifications();
} }