Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

remove MaybeUninit::uninit_array #134585

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cyrgani
Copy link
Contributor

@cyrgani cyrgani commented Dec 20, 2024

Closes #134584.
The future of this unstable method was described in #125082 (comment). Since inline_const was stabilized in 1.79 (4 stable releases away) and no one expressed interest for keeping it in #96097, I think it can be removed now as it is not a stable method.

@rustbot
Copy link
Collaborator

rustbot commented Dec 20, 2024

r? @joboet

rustbot has assigned @joboet.
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

@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 Dec 20, 2024
@joboet
Copy link
Member

joboet commented Dec 27, 2024

LGTM. Just to make sure though that the plan is still current, let me reassign this to T-libs-API
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 Dec 27, 2024
@rustbot rustbot assigned m-ou-se and unassigned joboet Dec 27, 2024
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. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MaybeUninit::uninit_array still refers to const as future
4 participants