From 43276dd0273f321945d7fe83c8073a0e00e4a46f Mon Sep 17 00:00:00 2001 From: Shashank Parekh Date: Wed, 15 Aug 2018 01:05:24 +0530 Subject: [PATCH] Update argparse.py Remove redundant if check from optional argument function in argparser. --- Lib/argparse.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Lib/argparse.py b/Lib/argparse.py index 83f47e35a73c74..c4d9db36d1f4f6 100644 --- a/Lib/argparse.py +++ b/Lib/argparse.py @@ -1469,10 +1469,8 @@ def _get_optional_kwargs(self, *args, **kwargs): # strings starting with two prefix characters are long options option_strings.append(option_string) - if option_string[0] in self.prefix_chars: - if len(option_string) > 1: - if option_string[1] in self.prefix_chars: - long_option_strings.append(option_string) + if len(option_string) > 1 and option_string[1] in self.prefix_chars: + long_option_strings.append(option_string) # infer destination, '--foo-bar' -> 'foo_bar' and '-x' -> 'x' dest = kwargs.pop('dest', None)