[issue26373] asyncio: add support for async context manager on streams?

[STINNER Victor]

STINNER Victor added the comment:

I'm not sure asynico anymore and lost track of this issue, so I just close it.

--
resolution:  -> out of date
stage:  -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue26373] asyncio: add support for async context manager on streams?

[Martin Panter]

Martin Panter added the comment:

Above two commits were actuall for Issue 26372

--
nosy: +martin.panter

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue26373] asyncio: add support for async context manager on streams?

[Roundup Robot]

Roundup Robot added the comment:

New changeset 883cfb3e28f9 by Gregory P. Smith in branch '3.5':
Fixes Issue #26373: subprocess.Popen.communicate now correctly ignores
https://hg.python.org/cpython/rev/883cfb3e28f9

New changeset 78e81de6d447 by Gregory P. Smith in branch 'default':
merge from 3.5 - Fixes Issue #26373: subprocess.Popen.communicate
https://hg.python.org/cpython/rev/78e81de6d447

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue26373] asyncio: add support for async context manager on streams?

[STINNER Victor]

New submission from STINNER Victor:

While working on the issue #24911 (Context manager of socket.socket is not 
documented), I recalled that asyncio objects don't support context manager. 
With the PEP 492, it becomes possible to add support for async context manager 
for a few asyncio objects. I suggest to start with StreamWriter.

Usually, my rationale to decide which object should implement context manager 
is to check if the destructor can emit ResourceWarning.

In asyncio, ResourceWarning is now emitted by transport and event loop 
destructors:
https://docs.python.org/dev/library/asyncio-dev.html#close-transports-and-event-loops

--
components: asyncio
messages: 260394
nosy: gvanrossum, haypo, yselivanov
priority: normal
severity: normal
status: open
title: asyncio: add support for async context manager on streams?
type: enhancement
versions: Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com