From c364a4e30f1368542b061dbd4ea4861fbc71a7ee Mon Sep 17 00:00:00 2001 From: Guillaume Gomez Date: Mon, 2 Aug 2021 16:33:37 +0200 Subject: [PATCH] Assign E0784 to union expression error --- compiler/rustc_error_codes/src/error_codes.rs | 3 +- .../src/error_codes/E0784.md | 32 +++++++++++++++++++ compiler/rustc_typeck/src/check/expr.rs | 8 ++++- 3 files changed, 41 insertions(+), 2 deletions(-) create mode 100644 compiler/rustc_error_codes/src/error_codes/E0784.md diff --git a/compiler/rustc_error_codes/src/error_codes.rs b/compiler/rustc_error_codes/src/error_codes.rs index 65999ba707c..6175e7a5a08 100644 --- a/compiler/rustc_error_codes/src/error_codes.rs +++ b/compiler/rustc_error_codes/src/error_codes.rs @@ -478,6 +478,7 @@ E0780: include_str!("./error_codes/E0780.md"), E0781: include_str!("./error_codes/E0781.md"), E0782: include_str!("./error_codes/E0782.md"), E0783: include_str!("./error_codes/E0783.md"), +E0784: include_str!("./error_codes/E0784.md"), ; // E0006, // merged with E0005 // E0008, // cannot bind by-move into a pattern guard @@ -636,7 +637,7 @@ E0783: include_str!("./error_codes/E0783.md"), E0711, // a feature has been declared with conflicting stability attributes E0717, // rustc_promotable without stability attribute // E0721, // `await` keyword -// E0723, unstable feature in `const` context +// E0723, unstable feature in `const` context E0726, // non-explicit (not `'_`) elided lifetime in unsupported position // E0738, // Removed; errored on `#[track_caller] fn`s in `extern "Rust" { ... }`. E0772, // `'static' obligation coming from `impl dyn Trait {}` or `impl Foo for dyn Bar {}`. diff --git a/compiler/rustc_error_codes/src/error_codes/E0784.md b/compiler/rustc_error_codes/src/error_codes/E0784.md new file mode 100644 index 00000000000..b20b7039bf4 --- /dev/null +++ b/compiler/rustc_error_codes/src/error_codes/E0784.md @@ -0,0 +1,32 @@ +A union expression does not have exactly one field. + +Erroneous code example: + +```compile_fail,E0784 +union Bird { + pigeon: u8, + turtledove: u16, +} + +let bird = Bird {}; // error +let bird = Bird { pigeon: 0, turtledove: 1 }; // error +``` + +The key property of unions is that all fields of a union share common storage. +As a result, writes to one field of a union can overwrite its other fields, and +size of a union is determined by the size of its largest field. + +You can find more information about the union types in the [Rust reference]. + +Working example: + +``` +union Bird { + pigeon: u8, + turtledove: u16, +} + +let bird = Bird { pigeon: 0 }; // OK +``` + +[Rust reference]: https://doc.rust-lang.org/reference/items/unions.html diff --git a/compiler/rustc_typeck/src/check/expr.rs b/compiler/rustc_typeck/src/check/expr.rs index 08258aac96f..3f3640aee4f 100644 --- a/compiler/rustc_typeck/src/check/expr.rs +++ b/compiler/rustc_typeck/src/check/expr.rs @@ -1304,7 +1304,13 @@ impl<'a, 'tcx> FnCtxt<'a, 'tcx> { // Make sure the programmer specified correct number of fields. if kind_name == "union" { if ast_fields.len() != 1 { - tcx.sess.span_err(span, "union expressions should have exactly one field"); + struct_span_err!( + tcx.sess, + span, + E0784, + "union expressions should have exactly one field", + ) + .emit(); } } else if check_completeness && !error_happened && !remaining_fields.is_empty() { let no_accessible_remaining_fields = remaining_fields