➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: minor inaccuracy in range_contains_long
Type: behavior Stage: resolved
Components: Documentation, Interpreter Core Versions: Python 3.6, Python 3.5
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: docs@python Nosy List: berker.peksag, docs@python, python-dev, wim.glenn
Priority: normal Keywords: patch

Created on 2016-09-09 17:19 by wim.glenn, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
mywork.patch wim.glenn, 2016-09-09 17:19 review
Messages (3)
msg275353 - (view) Author: wim glenn (wim.glenn) * Date: 2016-09-09 17:19 
The [commented] code summarising this algorithm is incorrect because % has precedence over -  

Hey, no big deal it's just a comment after all, but may as well fix it up.
msg275958 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2016-09-12 04:46 
New changeset 957084395042 by Berker Peksag in branch '3.5':
Issue #28045: Fix comment in range_contains_long()
https://hg.python.org/cpython/rev/957084395042

New changeset 924a64745168 by Berker Peksag in branch 'default':
Issue #28045: Merge from 3.5
https://hg.python.org/cpython/rev/924a64745168
msg275959 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2016-09-12 04:48 
Good catch, thanks!
History
Date User Action Args
2022-04-11 14:58:36adminsetgithub: 72232
2016-09-12 04:48:02berker.peksagsetstatus: open -> closed

type: enhancement -> behavior
versions: - Python 2.7, Python 3.3, Python 3.4, Python 3.7
nosy: + berker.peksag

messages: + msg275959
resolution: fixed
stage: resolved
2016-09-12 04:46:58python-devsetnosy: + python-dev
messages: + msg275958
2016-09-09 17:19:26wim.glenncreate