-
Notifications
You must be signed in to change notification settings - Fork 13.4k
Soft-destabilize RustcEncodable
& RustcDecodable
, remove from prelude in next edition
#116016
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,12 +1,8 @@ | ||
#[crate_type="lib"] | ||
|
||
// #13544 | ||
|
||
extern crate rustc_serialize; | ||
|
||
#[derive(RustcEncodable)] pub struct A; | ||
#[derive(RustcEncodable)] pub struct B(isize); | ||
#[derive(RustcEncodable)] pub struct C { x: isize } | ||
#[derive(RustcEncodable)] pub enum D {} | ||
#[derive(RustcEncodable)] pub enum E { y } | ||
#[derive(RustcEncodable)] pub enum F { z(isize) } | ||
#[derive(Debug)] pub struct A; | ||
#[derive(Debug)] pub struct B(isize); | ||
#[derive(Debug)] pub struct C { x: isize } | ||
#[derive(Debug)] pub enum D {} | ||
#[derive(Debug)] pub enum E { y } | ||
#[derive(Debug)] pub enum F { z(isize) } |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
#![crate_type = "lib"] | ||
|
||
// This isn't intended to compile, so it's easiest to just ignore this error. | ||
extern crate rustc_serialize; //~ERROR can't find crate for `rustc_serialize` | ||
|
||
#[derive( | ||
RustcEncodable, | ||
//~^ ERROR use of unstable library feature 'rustc_encodable_decodable' | ||
//~^^ WARNING this was previously accepted by the compiler | ||
//~^^^ WARNING use of deprecated macro `RustcEncodable` | ||
RustcDecodable, | ||
//~^ ERROR use of unstable library feature 'rustc_encodable_decodable' | ||
//~^^ WARNING this was previously accepted by the compiler | ||
//~^^^ WARNING use of deprecated macro `RustcDecodable` | ||
)] | ||
struct S; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,66 @@ | ||
error[E0463]: can't find crate for `rustc_serialize` | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:4:1 | ||
| | ||
LL | extern crate rustc_serialize; | ||
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate | ||
| | ||
= help: maybe you need to install the missing components with: `rustup component add rust-src rustc-dev llvm-tools-preview` | ||
|
||
error: use of unstable library feature 'rustc_encodable_decodable': derive macro for `rustc-serialize`; should not be used in new code | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:7:5 | ||
| | ||
LL | RustcEncodable, | ||
| ^^^^^^^^^^^^^^ | ||
| | ||
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! | ||
= note: for more information, see issue #64266 <https://github.com/rust-lang/rust/issues/64266> | ||
jhpratt marked this conversation as resolved.
Show resolved
Hide resolved
|
||
= note: `#[deny(soft_unstable)]` on by default | ||
|
||
warning: use of deprecated macro `RustcEncodable`: rustc-serialize is deprecated and no longer supported | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:7:5 | ||
| | ||
LL | RustcEncodable, | ||
| ^^^^^^^^^^^^^^ | ||
| | ||
= note: `#[warn(deprecated)]` on by default | ||
|
||
error: use of unstable library feature 'rustc_encodable_decodable': derive macro for `rustc-serialize`; should not be used in new code | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:11:5 | ||
| | ||
LL | RustcDecodable, | ||
| ^^^^^^^^^^^^^^ | ||
| | ||
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! | ||
= note: for more information, see issue #64266 <https://github.com/rust-lang/rust/issues/64266> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would it be worth making this warning show up in cargo's future-compat reports? Not all future-compat warnings show up there, unfortunately. It needs to have There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think that would be worthwhile, but in a separate PR as it's only tangentially related to this. |
||
|
||
warning: use of deprecated macro `RustcDecodable`: rustc-serialize is deprecated and no longer supported | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:11:5 | ||
| | ||
LL | RustcDecodable, | ||
| ^^^^^^^^^^^^^^ | ||
|
||
error: aborting due to 3 previous errors; 2 warnings emitted | ||
|
||
For more information about this error, try `rustc --explain E0463`. | ||
Future incompatibility report: Future breakage diagnostic: | ||
error: use of unstable library feature 'rustc_encodable_decodable': derive macro for `rustc-serialize`; should not be used in new code | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:7:5 | ||
| | ||
LL | RustcEncodable, | ||
| ^^^^^^^^^^^^^^ | ||
| | ||
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! | ||
= note: for more information, see issue #64266 <https://github.com/rust-lang/rust/issues/64266> | ||
= note: `#[deny(soft_unstable)]` on by default | ||
|
||
Future breakage diagnostic: | ||
error: use of unstable library feature 'rustc_encodable_decodable': derive macro for `rustc-serialize`; should not be used in new code | ||
--> $DIR/feature-gate-rustc_encodable_decodable.rs:11:5 | ||
| | ||
LL | RustcDecodable, | ||
| ^^^^^^^^^^^^^^ | ||
| | ||
= warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release! | ||
= note: for more information, see issue #64266 <https://github.com/rust-lang/rust/issues/64266> | ||
= note: `#[deny(soft_unstable)]` on by default | ||
|
Uh oh!
There was an error while loading. Please reload this page.