From: Ralf Jung <post@ralfj.de>
Date: Thu, 10 Oct 2013 13:47:10 +0000 (+0200)
Subject: Inherit superclass constrcutor where applicable; avoid repeating superclass name
X-Git-Url: https://git.ralfj.de/saartuer.git/commitdiff_plain/c0afad48bf55284285c1491965c971c9573fcd1a?ds=sidebyside

Inherit superclass constrcutor where applicable; avoid repeating superclass name
---

diff --git a/statemachine.py b/statemachine.py
index 1fa66b9..81b6b87 100644
--- a/statemachine.py
+++ b/statemachine.py
@@ -73,8 +73,6 @@ class StateMachine():
 				raise Exception("Unknown command number: %d" % ev)
 	
 	class StateStart(State):
-		def __init__(self, sm):
-			State.__init__(self,sm)
 		def handle_pins_event(self):
 			thepins = self.pins()
 			for pin in thepins:
@@ -86,8 +84,6 @@ class StateMachine():
 				return StateAuf
 
 	class StateZu(State):
-		def __init__(self,sm):
-			State.__init__(self,sm)
 		def handle_pins_event(self):
 			pins = self.pins()
 			if not pins.door_locked:
@@ -97,7 +93,7 @@ class StateMachine():
 	
 	class StateOpening(State):
 		def __init__(self,callback,sm):
-			State.__init__(self,sm)
+			super().__init__(self,sm)
 			self.callbacks=[callback]
 			# FIXME: can we send "202 processing: Trying to open the door" here? Are the callbacks multi-use?
 			self.tries = 0
@@ -128,16 +124,13 @@ class StateMachine():
 					return StateZu(self.state_machine)
 	
 	class AbstractStateWhereOpeningIsRedundant(State):
-		def __init__ (self,sm):
-			State.__init__(sm):
 		def handle_open_event(self, callback):
+			# FIXME contradicting original plan where open would be ignored in StateAboutToOpen?
 			callback("299 redundant: Space seems to be already open. Still processing your request tough.")
 			logger.warning("Received OPEN command in StateAboutToOpen. This should not be necessary.")
 			self.actor().act(Actor.CMD_OPEN)
 	
 	class StateAboutToOpen(AbstractStateWhereOpeningIsRedundant):
-		def __init__(self, sm):
-			AbstractStateWhereOpeningIsRedundant.__init__(sm)
 		def handle_pins_event(self):
 			pins = self.pins()
 			if pins.door_locked:
@@ -155,7 +148,7 @@ class StateMachine():
 	
 	class StateAuf(AbstractStateWhereOpeningIsRedundant):
 		def __init__(self,sm):
-			AbstractStateWhereOpeningIsRedundant.__init__(sm)
+			super().__init__(sm)
 			self.last_buzzed = None
 		def handle_pins_event(self):
 			pins = self.pins()