classification
Title: Add sock_recv_into to AbstractEventLoop
Type: enhancement Stage: resolved
Components: asyncio, Library (Lib) Versions: Python 3.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: giampaolo.rodola, gvanrossum, pitrou, vstinner, yselivanov
Priority: normal Keywords: patch

Created on 2017-10-19 17:37 by pitrou, last changed 2017-10-19 19:47 by yselivanov. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 4051 merged pitrou, 2017-10-19 17:38
Messages (5)
msg304623 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2017-10-19 17:37
As sock_recv() is already exposed, it is a no-brainer to add sock_recv_into(), allowing finer buffer management when people are willing to handle a socket object.
msg304629 - (view) Author: Yury Selivanov (yselivanov) * (Python committer) Date: 2017-10-19 18:05
I'm going to approve this unless there are any objections.
msg304639 - (view) Author: Guido van Rossum (gvanrossum) * (Python committer) Date: 2017-10-19 19:42
SGTM.

On Oct 19, 2017 11:05 AM, "Yury Selivanov" <report@bugs.python.org> wrote:

>
> Yury Selivanov <yselivanov@gmail.com> added the comment:
>
> I'm going to approve this unless there are any objections.
>
> ----------
> nosy: +gvanrossum
>
> _______________________________________
> Python tracker <report@bugs.python.org>
> <https://bugs.python.org/issue31819>
> _______________________________________
>
msg304640 - (view) Author: Yury Selivanov (yselivanov) * (Python committer) Date: 2017-10-19 19:46
New changeset 525f40d231aba2c004619fc7a5207171ed65b0cb by Yury Selivanov (Antoine Pitrou) in branch 'master':
bpo-31819: Add AbstractEventLoop.sock_recv_into() (#4051)
https://github.com/python/cpython/commit/525f40d231aba2c004619fc7a5207171ed65b0cb
msg304641 - (view) Author: Yury Selivanov (yselivanov) * (Python committer) Date: 2017-10-19 19:47
Merged. Thank you, Antoine!
History
Date User Action Args
2017-10-19 19:47:17yselivanovsetstatus: open -> closed
resolution: fixed
messages: + msg304641

stage: patch review -> resolved
2017-10-19 19:46:42yselivanovsetmessages: + msg304640
2017-10-19 19:42:38gvanrossumsetmessages: + msg304639
2017-10-19 18:05:30yselivanovsetnosy: + gvanrossum
messages: + msg304629
2017-10-19 17:38:43pitrousetkeywords: + patch
stage: patch review
pull_requests: + pull_request4020
2017-10-19 17:37:49pitroucreate