Skip to content

Move std::io::read_to_string to alloc::io#158545

Open
bushrat011899 wants to merge 17 commits into
rust-lang:mainfrom
bushrat011899:alloc_io_read_to_string
Open

Move std::io::read_to_string to alloc::io#158545
bushrat011899 wants to merge 17 commits into
rust-lang:mainfrom
bushrat011899:alloc_io_read_to_string

Conversation

@bushrat011899

@bushrat011899 bushrat011899 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

ACP: rust-lang/libs-team#755
Tracking issue: #154046
Split From: #156527
Blocked On: #158544

Description

Moves std::io::read_to_string to alloc::io. This is a trivial move. Blocked on #158544.

Notes

@bushrat011899
Move std::io::IoHandle to core::io
9465c10
@bushrat011899
Move std::io::SizeHint to core::io
6bda252
@bushrat011899
Move std::io::SeekFrom to core::io
1c90c57
@bushrat011899
Move std::io::Seek to core::io
71ae6cd
@bushrat011899
Fix documentation links to Seek
f6b7029
@bushrat011899
Add std::io::cursor::WriteThroughCursor
f5a0666 
Used to work around `Cursor` not being a fundamental type when implementing `Write` for `Cursor<Vec<u8>>` and other `alloc` types once `Cursor` and `Write` are both located in `core`.
@bushrat011899
Move std::io::default_write_vectored to core::io
9526f7c
@bushrat011899
Move std::io::cursor::slice_write to core::io
7f4e5d2 
Call to `<&mut [u8] as Write>::write` inlined as `Write` trait not yet moved into `core`.
@bushrat011899
Move std::io::cursor::slice_write_vectored to core::io
fc1f8ae
@bushrat011899
Move std::io::cursor::slice_write_all to core::io
8f67ee4
@bushrat011899
Move std::io::cursor::slice_write_all_vectored to core::io
c827e38
@bushrat011899
Move std::io::Write to core::io
b664520
@bushrat011899
Fix documentation links to Write
4cabc47
@bushrat011899
Move std::io::Read internals to alloc::io
1905456
@bushrat011899
Move std::io::append_to_string to alloc::io
955dd2e
@bushrat011899
Move std::io::Read to alloc::io
070f497
@bushrat011899
Move std::io::read_to_string to alloc::io
63e0a84
@rustbot

rustbot commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

r? @jhpratt

rustbot has assigned @jhpratt.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ChrisDenton, libs
  • @ChrisDenton, libs expanded to 13 candidates
  • Random selection from 6 candidates

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output. labels Jun 29, 2026
@bushrat011899

bushrat011899 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

@rustbot blocked

@rustbot rustbot added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 29, 2026
This was referenced Jun 29, 2026
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jhpratt jhpratt

Labels

S-blocked Status: Blocked on something else such as an RFC or other implementation work. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. T-rustdoc-frontend Relevant to the rustdoc-frontend team, which will review and decide on the web UI/UX output.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bushrat011899 @rustbot @jhpratt