[5/9] Remove bare "except" from disassembler.py
Checks
Context |
Check |
Description |
linaro-tcwg-bot/tcwg_gdb_build--master-aarch64 |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_build--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_check--master-aarch64 |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gdb_check--master-arm |
success
|
Testing passed
|
Commit Message
flake8 complains about a bare "except" in disassembler.py. In this
case, the code purports to guard against some kind of user error
involving data structure corruption. I think it's better here to just
let the error occur -- py-disasm.c will show a stack trace in this
case.
---
gdb/python/lib/gdb/disassembler.py | 21 +++++++--------------
1 file changed, 7 insertions(+), 14 deletions(-)
@@ -93,21 +93,14 @@ def _print_insn(info):
disassembled."""
def lookup_disassembler(arch):
- try:
- name = arch.name()
- if name is None:
- return None
- if name in _disassemblers_dict:
- return _disassemblers_dict[name]
- if None in _disassemblers_dict:
- return _disassemblers_dict[None]
- return None
- except:
- # It's pretty unlikely this exception case will ever
- # trigger, one situation would be if the user somehow
- # corrupted the _disassemblers_dict variable such that it
- # was no longer a dictionary.
+ name = arch.name()
+ if name is None:
return None
+ if name in _disassemblers_dict:
+ return _disassemblers_dict[name]
+ if None in _disassemblers_dict:
+ return _disassemblers_dict[None]
+ return None
disassembler = lookup_disassembler(info.architecture)
if disassembler is None: