pre-commit/flake8
1
0
Fork 0
mirror of https://github.com/PyCQA/flake8.git synced 2026-04-01 11:26:54 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Re-factor normalization in parse_args

Browse source
This commit is contained in:
Ian Cordasco 2016-01-09 21:17:22 -06:00
parent cc3a9210fd
commit 684f19f5e3
1 changed files with 15 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

31
flake8/options/manager.py
View file

@ -111,6 +111,20 @@ class Option(object):
return self.long_option_name[2:].replace('-', '_')
return self.short_option_name[1]
def normalize(self, options):
dest = self.dest
if self.normalize_paths:
old_value = getattr(options, dest)
# Decide whether to parse a list of paths or a single path
normalize = utils.normalize_path
if self.comma_separated_list:
normalize = utils.normalize_paths
setattr(options, dest, normalize(old_value))
elif self.comma_separated_list:
old_value = getattr(options, dest)
setattr(options, dest,
utils.parse_comma_separated_list(old_value))
def to_optparse(self):
"""Convert a Flake8 Option to an optparse Option."""
if not hasattr(self, '_opt'):
@ -153,21 +167,6 @@ class OptionManager(object):
"""Simple proxy to calling the OptionParser's parse_args method."""
options, xargs = self.parser.parse_args(args, values)
for option in self.options:
_normalize_option(options, option)
option.normalize(options)
return options, xargs
def _normalize_option(options, option):
dest = option.dest
if option.normalize_paths:
old_value = getattr(options, dest)
# Decide whether to parse a list of paths or a single path
normalize = utils.normalize_path
if option.comma_separated_list:
normalize = utils.normalize_paths
setattr(options, dest, normalize(old_value))
elif option.comma_separated_list:
old_value = getattr(options, dest)
setattr(options, dest,
utils.parse_comma_separated_list(old_value))