diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2019-09-09 15:20:36 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-09-09 15:20:36 (GMT) |
commit | 0d4396c04cba5ac2b66fdaa23c01db84b1b54227 (patch) | |
tree | 1b141ba7fe161f85ff5dfd392be74d5a80975531 | |
parent | 3bd4bed78a0b068e28bcf2242d33aed227c2532c (diff) | |
download | cpython-0d4396c04cba5ac2b66fdaa23c01db84b1b54227.zip cpython-0d4396c04cba5ac2b66fdaa23c01db84b1b54227.tar.gz cpython-0d4396c04cba5ac2b66fdaa23c01db84b1b54227.tar.bz2 |
bpo-38059: Using sys.exit() over exit() in inspect.py (GH-15666)
Constants added by the site module like exit() "should not be used in programs"
(cherry picked from commit e3c59a75279b0df4e7553d6f0031e202de434e43)
Co-authored-by: Alan Yee <alanyee@users.noreply.github.com>
-rw-r--r-- | Lib/inspect.py | 4 | ||||
-rw-r--r-- | Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst | 1 |
2 files changed, 3 insertions, 2 deletions
diff --git a/Lib/inspect.py b/Lib/inspect.py index a616f2d..0a57749 100644 --- a/Lib/inspect.py +++ b/Lib/inspect.py @@ -3118,7 +3118,7 @@ def _main(): type(exc).__name__, exc) print(msg, file=sys.stderr) - exit(2) + sys.exit(2) if has_attrs: parts = attrs.split(".") @@ -3128,7 +3128,7 @@ def _main(): if module.__name__ in sys.builtin_module_names: print("Can't get info for builtin modules.", file=sys.stderr) - exit(1) + sys.exit(1) if args.details: print('Target: {}'.format(target)) diff --git a/Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst b/Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst new file mode 100644 index 0000000..001952a --- /dev/null +++ b/Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst @@ -0,0 +1 @@ +inspect.py now uses sys.exit() instead of exit() |