diff options
| author | Kevin O'Connor <kevin@koconnor.net> | 2020-09-16 12:15:19 -0400 |
|---|---|---|
| committer | Kevin O'Connor <kevin@koconnor.net> | 2020-09-16 23:53:45 -0400 |
| commit | 760a0f8df538ecce87812004ecea73e0db95d99b (patch) | |
| tree | 4fcebbe13d59f8d7a6a889a68a80a1758cc3cd8b /klippy/klippy.py | |
| parent | a3fa11ffd454d6efec414646213eccd3a8525165 (diff) | |
| download | kutter-760a0f8df538ecce87812004ecea73e0db95d99b.tar.gz kutter-760a0f8df538ecce87812004ecea73e0db95d99b.tar.xz kutter-760a0f8df538ecce87812004ecea73e0db95d99b.zip | |
reactor: Add explicit finalize() method to clean up reactor state
The existence of a __del__() method prevents deallocation on python2
if there are circular references. Replace the __del__() method with a
new finalize() call and arrange for it to be called when the main
reactor is released.
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'klippy/klippy.py')
| -rw-r--r-- | klippy/klippy.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/klippy/klippy.py b/klippy/klippy.py index 3d090886..187b45e5 100644 --- a/klippy/klippy.py +++ b/klippy/klippy.py @@ -48,10 +48,10 @@ Printer is shutdown class Printer: config_error = configfile.error command_error = homing.CommandError - def __init__(self, bglogger, start_args): + def __init__(self, main_reactor, bglogger, start_args): self.bglogger = bglogger self.start_args = start_args - self.reactor = reactor.Reactor() + self.reactor = main_reactor self.reactor.register_callback(self._connect) self.state_message = message_startup self.in_shutdown_state = False @@ -312,11 +312,13 @@ def main(): if bglogger is not None: bglogger.clear_rollover_info() bglogger.set_rollover_info('versions', versions) - printer = Printer(bglogger, start_args) + main_reactor = reactor.Reactor() + printer = Printer(main_reactor, bglogger, start_args) res = printer.run() if res in ['exit', 'error_exit']: break time.sleep(1.) + main_reactor.finalize() logging.info("Restarting printer") start_args['start_reason'] = res |