From: Ariel Ben-Yehuda Date: Mon, 13 Jul 2015 18:04:00 +0000 (+0300) Subject: don't ICE when FRU is used on an enum variant X-Git-Url: https://git.lizzy.rs/?a=commitdiff_plain;h=a878f35d3b6669ca473dce7b741788abb95e061f;p=rust.git don't ICE when FRU is used on an enum variant Fixes #26948. --- diff --git a/src/librustc_typeck/check/mod.rs b/src/librustc_typeck/check/mod.rs index 5a71d1ed0b5..a2fee091689 100644 --- a/src/librustc_typeck/check/mod.rs +++ b/src/librustc_typeck/check/mod.rs @@ -3426,6 +3426,11 @@ fn check_struct_fields_on_error<'a,'tcx>(fcx: &FnCtxt<'a,'tcx>, let def = lookup_full_def(tcx, path.span, id); let struct_id = match def { def::DefVariant(enum_id, variant_id, true) => { + if let &Some(ref base_expr) = base_expr { + span_err!(tcx.sess, base_expr.span, E0401, + "functional record update syntax requires a struct"); + fcx.write_error(base_expr.id); + } check_struct_enum_variant(fcx, id, expr.span, enum_id, variant_id, &fields[..]); enum_id diff --git a/src/librustc_typeck/diagnostics.rs b/src/librustc_typeck/diagnostics.rs index 5027be5fb62..636a8aeb544 100644 --- a/src/librustc_typeck/diagnostics.rs +++ b/src/librustc_typeck/diagnostics.rs @@ -2209,6 +2209,7 @@ impl Baz for Bar { } // Note: This is OK E0392, // parameter `{}` is never used E0393, // the type parameter `{}` must be explicitly specified in an object // type because its default value `{}` references the type `Self`" - E0399 // trait items need to be implemented because the associated + E0399, // trait items need to be implemented because the associated // type `{}` was overridden + E0401 // functional record update requires a struct } diff --git a/src/test/compile-fail/issue-26948.rs b/src/test/compile-fail/issue-26948.rs new file mode 100644 index 00000000000..c63cb5defb7 --- /dev/null +++ b/src/test/compile-fail/issue-26948.rs @@ -0,0 +1,16 @@ +// Copyright 2015 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +fn main() { + enum Foo { A { x: u32 } } + let orig = Foo::A { x: 5 }; + Foo::A { x: 6, ..orig }; + //~^ ERROR functional record update syntax requires a struct +}