Skip to content

std::io: vectored reads with uninitialized memory (Read::read_buf_vec) #101842

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

std::io: vectored reads with uninitialized memory (Read::read_buf_vec) #101842

wants to merge 2 commits into from

Conversation

nrc
Copy link
Member

@nrc nrc commented Sep 15, 2022 •
edited by Dylan-DPC
Loading

Proposed API for vectored reads with uninitialized memory. The major contribution is BorrowedSliceBuf and BorrowedSliceCursor which extend the BorrowedBuf/BorrowedCursor API to slices of slices for vectored IO.

libs-api ACP [Accepted]

@nrc
WIP: vectorised borrow cursor
e8b2c6d 
Signed-off-by: Nick Cameron <nrc@ncameron.org>
@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Sep 15, 2022
@rust-highfive
Copy link
Contributor

r? @m-ou-se

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 15, 2022
@nrc
Copy link
Member Author

nrc commented Sep 15, 2022

@rustbot label -T-libs +T-libs-api

@rustbot rustbot added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. and removed T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Sep 15, 2022
@nrc nrc mentioned this pull request Sep 15, 2022
Closed
@rust-log-analyzer

This comment has been minimized.

# to view
@bors
Copy link
Collaborator

bors commented Oct 12, 2022

☔ The latest upstream changes (presumably #102948) made this pull request unmergeable. Please resolve the merge conflicts.

@nrc
Address some safety/UB issues
03530ce 
Signed-off-by: Nick Cameron <nrc@ncameron.org>
@Dylan-DPC Dylan-DPC added S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 16, 2023
@Dylan-DPC Dylan-DPC added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 5, 2024
@Dylan-DPC
Copy link
Member

@nrc any updates on this? thanks

@Dylan-DPC
Copy link
Member

Closing this as it's inactive and has collected several conflicts already

@Dylan-DPC Dylan-DPC closed this Jul 25, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@m-ou-se m-ou-se

Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@nrc @rust-highfive @rust-log-analyzer @bors @Dylan-DPC @m-ou-se @rustbot