Skip to content

Commit

Permalink
pythongh-104273: Remove redundant len() calls in argparse function (p…
Browse files Browse the repository at this point in the history
  • Loading branch information
buraksaler authored and jbower-fb committed May 8, 2023
1 parent 67b4b55 commit 52539f0
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions Lib/argparse.py
Original file line number Diff line number Diff line change
Expand Up @@ -345,21 +345,22 @@ def _format_usage(self, usage, actions, groups, prefix):
def get_lines(parts, indent, prefix=None):
lines = []
line = []
indent_length = len(indent)
if prefix is not None:
line_len = len(prefix) - 1
else:
line_len = len(indent) - 1
line_len = indent_length - 1
for part in parts:
if line_len + 1 + len(part) > text_width and line:
lines.append(indent + ' '.join(line))
line = []
line_len = len(indent) - 1
line_len = indent_length - 1
line.append(part)
line_len += len(part) + 1
if line:
lines.append(indent + ' '.join(line))
if prefix is not None:
lines[0] = lines[0][len(indent):]
lines[0] = lines[0][indent_length:]
return lines

# if prog is short, follow it with optionals or positionals
Expand Down

0 comments on commit 52539f0

Please sign in to comment.