commit b9600b0fbd71908f9fb3dd1bf78ebba6c4a16115
author Shashank Parekh <shashank201101193@gmail.com> Fri Jun 21 08:32:22 2019 +0530
committer Benjamin Peterson <benjamin@python.org> Thu Jun 20 20:02:22 2019 -0700
tree 084faa8a4e53807c6993a07a83a146db4ed7071e
parent 1e61504b8b941494f3ac03e0449ddbbba8960175

Remove redundant if check from optional argument function in argparse. (GH-8766)

Lib/argparse.py

diff --git a/Lib/argparse.py b/Lib/argparse.py
index 9a67b41..4f3aea9 100644
--- a/Lib/argparse.py
+++ b/Lib/argparse.py
@@ -1479,10 +1479,8 @@
 
             # 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)