Skip to content

constify TryFrom<Vec> for array#155616

Open
Lars-Schumann wants to merge 1 commit intorust-lang:mainfrom
Lars-Schumann:const-array-try-from-vec
Open

constify TryFrom<Vec> for array#155616
Lars-Schumann wants to merge 1 commit intorust-lang:mainfrom
Lars-Schumann:const-array-try-from-vec

Conversation

@Lars-Schumann
Copy link
Copy Markdown
Contributor

@Lars-Schumann Lars-Schumann commented Apr 21, 2026 •
edited
Loading

Related issues
const_heap: #79597
const_convert: #143773
const_destruct: #133214

this makes Vec const Destruct along the way.

@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. labels Apr 21, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 21, 2026

r? @jackh726

rustbot has assigned @jackh726.
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: compiler
  • compiler expanded to 72 candidates
  • Random selection from 18 candidates

@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Lars-Schumann Lars-Schumann marked this pull request as draft April 21, 2026 21:26
@rustbot rustbot added 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-review Status: Awaiting review from the assignee but also interested parties. labels Apr 21, 2026
@Lars-Schumann Lars-Schumann force-pushed the const-array-try-from-vec branch from 9c141c1 to f59366f Compare April 21, 2026 21:29
@rust-log-analyzer

This comment has been minimized.

Sign in to view
@Lars-Schumann Lars-Schumann force-pushed the const-array-try-from-vec branch from f59366f to 8acdb36 Compare April 22, 2026 07:26
Lars-Schumann
Lars-Schumann commented Apr 22, 2026
View reviewed changes
Comment thread library/alloc/src/vec/mod.rs
#[cfg(not(no_global_oom_handling))]
use core::marker::Destruct;
use core::marker::{Freeze, PhantomData};
use core::marker::{Destruct, Freeze, PhantomData};
Copy link
Copy Markdown
Contributor Author

@Lars-Schumann Lars-Schumann Apr 22, 2026 •
edited by rustbot
Loading

Choose a reason for hiding this comment

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

I assume this cfg was previously here just to silence a warning, so it was fine to remove.

View changes since the review

@Lars-Schumann Lars-Schumann marked this pull request as ready for review April 22, 2026 08:01
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jackh726 jackh726

Labels

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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Lars-Schumann @rustbot @rust-log-analyzer @jackh726