python/jc
1
0
Fork 0
mirror of https://github.com/kellyjonbrazil/jc.git synced 2025-06-19 00:17:51 +02:00
Code Issues Releases Activity

simplify by removing unnecessary getattr calls

Browse Source
This commit is contained in:
Kelly Brazil
2020-02-05 14:10:22 -08:00
parent bb5ba7ddb1
commit 6bad164b5e
1 changed files with 7 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
jc/cli.py
View File

@ -92,7 +92,7 @@ def parsers_text(indent=0, pad=0):
parser_mod = parser_module(parser) parser_mod = parser_module(parser)
if hasattr(parser_mod, 'info'): if hasattr(parser_mod, 'info'):
parser_desc = getattr(parser_mod.info, 'description') parser_desc = parser_mod.info.description
padding = pad - len(parser_arg) padding = pad - len(parser_arg)
padding_char = ' ' padding_char = ' '
indent_text = padding_char * indent indent_text = padding_char * indent
@ -110,9 +110,9 @@ def about_jc():
if hasattr(parser_mod, 'info'): if hasattr(parser_mod, 'info'):
info_dict = {} info_dict = {}
info_dict['name'] = getattr(parser_mod, '__name__').split('.')[-1] info_dict['name'] = parser_mod.__name__.split('.')[-1]
info_dict['argument'] = parser_argument(parser) info_dict['argument'] = parser_argument(parser)
parser_entry = vars(getattr(parser_mod, 'info')) parser_entry = vars(parser_mod.info)
for k, v in parser_entry.items(): for k, v in parser_entry.items():
if not k.startswith('__'): if not k.startswith('__'):
@ -200,9 +200,8 @@ def main():
if parser_name in parsers: if parser_name in parsers:
# load parser module just in time so we don't need to load all modules # load parser module just in time so we don't need to load all modules
parser_mod = parser_module(arg) parser = parser_module(arg)
parser = getattr(parser_mod, 'parse') result = parser.parse(data, raw=raw, quiet=quiet)
result = parser(data, raw=raw, quiet=quiet)
found = True found = True
break break
else: else:
@ -211,10 +210,9 @@ def main():
if parser_name in parsers: if parser_name in parsers:
# load parser module just in time so we don't need to load all modules # load parser module just in time so we don't need to load all modules
parser_mod = parser_module(arg) parser = parser_module(arg)
try: try:
parser = getattr(parser_mod, 'parse') result = parser.parse(data, raw=raw, quiet=quiet)
result = parser(data, raw=raw, quiet=quiet)
found = True found = True
break break
except: except: