Added a TODO
This commit is contained in:
parent
17f8ebb686
commit
958be9dc25
1 changed files with 4 additions and 2 deletions
|
@ -35,6 +35,9 @@ BatteryPlugin::BatteryPlugin(QObject *parent, const QVariantList &args)
|
||||||
, batteryDbusInterface(new BatteryDbusInterface(parent))
|
, batteryDbusInterface(new BatteryDbusInterface(parent))
|
||||||
{
|
{
|
||||||
|
|
||||||
|
//TODO: Add battery reporting, could be based on:
|
||||||
|
// http://kde-apps.org/content/show.php/battery+plasmoid+with+remaining+time?content=120309
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
void BatteryPlugin::connected()
|
void BatteryPlugin::connected()
|
||||||
|
@ -50,8 +53,7 @@ BatteryPlugin::~BatteryPlugin()
|
||||||
// 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 does not remove the interface (that will
|
||||||
// return outdated values) and that enabling it again instantiates a second
|
// return outdated values) and that enabling it again instantiates a second
|
||||||
// adaptor. This is a partial memory leak (the memory will be freed when the
|
// adaptor. This is also a memory leak until the entire device is destroyed.
|
||||||
// device is destroyed anyway)
|
|
||||||
|
|
||||||
//batteryDbusInterface->deleteLater();
|
//batteryDbusInterface->deleteLater();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue