change state meanings
authorHelmut Grohne <helmut@subdivi.de>
Sun, 16 Jun 2013 19:55:56 +0000 (21:55 +0200)
committerHelmut Grohne <helmut@subdivi.de>
Sun, 16 Jun 2013 19:55:56 +0000 (21:55 +0200)
The ST_ACTIVE bit is now to define the desired state. The previous
definition of using the state we are coming from does not make much
sense. Consider the case when being deactivated while transitioning to
active. Then we are coming from inactive and transitioning to inactive.

dbus_service.py
onoff/common.py

index 48a42b3..a00b0df 100755 (executable)
@@ -31,7 +31,7 @@ class OnoffControl(dbus.service.Object):
         GObject.timeout_add(duration * 1000, self.unuse)
         if self.usecount > 1:
             return self.curstate
-        self.state(ST_TRANSITION)
+        self.state(ST_ACTIVE|ST_TRANSITION)
         def finish():
             self.state(ST_ACTIVE)
         GObject.timeout_add(200, finish)
index 26e32bf..e8fde4a 100644 (file)
@@ -1,9 +1,9 @@
 """
 Defined states:
  * 0: The device is inactive.
- * ST_TRANSITION: The device is transitioning from inactive to active.
+ * ST_ACTIVE|ST_TRANSITION: The device is transitioning from inactive to active.
  * ST_ACTIVE: The device is active.
- * ST_ACTIVE|ST_TRANSITION: The device is transitioning from active to inactive.
+ * ST_TRANSITION: The device is transitioning from active to inactive.
 """
 
 ST_ACTIVE = 1