Skip to content

Conversation

@StanFromIreland
Copy link
Member

@StanFromIreland StanFromIreland commented Jul 10, 2025

This is a bug since, per docs:

StreamReaderWriter instances define the combined interfaces of StreamReader and StreamWriter classes. They inherit all other methods and attributes from the underlying stream.

Copy link
Member

@malemburg malemburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@malemburg
Copy link
Member

I'm not sure about the backports, though. This could be seen as a new feature.

@malemburg malemburg merged commit 35e2c35 into python:main Jul 10, 2025
44 checks passed
@miss-islington-app
Copy link

Thanks @StanFromIreland for the PR, and @malemburg for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 10, 2025
…iter` functions (pythonGH-136498)

Closes pythonGH-52876
(cherry picked from commit 35e2c35)

Co-authored-by: Stan Ulbrych <[email protected]>
@StanFromIreland
Copy link
Member Author

StanFromIreland commented Jul 10, 2025

It currently contradicts the docs, and this would bring it to the expected behaviour. There was a similar consensus on the issue.

@bedevere-app
Copy link

bedevere-app bot commented Jul 10, 2025

GH-136514 is a backport of this pull request to the 3.14 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 10, 2025
…iter` functions (pythonGH-136498)

Closes pythonGH-52876
(cherry picked from commit 35e2c35)

Co-authored-by: Stan Ulbrych <[email protected]>
@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jul 10, 2025
@bedevere-app
Copy link

bedevere-app bot commented Jul 10, 2025

GH-136515 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jul 10, 2025
@malemburg
Copy link
Member

It currently contradicts the docs, and this would bring it to the expected behaviour. There was a similar consensus on the issue.

Fair enough.

malemburg pushed a commit that referenced this pull request Jul 10, 2025
…riter` functions (GH-136498) (#136515)

gh-52876: Implement missing parameter in `codecs.StreamReaderWriter` functions (GH-136498)

Closes GH-52876
(cherry picked from commit 35e2c35)

Co-authored-by: Stan Ulbrych <[email protected]>
malemburg pushed a commit that referenced this pull request Jul 10, 2025
…riter` functions (GH-136498) (#136514)

gh-52876: Implement missing parameter in `codecs.StreamReaderWriter` functions (GH-136498)

Closes GH-52876
(cherry picked from commit 35e2c35)

Co-authored-by: Stan Ulbrych <[email protected]>
AndPuQing pushed a commit to AndPuQing/cpython that referenced this pull request Jul 11, 2025
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
picnixz pushed a commit to picnixz/cpython that referenced this pull request Jul 13, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Agent-Hellboy pushed a commit to Agent-Hellboy/cpython that referenced this pull request Aug 19, 2025
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…eaderWriter` functions (pythonGH-136498) (python#136514)

pythongh-52876: Implement missing parameter in `codecs.StreamReaderWriter` functions (pythonGH-136498)

Closes pythonGH-52876
(cherry picked from commit 35e2c35)

Co-authored-by: Stan Ulbrych <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants