classification
Title: doc Remove parentheses from sequence subscription description
Type: enhancement Stage: resolved
Components: Documentation Versions: Python 3.8, Python 3.7, Python 3.6, Python 2.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: terry.reedy Nosy List: adelfino, docs@python, gvanrossum, miss-islington, serhiy.storchaka, terry.reedy
Priority: normal Keywords: patch

Created on 2018-06-13 13:12 by adelfino, last changed 2018-06-15 19:25 by terry.reedy. 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) * (Python committer) 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) * (Python committer) 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) * (Python committer) 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) * (Python committer) Date: 2018-06-15 19:25
Guido approved the change on the PR.
History
Date User Action Args
2018-06-15 19:25:01terry.reedysetstatus: open -> closed
messages: + msg319653

assignee: docs@python -> terry.reedy
resolution: fixed
stage: patch review -> resolved
2018-06-15 19:23:40terry.reedysetmessages: + msg319652
2018-06-15 18:49:43adelfinosetpull_requests: + pull_request7331
2018-06-15 18:48:24miss-islingtonsetmessages: + msg319646
2018-06-15 18:45:42miss-islingtonsetnosy: + miss-islington
messages: + msg319643
2018-06-15 18:27:38miss-islingtonsetpull_requests: + pull_request7329
2018-06-15 18:25:44miss-islingtonsetkeywords: + patch
stage: patch review
pull_requests: + pull_request7328
2018-06-15 18:24:30terry.reedysetnosy: + terry.reedy
messages: + msg319637
2018-06-13 13:45:01adelfinosetnosy: + gvanrossum
messages: + msg319465
2018-06-13 13:19:51serhiy.storchakasetnosy: + serhiy.storchaka
messages: + msg319462
2018-06-13 13:14:33adelfinosettitle: doc Remove parentheses from -> doc Remove parentheses from sequence subscription description
2018-06-13 13:12:22adelfinocreate