Skip to content

gh-148467: Move BufferedIOBase close to concrete implementations#150838

Open
serhiy-storchaka wants to merge 1 commit into
python:mainfrom
serhiy-storchaka:docs-move-BufferedIOBase
Open

gh-148467: Move BufferedIOBase close to concrete implementations#150838
serhiy-storchaka wants to merge 1 commit into
python:mainfrom
serhiy-storchaka:docs-move-BufferedIOBase

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Jun 3, 2026

Copy link
Copy Markdown
Member

@serhiy-storchaka serhiy-storchaka added docs Documentation in the Doc dir skip news needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Jun 3, 2026
@serhiy-storchaka serhiy-storchaka added the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Jun 3, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Jun 3, 2026
@serhiy-storchaka

Copy link
Copy Markdown
Member Author

@birkenfeld, @benjaminp, please take a look.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #32968018 | 📁 Comparing 3cb0a65 against main (494f2e3)

  🔍 Preview build  

1 file changed
± library/io.html

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. You have to fix the conflicts, however.

Comment thread Doc/library/io.rst
implies writing, so this mode behaves in a similar way to ``'w'``. Add a
``'+'`` to the mode to allow simultaneous reading and writing.

The :meth:`~RawIOBase.read` (when called with a positive argument),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

After this lands it looks like I missed this paragraph in gh-129011, these do have retries per PEP-475

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

(They retry for EINTR, they don't retry for partial read or write)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge docs Documentation in the Doc dir needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants