Issue #19169: Micro refactoring with a micro benefit for brevity and speed.
authorRaymond Hettinger <python@rcn.com>
Sun, 6 Oct 2013 00:18:36 +0000 (17:18 -0700)
committerRaymond Hettinger <python@rcn.com>
Sun, 6 Oct 2013 00:18:36 +0000 (17:18 -0700)
Lib/random.py

index 4ebc7579884f011f88039e3ae23fc14907d3839f..3fac699d937db39786b5d24dbbe63662665b0b2a 100644 (file)
@@ -220,10 +220,11 @@ class Random(_random.Random):
                    Method=_MethodType, BuiltinMethod=_BuiltinMethodType):
         "Return a random int in the range [0,n).  Raises ValueError if n==0."
 
+        random = self.random
         getrandbits = self.getrandbits
         # Only call self.getrandbits if the original random() builtin method
         # has not been overridden or if a new getrandbits() was supplied.
-        if type(self.random) is BuiltinMethod or type(getrandbits) is Method:
+        if type(random) is BuiltinMethod or type(getrandbits) is Method:
             k = n.bit_length()  # don't use (n-1) here because n can be 1
             r = getrandbits(k)          # 0 <= r < 2**k
             while r >= n:
@@ -231,7 +232,6 @@ class Random(_random.Random):
             return r
         # There's an overriden random() method but no new getrandbits() method,
         # so we can only use random() from here.
-        random = self.random
         if n >= maxsize:
             _warn("Underlying random() generator does not supply \n"
                 "enough bits to choose from a population range this large.\n"