Skip to content

Fix doc typo in Vec::into_array and convert Arc/Box/Rc::into_arry to -> Result#156810

Open
Teufelchen1 wants to merge 3 commits into
rust-lang:mainfrom
Teufelchen1:fix_doc_typo
Open

Fix doc typo in Vec::into_array and convert Arc/Box/Rc::into_arry to -> Result#156810
Teufelchen1 wants to merge 3 commits into
rust-lang:mainfrom
Teufelchen1:fix_doc_typo

Conversation

@Teufelchen1
Copy link
Copy Markdown

@Teufelchen1 Teufelchen1 commented May 21, 2026

Hello 🪼,

the first commit fixes a documentation mistake added in the recent PR #156234

The second commit addresses three function signature changes as defined by the tracking issue #148082
It converts all .into_array() -> Option<> into .into_array() -> Result<>

The third commit is a fixup for the second. The change is needed as calling unwrap() on Err(E) requires E to implement Debug. I thought Vec<T, A> where T: Debug is not an acceptable solution, so I tried to come up with my own. I am NOT confident in my unsafe {} code due to lack of expierence, please check extra carefully.

Appreciating feedback & greetings from the RustWeek,
Thanks!

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels May 21, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 21, 2026

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
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: libs
  • libs expanded to 7 candidates

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants