From 3cbe9baf6d3c4aed9d30b90e3015cffca82643ef Mon Sep 17 00:00:00 2001 From: David Wilson Date: Tue, 28 Nov 2017 13:19:10 +0530 Subject: [PATCH] issue #70: remove redundant 'raise SystemExit' Either execvp succeeds, in which case the process image is replaced, or execvp throws an exception, in which case control flow exits the frame anyway. --- mitogen/master.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/mitogen/master.py b/mitogen/master.py index 7afa8159..9327d365 100644 --- a/mitogen/master.py +++ b/mitogen/master.py @@ -107,7 +107,6 @@ def create_child(*args): childfp.close() parentfp.close() os.execvp(args[0], args) - raise SystemExit childfp.close() LOG.debug('create_child() child %d fd %d, parent %d, cmd: %s', @@ -170,7 +169,6 @@ def tty_create_child(*args): os.setsid() os.close(os.open(os.ttyname(1), os.O_RDWR)) os.execvp(args[0], args) - raise SystemExit os.close(slave_fd) LOG.debug('tty_create_child() child %d fd %d, parent %d, cmd: %s',