Remove redundant if check from optional argument function in argparse. (GH-8766)
authorShashank Parekh <shashank201101193@gmail.com>
Fri, 21 Jun 2019 03:02:22 +0000 (08:32 +0530)
committerBenjamin Peterson <benjamin@python.org>
Fri, 21 Jun 2019 03:02:22 +0000 (20:02 -0700)
Lib/argparse.py

index 9a67b41ae00ead42ba9fc950e9d89e96d9f0a55e..4f3aea928bf6f449110e4ebac8e4aed0656aaabb 100644 (file)
@@ -1479,10 +1479,8 @@ class _ActionsContainer(object):
 
             # 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)