From 8f8de7380cd7fee4972a10240ad2b0fdc332b14d Mon Sep 17 00:00:00 2001 From: Raymond Hettinger Date: Sat, 2 Jan 2021 12:09:56 -0800 Subject: No need to test "istep==1" twice. (GH-24064) --- Lib/random.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Lib/random.py b/Lib/random.py index 97495f0..4142e2e 100644 --- a/Lib/random.py +++ b/Lib/random.py @@ -351,9 +351,9 @@ class Random(_random.Random): DeprecationWarning, 2) raise ValueError("non-integer step for randrange()") width = istop - istart - if istep == 1 and width > 0: - return istart + self._randbelow(width) if istep == 1: + if width > 0: + return istart + self._randbelow(width) raise ValueError("empty range for randrange() (%d, %d, %d)" % (istart, istop, width)) # Non-unit step argument supplied. @@ -363,10 +363,8 @@ class Random(_random.Random): n = (width + istep + 1) // istep else: raise ValueError("zero step for randrange()") - if n <= 0: raise ValueError("empty range for randrange()") - return istart + istep * self._randbelow(n) def randint(self, a, b): -- cgit v0.12