diff options
author | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-05-19 15:51:51 +0900 |
---|---|---|
committer | Masahiro Yamada <yamada.masahiro@socionext.com> | 2016-06-13 07:46:28 +0900 |
commit | ff8725bbe06b3b890beb2044c02ef332c8d028b4 (patch) | |
tree | 0edabd4f98482da290239d8b8987c2d912c8ba37 /tools/moveconfig.py | |
parent | 6ff36d21746d61d7a2af08ea0bd973059d91d306 (diff) | |
download | u-boot-ff8725bbe06b3b890beb2044c02ef332c8d028b4.zip u-boot-ff8725bbe06b3b890beb2044c02ef332c8d028b4.tar.gz u-boot-ff8725bbe06b3b890beb2044c02ef332c8d028b4.tar.bz2 |
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>
Diffstat (limited to 'tools/moveconfig.py')
-rwxr-xr-x | tools/moveconfig.py | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/tools/moveconfig.py b/tools/moveconfig.py index 9029287..1332bd2 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) |