- Improve comments

- Useless signaling

Author: Rémy Voet (ryv)

Lib/threading.py

@@ -1001,8 +1001,8 @@ def start(self):
             raise
         self._os_thread_handle.wait_bootstraped()
 
-        # It's possible that the _started event never occurs from the new Thread;
-        # e.g., it didn't have enough memory to call the initialization part of _bootstrap_inner.
+        # It's possible that the _bootstrap(_inner) fails in the new Thread (e.g. Memory Error);
+        # We have to clean `_limbo` and `_active` here to avoid inconsistent state as much as possible
         if self._os_thread_handle.is_failed():
             with _active_limbo_lock:
                 _limbo.pop(self, None)

Modules/_threadmodule.c

@@ -398,8 +398,8 @@ thread_run(void *boot_raw)
             PyErr_FormatUnraisable(
                 "Exception ignored in thread started by %R", boot->func);
         }
+        // Notify that the bootstraped is done and failed (e.g. Memory error).
         set_thread_handle_state(handle, THREAD_HANDLE_FAILED);
-        // Notify that the bootstraped is done, in case the Python didn't bootstrap correctly (e.g. Memory error)
         _PyEvent_Notify(&handle->thread_is_bootstraped);
     }
     else {
@@ -432,7 +432,6 @@ force_done(void *arg)
     assert(get_thread_handle_state(handle) == THREAD_HANDLE_STARTING);
     _PyEvent_Notify(&handle->thread_is_exiting);
     set_thread_handle_state(handle, THREAD_HANDLE_DONE);
-    _PyEvent_Notify(&handle->thread_is_bootstraped);
     return 0;
 }