-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
codecs.StreamWriter.write() returns None #134706
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
Comments
Copied here from the discussion on the PR... I have thought a bit more about this PR and "bug" report:
Given all this, I'm not in favor of the change and will close the PR and ticket. |
Update the close reason... |
Uh oh!
There was an error while loading. Please reload this page.
Bug report
Bug description:
This violates the usual
write()
contract of returning the number of items written (for synchronous I/O) and also affectsStreamReaderWriter.write()
transitively.In this case it seems trivial to just return the result of the called
self.stream.write()
.CPython versions tested on:
CPython main branch
Operating systems tested on:
No response
Linked PRs
The text was updated successfully, but these errors were encountered: