diff options
| author |   Kyle Meyer <kyle@kyleam.com> | 2020-02-18 04:48:57 -0500 |
|---|---|---|
| committer |   GitHub <noreply@github.com> | 2020-02-18 01:48:57 -0800 |
| commit | 8edfc47baec7ff4cb1b9db83dd35c8ffc1d498a4 (patch) | |
| tree | f4127d288dadd66c7546da9f1c1b4f5e0b7550a3 /Misc/ACKS | |
| parent | bpo-39663: IDLE: Add additional tests for pyparse (GH-18536) (diff) | |
| download | cpython-8edfc47baec7ff4cb1b9db83dd35c8ffc1d498a4.tar.gz cpython-8edfc47baec7ff4cb1b9db83dd35c8ffc1d498a4.tar.bz2 cpython-8edfc47baec7ff4cb1b9db83dd35c8ffc1d498a4.zip | |
bpo-39546: argparse: Honor allow_abbrev=False for specified prefix_chars (GH-18337)
When `allow_abbrev` was first added, disabling the abbreviation of
long options broke the grouping of short flags ([bpo-26967](https://bugs.python.org/issue26967)). As a fix,
b1e4d1b603 (contained in v3.8) ignores `allow_abbrev=False` for a
given argument string if the string does _not_ start with "--"
(i.e. it doesn't look like a long option).
This fix, however, doesn't take into account that long options can
start with alternative characters specified via `prefix_chars`,
introducing a regression: `allow_abbrev=False` has no effect on long
options that start with an alternative prefix character.
The most minimal fix would be to replace the "starts with --" check
with a "starts with two prefix_chars characters". But
`_get_option_tuples` already distinguishes between long and short
options, so let's instead piggyback off of that check by moving the
`allow_abbrev` condition into `_get_option_tuples`.
https://bugs.python.org/issue39546
Diffstat (limited to 'Misc/ACKS')
| -rw-r--r-- | Misc/ACKS | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Misc/ACKS b/Misc/ACKS index 933402069b4..e8ce30310bf 100644 --- a/Misc/ACKS +++ b/Misc/ACKS @@ -1112,6 +1112,7 @@ Bruce Merry Alexis Métaireau Luke Mewburn Carl Meyer +Kyle Meyer Mike Meyer Piotr Meyer Steven Miale |