Reimplement tempfile._RandomNameSequence using a generator function. (#1075)

Lib/tempfile.py

@@ -133,32 +133,21 @@ def _sanitize_params(prefix, suffix, dir):
     return prefix, suffix, dir, output_type
 
 
-class _RandomNameSequence:
+def _RandomNameSequence():
-    """An instance of _RandomNameSequence generates an endless
+    """Generate an endless sequence of unpredictable strings which
-    sequence of unpredictable strings which can safely be incorporated
+    can safely be incorporated into file names.  Each string is 8
-    into file names.  Each string is six characters long.  Multiple
+    characters long.  Multiple threads and forked processes can
-    threads can safely use the same instance at the same time.
+    safely use the same instance at the same time."""
-
-    _RandomNameSequence is an iterator."""
 
     characters = "abcdefghijklmnopqrstuvwxyz0123456789_"
-
+    rng_pid = None
-    @property
+    while True:
-    def rng(self):
         cur_pid = _os.getpid()
-        if cur_pid != getattr(self, '_rng_pid', None):
+        if cur_pid != rng_pid:
-            self._rng = _Random()
+            choose = _Random().choice
-            self._rng_pid = cur_pid
+            rng_pid = cur_pid
-        return self._rng
+        letters = [choose(characters) for dummy in range(8)]
-
+        yield ''.join(letters)
-    def __iter__(self):
-        return self
-
-    def __next__(self):
-        c = self.characters
-        choose = self.rng.choice
-        letters = [choose(c) for dummy in range(8)]
-        return ''.join(letters)
 
 def _candidate_tempdir_list():
     """Generate a list of candidate temporary directories which

Lib/test/test_tempfile.py

@@ -1,4 +1,5 @@
 # tempfile.py unit tests.
+import collections.abc
 import tempfile
 import errno
 import io
@@ -290,9 +291,9 @@ class TestGetCandidateNames(BaseTestCase):
     """Test the internal function _get_candidate_names."""
 
     def test_retval(self):
-        # _get_candidate_names returns a _RandomNameSequence object
+        # _get_candidate_names returns an iterator
         obj = tempfile._get_candidate_names()
-        self.assertIsInstance(obj, tempfile._RandomNameSequence)
+        self.assertIsInstance(obj, collections.abc.Iterator)
 
     def test_same_thing(self):
         # _get_candidate_names always returns the same object