Skip to content

bpo-42222: Modernize integer test/conversion in randrange() #23064

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rhettinger merged 9 commits into from
Dec 28, 2020
Merged

bpo-42222: Modernize integer test/conversion in randrange() #23064

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
e11b1dc
Use index() instead of testing int(x)==x
rhettinger Oct 31, 2020
ebe8391
Add blurb
rhettinger Oct 31, 2020
ed2f6fd
Deprecate float arguments and the ValueError
rhettinger Oct 31, 2020
ab9aeb7
Add more detail to blurb
rhettinger Dec 24, 2020
f32d175
Add tests
rhettinger Dec 24, 2020
ebcb257
Update the docs
rhettinger Dec 24, 2020
832d885
Improve blurb
rhettinger Dec 24, 2020
9e6945d
Cover the case with a unit step
rhettinger Dec 24, 2020
85cc9ee
More concise blurb
rhettinger Dec 25, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Deprecate float arguments and the ValueError
  • Loading branch information
@rhettinger
rhettinger committed Oct 31, 2020
commit ed2f6fdecb72405550b7e3a8ae104231c5818350
33 changes: 30 additions & 3 deletions Lib/random.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,16 @@ def randrange(self, start, stop=None, step=1):
try:
istart = _index(start)
except TypeError:
raise ValueError("non-integer arg 1 for randrange()")
if int(start) == start:
istart = int(start)
_warn('Float arguments to randrange() have been deprecated\n'
'since Python 3.10 and will be removed in a subsequent '
'version.',
DeprecationWarning, 2)
else:
_warn('randrange() will raise TypeError in the future',
DeprecationWarning, 2)
raise ValueError("non-integer arg 1 for randrange()")
if stop is None:
if istart > 0:
return self._randbelow(istart)
Expand All @@ -311,7 +320,16 @@ def randrange(self, start, stop=None, step=1):
try:
istop = _index(stop)
except TypeError:
raise ValueError("non-integer stop for randrange()")
if int(stop) == stop:
istop = int(stop)
_warn('Float arguments to randrange() have been deprecated\n'
'since Python 3.10 and will be removed in a subsequent '
'version.',
DeprecationWarning, 2)
else:
_warn('randrange() will raise TypeError in the future',
DeprecationWarning, 2)
raise ValueError("non-integer stop for randrange()")
width = istop - istart
if step == 1 and width > 0:
return istart + self._randbelow(width)
Expand All @@ -322,7 +340,16 @@ def randrange(self, start, stop=None, step=1):
try:
istep = _index(step)
except TypeError:
raise ValueError("non-integer step for randrange()")
if int(step) == step:
istep = int(step)
_warn('Float arguments to randrange() have been deprecated\n'
'since Python 3.10 and will be removed in a subsequent '
'version.',
DeprecationWarning, 2)
else:
_warn('randrange() will raise TypeError in the future',
DeprecationWarning, 2)
raise ValueError("non-integer step for randrange()")
if istep > 0:
n = (width + istep - 1) // istep
elif istep < 0:
Expand Down