Issue 33852: doc Remove parentheses from sequence subscription description
Created on 2018-06-13 13:12 by adelfino, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 7672 | merged | adelfino, 2018-06-13 13:12 | |
| PR 7713 | merged | miss-islington, 2018-06-15 18:25 | |
| PR 7714 | merged | miss-islington, 2018-06-15 18:27 | |
| PR 7716 | merged | adelfino, 2018-06-15 18:49 | |
| Messages (8) | |||
|---|---|---|---|
| msg319460 - (view) | Author: Andrés Delfino (adelfino) *  |
Date: 2018-06-13 13:12 | |
I think the idea of having "list" inside parentheses is to document that in case of sequences, only "one-expression" expression lists are legal. That being said, IMHO, explaining that in actual prose would be better, but removing the parentheses (taking into account that there's a description of what the expected evaluation of sequences subscription expression lists is) is enough, plus, it's the production list term's name. If a prose is considered better, it could read: "If the primary is a sequence, the expression list must be formed by only one expression, and it must evaluate to an integer or a slice (as discussed in the following section)." Couldn't find the original author. It was committed before the move to mercurial, and I don't know if there's a subversion repo still available to see historical changes. |
|||
| msg319462 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) *  |
Date: 2018-06-13 13:19 | |
This wording was introduced in 3a0ad6089bccf0b167fe8ebd2457fedec8f851a3. |
|||
| msg319465 - (view) | Author: Andrés Delfino (adelfino) * |
Date: 2018-06-13 13:45 | |
Thanks, Serhiy! Didn't know I should trace the LaTeX version. Guido, I'm adding you because it seems you authored or at least agreed on the current wording. Do you believe this proposal for improvement makes sense? |
|||
| msg319637 - (view) | Author: Terry J. Reedy (terry.reedy) * |
Date: 2018-06-15 18:24 | |
New changeset 4fddd4e4069aad9efad999d8d9ce3cd9fb523a5c by Terry Jan Reedy (Andrés Delfino) in branch 'master': bpo-33852: Remove misplaced parentheses around 'list'. (GH-7672) https://github.com/python/cpython/commit/4fddd4e4069aad9efad999d8d9ce3cd9fb523a5c |
|||
| msg319643 - (view) | Author: miss-islington (miss-islington) | Date: 2018-06-15 18:45 | |
New changeset 0113391e3f3db0fea6a9545310dc06aa5ea41461 by Miss Islington (bot) in branch '3.7': bpo-33852: Remove misplaced parentheses around 'list'. (GH-7672) https://github.com/python/cpython/commit/0113391e3f3db0fea6a9545310dc06aa5ea41461 |
|||
| msg319646 - (view) | Author: miss-islington (miss-islington) | Date: 2018-06-15 18:48 | |
New changeset eb4989970ce639845ca17e9a040f55fe704b862e by Miss Islington (bot) in branch '3.6': bpo-33852: Remove misplaced parentheses around 'list'. (GH-7672) https://github.com/python/cpython/commit/eb4989970ce639845ca17e9a040f55fe704b862e |
|||
| msg319652 - (view) | Author: Terry J. Reedy (terry.reedy) * |
Date: 2018-06-15 19:23 | |
New changeset 184e8ed1eeda3ba869755ff37374dd86f332f30d by Terry Jan Reedy (Andrés Delfino) in branch '2.7': [2.7] bpo-33852: Remove misplaced parentheses around 'list'. (GH-7672) (#7716) https://github.com/python/cpython/commit/184e8ed1eeda3ba869755ff37374dd86f332f30d |
|||
| msg319653 - (view) | Author: Terry J. Reedy (terry.reedy) * |
Date: 2018-06-15 19:25 | |
Guido approved the change on the PR. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:01 | admin | set | github: 78033 |
| 2018-06-15 19:25:01 | terry.reedy | set | status: open -> closed messages: + msg319653 assignee: docs@python -> terry.reedy |
| 2018-06-15 19:23:40 | terry.reedy | set | messages: + msg319652 |
| 2018-06-15 18:49:43 | adelfino | set | pull_requests: + pull_request7331 |
| 2018-06-15 18:48:24 | miss-islington | set | messages: + msg319646 |
| 2018-06-15 18:45:42 | miss-islington | set | nosy:
+ miss-islington messages: + msg319643 |
| 2018-06-15 18:27:38 | miss-islington | set | pull_requests: + pull_request7329 |
| 2018-06-15 18:25:44 | miss-islington | set | keywords:
+ patch stage: patch review pull_requests: + pull_request7328 |
| 2018-06-15 18:24:30 | terry.reedy | set | nosy:
+ terry.reedy messages: + msg319637 |
| 2018-06-13 13:45:01 | adelfino | set | nosy:
+ gvanrossum messages: + msg319465 |
| 2018-06-13 13:19:51 | serhiy.storchaka | set | nosy:
+ serhiy.storchaka messages: + msg319462 |
| 2018-06-13 13:14:33 | adelfino | set | title: doc Remove parentheses from -> doc Remove parentheses from sequence subscription description |
| 2018-06-13 13:12:22 | adelfino | create | |