Skip to content

Commit

Permalink
package: Seperate imports of sys package from others (#260)
Browse files Browse the repository at this point in the history 
Signed-off-by: Peace Lee <iipeace5@gmail.com>
  • Loading branch information
@iipeace
iipeace committed Sep 25, 2021
1 parent f9fee33 commit b0a37f8
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions guider/guider.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,18 @@



# import sys package #
try:
import sys
except ImportError:
print("[ERROR] fail to import sys package")
sys.exit(0)

This comment has been minimized.

Sign in to view

Copy link
@elfring

elfring Sep 25, 2021

I find this statement still questionable here.

# import essential packages #
try:
import os
import re
import gc
import sys
import time
import errno
import signal
Expand All @@ -31,7 +37,7 @@
#from ctypes import *
except ImportError:
err = sys.exc_info()[1]
print("[ERROR] fail to import essential packages: %s" % err.args[0])
print("[ERROR] fail to import essential package: %s" % err.args[0])
sys.exit(0)

This comment has been minimized.

Sign in to view

Copy link
@elfring

elfring Sep 25, 2021

I suggest to pass the error message to the exit method call.

# convert an unsupported type #
Expand Down

1 comment on commit b0a37f8

@elfring
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Would you like to amend a typo in the commit subject?
  • Will a corresponding commit description become helpful?

Please sign in to comment.