Skip to content

bpo-38557: Improve documentation for list and tuple C API.#16925

Merged
serhiy-storchaka merged 1 commit intopython:masterfrom
serhiy-storchaka:docs-tuple-getslice
Oct 26, 2019
Merged

bpo-38557: Improve documentation for list and tuple C API.#16925
serhiy-storchaka merged 1 commit intopython:masterfrom
serhiy-storchaka:docs-tuple-getslice

Conversation

@serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Oct 25, 2019 •
edited by bedevere-bot
Loading

@serhiy-storchaka serhiy-storchaka added needs backport to 2.7 docs Documentation in the Doc dir labels Oct 25, 2019
terryjreedy
terryjreedy approved these changes Oct 26, 2019
View reviewed changes
Copy link
Member

@terryjreedy terryjreedy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume that the re-wordings and additions are technically correct. They otherwise look good to me.

Doc/c-api/list.rst
and *high*. Return *NULL* and set an exception if unsuccessful. Analogous
to ``list[low:high]``. Negative indices, as when slicing from Python, are not
supported.
to ``list[low:high]``. Indexing from the end of the list is not supported.
Copy link
Member

@terryjreedy terryjreedy Oct 26, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By itself, this is not obviously much of an improvement, but in the context of the earlier sentence for PyList_GetItem, "The position must be non-negative; indexing from the end of the list is not supported.", I think it is.

Copy link
Member Author

@serhiy-storchaka serhiy-storchaka Oct 26, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically, negative indices are supported. But they do not count from the end.

@serhiy-storchaka serhiy-storchaka merged commit d898d20 into python:master Oct 26, 2019
@serhiy-storchaka serhiy-storchaka deleted the docs-tuple-getslice branch October 26, 2019 19:59
@miss-islington
Copy link
Contributor

miss-islington commented Oct 26, 2019

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.7, 3.8.
🐍🍒⛏🤖

@bedevere-bot
Copy link

bedevere-bot commented Oct 26, 2019

GH-16933 is a backport of this pull request to the 3.8 branch.

@bedevere-bot
Copy link

bedevere-bot commented Oct 26, 2019

GH-16934 is a backport of this pull request to the 3.7 branch.

@bedevere-bot
Copy link

bedevere-bot commented Oct 26, 2019

GH-16935 is a backport of this pull request to the 2.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 26, 2019
@serhiy-storchaka @miss-islington
bpo-38557: Improve documentation for list and tuple C API. (pythonGH-…
89c3972 
…16925)

(cherry picked from commit d898d20)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 26, 2019
@serhiy-storchaka @miss-islington
bpo-38557: Improve documentation for list and tuple C API. (pythonGH-…
1416db0 
…16925)

(cherry picked from commit d898d20)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington added a commit that referenced this pull request Oct 26, 2019
@miss-islington @serhiy-storchaka
bpo-38557: Improve documentation for list and tuple C API. (GH-16925)
7356e10 
(cherry picked from commit d898d20)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington added a commit that referenced this pull request Oct 26, 2019
@miss-islington @serhiy-storchaka
bpo-38557: Improve documentation for list and tuple C API. (GH-16925)
334fc92 
(cherry picked from commit d898d20)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington added a commit that referenced this pull request Oct 26, 2019
@miss-islington @serhiy-storchaka
bpo-38557: Improve documentation for list and tuple C API. (GH-16925)
4992dc6 
(cherry picked from commit d898d20)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
jacobneiltaylor pushed a commit to jacobneiltaylor/cpython that referenced this pull request Dec 5, 2019
@serhiy-storchaka
bpo-38557: Improve documentation for list and tuple C API. (pythonGH-…
d8abaa7 
…16925)
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
@serhiy-storchaka @shihai1991
bpo-38557: Improve documentation for list and tuple C API. (pythonGH-…
720cc96 
…16925)
wimglenn added a commit to wimglenn/cpython that referenced this pull request Oct 23, 2022
@wimglenn
typo fix in c-api/tuple.rst
0dcceda 
copy-and-paste error from python#16925
@wimglenn wimglenn mentioned this pull request Oct 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@terryjreedy terryjreedy terryjreedy approved these changes

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@serhiy-storchaka @miss-islington @bedevere-bot @terryjreedy @the-knights-who-say-ni