diff options
| author | Victor Stinner <vstinner@python.org> | 2023-09-20 13:54:19 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-09-20 13:54:19 (GMT) |
| commit | ced6924630037f1e5b3d1dbef2b600152fb07fbb (patch) | |
| tree | 9dd1a37eb08179229e59319f3feba36161325486 /Lib/asyncio | |
| parent | 850cc8d0b1db0a912a6e458720e265e6a6e5c1ba (diff) | |
| download | cpython-ced6924630037f1e5b3d1dbef2b600152fb07fbb.zip cpython-ced6924630037f1e5b3d1dbef2b600152fb07fbb.tar.gz cpython-ced6924630037f1e5b3d1dbef2b600152fb07fbb.tar.bz2 | |
gh-108973: Fix asyncio test_subprocess_consistent_callbacks() (#109431)
SubprocessProtocol process_exited() method can be called before
pipe_data_received() and pipe_connection_lost() methods. Document it
and adapt the test for that.
Revert commit 282edd7b2a74c4dfe1bfe3c5b1d30f9c21d554d6.
_child_watcher_callback() calls immediately _process_exited(): don't
add an additional delay with call_soon(). The reverted change didn't
make _process_exited() more determistic: it can still be called
before pipe_connection_lost() for example.
Co-authored-by: Davide Rizzo <sorcio@gmail.com>
Diffstat (limited to 'Lib/asyncio')
| -rw-r--r-- | Lib/asyncio/unix_events.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/Lib/asyncio/unix_events.py b/Lib/asyncio/unix_events.py index a268086..28cef96 100644 --- a/Lib/asyncio/unix_events.py +++ b/Lib/asyncio/unix_events.py @@ -226,8 +226,7 @@ class _UnixSelectorEventLoop(selector_events.BaseSelectorEventLoop): return transp def _child_watcher_callback(self, pid, returncode, transp): - # Skip one iteration for callbacks to be executed - self.call_soon_threadsafe(self.call_soon, transp._process_exited, returncode) + self.call_soon_threadsafe(transp._process_exited, returncode) async def create_unix_connection( self, protocol_factory, path=None, *, |