minor inaccuracy in range_contains_long

[wimglenn]

BPO	28045
Nosy	@berkerpeksag, @wimglenn
Files	mywork.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2016-09-12.04:48:02.275>
created_at = <Date 2016-09-09.17:19:26.311>
labels = ['interpreter-core', 'type-bug', 'docs']
title = 'minor inaccuracy in range_contains_long'
updated_at = <Date 2016-09-12.04:48:02.272>
user = 'https://github.com/wimglenn'

bugs.python.org fields:

activity = <Date 2016-09-12.04:48:02.272>
actor = 'berker.peksag'
assignee = 'docs@python'
closed = True
closed_date = <Date 2016-09-12.04:48:02.275>
closer = 'berker.peksag'
components = ['Documentation', 'Interpreter Core']
creation = <Date 2016-09-09.17:19:26.311>
creator = 'wim.glenn'
dependencies = []
files = ['44505']
hgrepos = []
issue_num = 28045
keywords = ['patch']
message_count = 3.0
messages = ['275353', '275958', '275959']
nosy_count = 4.0
nosy_names = ['docs@python', 'python-dev', 'berker.peksag', 'wim.glenn']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue28045'
versions = ['Python 3.5', 'Python 3.6']

[wimglenn]

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.

[python-dev]

New changeset 957084395042 by Berker Peksag in branch '3.5':
Issue bpo-28045: Fix comment in range_contains_long()
https://hg.python.org/cpython/rev/957084395042

New changeset 924a64745168 by Berker Peksag in branch 'default':
Issue bpo-28045: Merge from 3.5
https://hg.python.org/cpython/rev/924a64745168

[berkerpeksag]

Good catch, thanks!