Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(169610)

#16418: argparse with many choices can generate absurdly long usage message

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 1 month ago by rini17
Modified:
7 years ago
Reviewers:
ezio.melotti, chris.jerdonek
CC:
terry.reedy, bethard, ezio.melotti, r.david.murray, cjerdonek, docs_python.org, paul.j3, rini17_gmail.com
Visibility:
Public.

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Doc/library/argparse.rst View 2 chunks +6 lines, -6 lines 2 comments Download

Messages

Total messages: 3
ezio.melotti
http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst File Doc/library/argparse.rst (right): http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst#newcode1025 Doc/library/argparse.rst:1025: values. These can be handled by passing a container ...
7 years ago #1
cjerdonek
http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst File Doc/library/argparse.rst (right): http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst#newcode1025 Doc/library/argparse.rst:1025: values. These can be handled by passing a container ...
7 years ago #2
ezio.melotti
7 years ago #3
On 2012/11/17 18:49:14, cjerdonek wrote:
> http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst
> File Doc/library/argparse.rst (right):
> 
>
http://bugs.python.org/review/16418/diff/6550/Doc/library/argparse.rst#newcod...
> Doc/library/argparse.rst:1025: values.  These can be handled by passing a
> container object as the ``choices``
> On 2012/11/17 18:38:56, ezio.melotti wrote:
> > Not your fault, but this should be *choices*.
> 
> I noticed and thought about this when creating the patch.  All of the
> surrounding documentation uses the `` formatting.  For example, ``choices``
> occurs that way four times in that section, and similarly for the
documentation
> of the other keyword arguments.  So it was for consistency.

OK, maybe they should be fixed in a separate commit.
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+