-
Notifications
You must be signed in to change notification settings - Fork 13k
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 the deprecated core::raw
and std::raw
module.
#86833
Conversation
This comment has been minimized.
This comment has been minimized.
2320ad9
to
efaf4d3
Compare
This comment has been minimized.
This comment has been minimized.
efaf4d3
to
4bc353b
Compare
This comment has been minimized.
This comment has been minimized.
4bc353b
to
0d1919c
Compare
Looks good, thanks! This removal was proposed and accepted in the pointer metadata RFC rust-lang/rfcs#2580 @bors r+ |
📌 Commit 0d1919c has been approved by |
Wait, what? This is stable, you can't just remove it. @bors r- cc @rust-lang/libs-api |
Oh I'm sorry, it was unstable but annotated in the module so I didn't see it. Sorry for the ping. @bors r=SimonSapin |
📌 Commit 0d1919c has been approved by |
Yeah this was never stable. warning: use of deprecated module `std::raw`: use pointer metadata APIs instead https://github.com/rust-lang/rust/issues/81513
--> src/lib.rs:1:5
|
1 | use core::raw;
| ^^^^^^^^^
|
= note: `#[warn(deprecated)]` on by default
error[E0658]: use of unstable library feature 'raw'
--> src/lib.rs:1:5
|
1 | use core::raw;
| ^^^^^^^^^
|
= note: see issue #27751 <https://github.com/rust-lang/rust/issues/27751> for more information
warning: use of deprecated module `std::raw`: use pointer metadata APIs instead https://github.com/rust-lang/rust/issues/81513
--> src/lib.rs:2:5
|
2 | use std::raw;
| ^^^^^^^^
error[E0658]: use of unstable library feature 'raw'
--> src/lib.rs:2:5
|
2 | use std::raw;
| ^^^^^^^^
|
= note: see issue #27751 <https://github.com/rust-lang/rust/issues/27751> for more information |
☀️ Test successful - checks-actions |
A few months has passed since #84207. I think now it's time for the final removal.
Closes #27751.
r? @m-ou-se