summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMasahiro Yamada <yamada.masahiro@socionext.com>2016-05-19 15:51:51 +0900
committerMasahiro Yamada <yamada.masahiro@socionext.com>2016-06-13 07:46:28 +0900
commitff8725bbe06b3b890beb2044c02ef332c8d028b4 (patch)
tree0edabd4f98482da290239d8b8987c2d912c8ba37
parent6ff36d21746d61d7a2af08ea0bd973059d91d306 (diff)
downloadtalos-obmc-uboot-ff8725bbe06b3b890beb2044c02ef332c8d028b4.tar.gz
talos-obmc-uboot-ff8725bbe06b3b890beb2044c02ef332c8d028b4.zip
tools: moveconfig: remove redundant else: after sys.exit()
Nesting by "else:" is not generally useful after such statements as return, break, sys.exit(), etc. Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com> Reviewed-by: Joe Hershberger <joe.hershberger@ni.com>
-rwxr-xr-xtools/moveconfig.py12
1 files changed, 5 insertions, 7 deletions
diff --git a/tools/moveconfig.py b/tools/moveconfig.py
index 9029287b10..1332bd2830 100755
--- a/tools/moveconfig.py
+++ b/tools/moveconfig.py
@@ -623,13 +623,11 @@ class Slot:
COLOR_LIGHT_CYAN, errout)
if self.options.exit_on_error:
sys.exit("Exit on error.")
- else:
- # If --exit-on-error flag is not set,
- # skip this board and continue.
- # Record the failed board.
- self.failed_boards.append(self.defconfig)
- self.state = STATE_IDLE
- return True
+ # If --exit-on-error flag is not set, skip this board and continue.
+ # Record the failed board.
+ self.failed_boards.append(self.defconfig)
+ self.state = STATE_IDLE
+ return True
if self.state == STATE_AUTOCONF:
self.parser.update_dotconfig(self.defconfig)
OpenPOWER on IntegriCloud