openstack/mistral
Code Issues Proposed changes

Using sys.exit(main()) instead of main()

Browse Source 
As we known, Exceptions are raised by the sys.exit() function.
When they are not handled, no stack traceback is printed in the
Python interpreter. In this patch we have known main() got return
values(eg:0, 1), but it can't specifies exit status when terminate
the main thread by unusing sys.exit(). So when using sys.exit(main())
instead of main() may be more readable and reasonable.

Change-Id: Ie69b3a22dd1ddcd5586fc72f452cd162787f2f6f
This commit is contained in:
gecong1973 2017-01-04 09:18:24 +08:00
parent 685e33feeb
commit d597dab65c
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tools/install_venv.py
View File

@ -70,4 +70,4 @@ def main(argv):
print_help(venv, root) print_help(venv, root)
if __name__ == '__main__': if __name__ == '__main__':
main(sys.argv) sys.exit(main(sys.argv))