Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bpo-31062: Allow socket.makefile to handle line buffering properly #12370

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

vxgmichel
Copy link
Contributor

@vxgmichel vxgmichel commented Mar 16, 2019

Line buffering is enabled by using the argument buffering=1 as defined in io.open.

I did not update the docs for socket.makefile since it already mentions the arguments being interpreted the same way as open:

Return a file object associated with the socket. The exact returned type depends on the arguments given to makefile(). These arguments are interpreted the same way as by the built-in open() function, except the only supported mode values are 'r' (default), 'w' and 'b'.

https://bugs.python.org/issue31062

Lib/socket.py Outdated Show resolved Hide resolved
Lib/test/test_socket.py Outdated Show resolved Hide resolved
Line buffering is enabled by using the argument `buffering=1` as
defined in `io.open`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants