[Kde-hardware-devel] powermanager with HAL backend (Questions)

[Martin (KaFai), Lau]

Hi!

I have some questions.

1. Should backend/hal/acadapter.cpp be changed as the following? My current `lshal` (0.5.7) shows "ac_adaptEr.present" instead of "ac_adaptOr.present"...um... the hal spec said the otherwise though.

Index: backends/hal/acadapter.cpp
===================================================================
--- backends/hal/acadapter.cpp  (revision 580530)
+++ backends/hal/acadapter.cpp  (working copy)
@@ -33,7 +33,7 @@

 bool AcAdapter::isPlugged() const
 {
-    return m_device->property( "ac_adaptor.present" ).toBool();
+    return m_device->property( "ac_adapter.present" ).toBool();
 }


2. In the attached code (halpowermanager.cpp), how can I delete the HalDevice object? In particular, what should I do in the 'HalPowerManager::~HalPowerManager()'? I guess I probably should not write things in this way? or some other parents should own it?

Many Thanks!!  I am still learning :)

--Martin
 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: halpowermanager.cpp
Type: text/x-c++src
Size: 2495 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20060903/09ab7c8b/attachment.bin