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

Only deletes dbus interfaces for the same device that had created it.

Browse source
This commit is contained in:
Lamarque V. Souza 2015-07-18 16:36:58 -03:00
parent 208c11de1a
commit 38aa9704a7
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
plugins/battery/batterydbusinterface.cpp
View file

@ -24,7 +24,7 @@
#include <QDebug>
#include <core/device.h>
BatteryDbusInterface *BatteryDbusInterface::s_currentInterface = 0;
QMap<QString, BatteryDbusInterface *> BatteryDbusInterface::s_dbusInterfaces;
BatteryDbusInterface::BatteryDbusInterface(const Device *device)
: QDBusAbstractAdaptor(const_cast<Device*>(device))
@ -35,12 +35,13 @@ BatteryDbusInterface::BatteryDbusInterface(const Device *device)
// This makes the old BatteryDdbusInterface be deleted only after the new one is
// fully operational. That seems to prevent the crash mentioned in BatteryPlugin's
// destructor.
if (s_currentInterface) {
qCDebug(KDECONNECT_PLUGIN_BATTERY) << "Deleting stale BattteryDbusInterface object.";
s_currentInterface->deleteLater();
QMap<QString, BatteryDbusInterface *>::iterator oldInterfaceIter = s_dbusInterfaces.find(device->id());
if (oldInterfaceIter != s_dbusInterfaces.end()) {
qCDebug(KDECONNECT_PLUGIN_BATTERY) << "Deleting stale BattteryDbusInterface for" << device->name();
oldInterfaceIter.value()->deleteLater();
}
s_currentInterface = this;
s_dbusInterfaces[device->id()] = this;
}
BatteryDbusInterface::~BatteryDbusInterface()

3
plugins/battery/batterydbusinterface.h
View file

@ -48,7 +48,8 @@ private:
int mCharge;
bool mIsCharging;
static BatteryDbusInterface *s_currentInterface;
// Map to save current BatteryDbusInterface for each device.
static QMap<QString, BatteryDbusInterface *> s_dbusInterfaces;
};
#endif