diff options
author | Guido van Rossum <guido@dropbox.com> | 2013-11-22 19:47:22 (GMT) |
---|---|---|
committer | Guido van Rossum <guido@dropbox.com> | 2013-11-22 19:47:22 (GMT) |
commit | 7a465647e47957e95acc3ae0f84d945b0c47707d (patch) | |
tree | 3c8ede712d84c77471c71f5deaad37508297a2af /Lib/asyncio | |
parent | 873d1226b7cd42c663a4e09fa43a561de2e9f463 (diff) | |
download | cpython-7a465647e47957e95acc3ae0f84d945b0c47707d.zip cpython-7a465647e47957e95acc3ae0f84d945b0c47707d.tar.gz cpython-7a465647e47957e95acc3ae0f84d945b0c47707d.tar.bz2 |
asyncio: Pass cancellation from wrapping Future to wrapped Future. By Saúl Ibarra Corretgé (mostly).
Diffstat (limited to 'Lib/asyncio')
-rw-r--r-- | Lib/asyncio/futures.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/Lib/asyncio/futures.py b/Lib/asyncio/futures.py index db27838..dd3e718 100644 --- a/Lib/asyncio/futures.py +++ b/Lib/asyncio/futures.py @@ -301,6 +301,8 @@ class Future: The other Future may be a concurrent.futures.Future. """ assert other.done() + if self.cancelled(): + return assert not self.done() if other.cancelled(): self.cancel() @@ -324,14 +326,17 @@ def wrap_future(fut, *, loop=None): """Wrap concurrent.futures.Future object.""" if isinstance(fut, Future): return fut - assert isinstance(fut, concurrent.futures.Future), \ 'concurrent.futures.Future is expected, got {!r}'.format(fut) - if loop is None: loop = events.get_event_loop() - new_future = Future(loop=loop) + + def _check_cancel_other(f): + if f.cancelled(): + fut.cancel() + + new_future.add_done_callback(_check_cancel_other) fut.add_done_callback( lambda future: loop.call_soon_threadsafe( new_future._copy_state, fut)) |