traverseda
/
cpython
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

No need to test "istep==1" twice. (GH-24064)

This commit is contained in:
Raymond Hettinger 2021-01-02 12:09:56 -08:00 committed by GitHub
parent 768fa145cf
commit 8f8de7380c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Lib/random.py
View File

@ -351,9 +351,9 @@ class Random(_random.Random):
DeprecationWarning, 2) DeprecationWarning, 2)
raise ValueError("non-integer step for randrange()") raise ValueError("non-integer step for randrange()")
width = istop - istart width = istop - istart
if istep == 1 and width > 0:
return istart + self._randbelow(width)
if istep == 1: if istep == 1:
if width > 0:
return istart + self._randbelow(width)
raise ValueError("empty range for randrange() (%d, %d, %d)" % (istart, istop, width)) raise ValueError("empty range for randrange() (%d, %d, %d)" % (istart, istop, width))
# Non-unit step argument supplied. # Non-unit step argument supplied.
@ -363,10 +363,8 @@ class Random(_random.Random):
n = (width + istep + 1) // istep n = (width + istep + 1) // istep
else: else:
raise ValueError("zero step for randrange()") raise ValueError("zero step for randrange()")
if n <= 0: if n <= 0:
raise ValueError("empty range for randrange()") raise ValueError("empty range for randrange()")
return istart + istep * self._randbelow(n) return istart + istep * self._randbelow(n)
def randint(self, a, b): def randint(self, a, b):