From c0afad48bf55284285c1491965c971c9573fcd1a Mon Sep 17 00:00:00 2001 From: Ralf Jung Date: Thu, 10 Oct 2013 15:47:10 +0200 Subject: [PATCH] Inherit superclass constrcutor where applicable; avoid repeating superclass name --- statemachine.py | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) 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() -- 2.30.2