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 #79597, #143773, #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
@jackh726
Copy link
Copy Markdown
Member

jackh726 commented Apr 23, 2026

r? libs-api

@rustbot rustbot added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Apr 23, 2026
@rustbot rustbot assigned the8472 and unassigned jackh726 Apr 23, 2026
@Lars-Schumann
Copy link
Copy Markdown
Contributor Author

Lars-Schumann commented Apr 24, 2026

assigned reviewer seems inactive atm
r? rust-lang/wg-const-eval

@rustbot rustbot assigned lcnr and unassigned the8472 Apr 24, 2026
@lcnr
Copy link
Copy Markdown
Contributor

lcnr commented May 5, 2026

r? oli

is this blocked on figuring out std constification

@lcnr
Copy link
Copy Markdown
Contributor

lcnr commented May 5, 2026

r? oli-obk

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 5, 2026

Failed to set assignee to oli: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@rustbot rustbot assigned oli-obk and unassigned lcnr May 5, 2026
@oli-obk
Copy link
Copy Markdown
Contributor

oli-obk commented May 6, 2026

Huh why does this affect mir-opts?

I think this is fine under std constification, because it tests a very specific use case of a core type like Vec. At minimum we should have const Drop for Vec

@oli-obk oli-obk 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 May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@oli-obk oli-obk

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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-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

@Lars-Schumann @rustbot @rust-log-analyzer @jackh726 @lcnr @oli-obk @the8472