Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Reword docs to avoid saying things that aren't accurate#2523

Closed
Rosuav wants to merge 1 commit into
python:masterpython/cpython:masterfrom
Rosuav:range-not-functionRosuav/cpython:range-not-functionCopy head branch name to clipboard
Closed

Reword docs to avoid saying things that aren't accurate#2523
Rosuav wants to merge 1 commit into
python:masterpython/cpython:masterfrom
Rosuav:range-not-functionRosuav/cpython:range-not-functionCopy head branch name to clipboard

Conversation

@Rosuav

@Rosuav Rosuav commented Jul 1, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@mention-bot

Copy link
Copy Markdown

@Rosuav, thanks for your PR! By analyzing the history of the files in this pull request, we identified @1st1, @birkenfeld and @tiran to be potential reviewers.

@Rosuav

Rosuav commented Jul 1, 2017

Copy link
Copy Markdown
Contributor Author

(Trivial)

@wm75

wm75 commented Jul 2, 2017

Copy link
Copy Markdown

Compare to https://docs.python.org/3/tutorial/controlflow.html#the-range-function, which has the same problem. PR #273 proposes to change this section, but leaves the range is a function part unaltered. It introduces the returned object as an iterable instead of a sequence and, of course, either is correct.

@serhiy-storchaka serhiy-storchaka added the docs Documentation in the Doc dir label Dec 8, 2018
@rhettinger

Copy link
Copy Markdown
Contributor

Sorry, but I don't see any improvement here.

@rhettinger rhettinger closed this Dec 24, 2018
@Rosuav

Rosuav commented Dec 24, 2018

Copy link
Copy Markdown
Contributor Author

Fine, whatevs. I opened the PR in direct response to someone who was asking about how come range() wasn't behaving like an iterator, so there definitely is some inaccuracy and some confusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.