Replace 'sys.exit()' and 'exit()' instructions with 'SystemExit()' equivalents
- `SystemExit` doesn't require importing the `sys` module - `exit()` depends on the `site` module (for this reason its usage is discouraged in production code) Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
This commit is contained in:
@@ -3,7 +3,6 @@
|
||||
|
||||
import argparse
|
||||
import os
|
||||
import sys
|
||||
|
||||
from whipper.common import config, drive
|
||||
|
||||
@@ -109,11 +108,11 @@ class BaseCommand:
|
||||
if hasattr(self, 'subcommands'):
|
||||
if not self.options.remainder:
|
||||
self.parser.print_help()
|
||||
sys.exit(0)
|
||||
raise SystemExit()
|
||||
if not self.options.remainder[0] in self.subcommands:
|
||||
logger.critical("incorrect subcommand: %s",
|
||||
self.options.remainder[0])
|
||||
sys.exit(1)
|
||||
raise SystemExit(1)
|
||||
self.cmd = self.subcommands[self.options.remainder[0]](
|
||||
self.options.remainder[1:],
|
||||
prog_name + " " + self.options.remainder[0],
|
||||
|
||||
@@ -18,8 +18,6 @@
|
||||
# You should have received a copy of the GNU General Public License
|
||||
# along with whipper. If not, see <http://www.gnu.org/licenses/>.
|
||||
|
||||
import sys
|
||||
|
||||
from whipper.command.basecommand import BaseCommand
|
||||
from whipper.common import accurip, config, program
|
||||
from whipper.extern.task import task
|
||||
@@ -63,7 +61,7 @@ Verifies the image from the given .cue files against the AccurateRip database.
|
||||
print('AccurateRip entry not found')
|
||||
accurip.print_report(prog.result)
|
||||
if not verified:
|
||||
sys.exit(1)
|
||||
raise SystemExit(1)
|
||||
|
||||
|
||||
class Image(BaseCommand):
|
||||
|
||||
Reference in New Issue
Block a user