From 254b89d209eccf4b5a5fd835db1089e1a814eef9 Mon Sep 17 00:00:00 2001 From: Obei Sideg <obei.sideg@gmail.com> Date: Wed, 27 Jul 2022 02:07:07 +0300 Subject: [PATCH 1/2] Recover from c++ style `enum struct` new error message: `enum` and `struct` are mutually exclusive new suggestion: replace `enum struct` with `enum` --- compiler/rustc_parse/src/parser/item.rs | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/compiler/rustc_parse/src/parser/item.rs b/compiler/rustc_parse/src/parser/item.rs index 87bc0d9762ea5..58203a6fbe336 100644 --- a/compiler/rustc_parse/src/parser/item.rs +++ b/compiler/rustc_parse/src/parser/item.rs @@ -1216,6 +1216,25 @@ impl<'a> Parser<'a> { /// Parses an enum declaration. fn parse_item_enum(&mut self) -> PResult<'a, ItemInfo> { + if self.token.is_keyword(kw::Struct) { + let mut err = self.struct_span_err( + self.prev_token.span.to(self.token.span), + "`enum` and `struct` are mutually exclusive", + ); + err.span_suggestion( + self.prev_token.span.to(self.token.span), + "replace `enum struct` with", + "enum", + Applicability::MachineApplicable, + ); + if self.look_ahead(1, |t| t.is_ident()) { + self.bump(); + err.emit(); + } else { + return Err(err); + } + } + let id = self.parse_ident()?; let mut generics = self.parse_generics()?; generics.where_clause = self.parse_where_clause()?; From 0ad06f14820e90cb22b02b3678b6af20e0fe358a Mon Sep 17 00:00:00 2001 From: Obei Sideg <obei.sideg@gmail.com> Date: Wed, 27 Jul 2022 02:17:07 +0300 Subject: [PATCH 2/2] Add ui test for #99625 --- ...issue-99625-enum-struct-mutually-exclusive.fixed | 13 +++++++++++++ .../issue-99625-enum-struct-mutually-exclusive.rs | 13 +++++++++++++ ...ssue-99625-enum-struct-mutually-exclusive.stderr | 8 ++++++++ 3 files changed, 34 insertions(+) create mode 100644 src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.fixed create mode 100644 src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.rs create mode 100644 src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.stderr diff --git a/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.fixed b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.fixed new file mode 100644 index 0000000000000..4b4a416b1ac82 --- /dev/null +++ b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.fixed @@ -0,0 +1,13 @@ +// run-rustfix + +pub enum Range { + //~^ ERROR `enum` and `struct` are mutually exclusive + Valid { + begin: u32, + len: u32, + }, + Out, +} + +fn main() { +} diff --git a/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.rs b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.rs new file mode 100644 index 0000000000000..9cc886641293b --- /dev/null +++ b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.rs @@ -0,0 +1,13 @@ +// run-rustfix + +pub enum struct Range { + //~^ ERROR `enum` and `struct` are mutually exclusive + Valid { + begin: u32, + len: u32, + }, + Out, +} + +fn main() { +} diff --git a/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.stderr b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.stderr new file mode 100644 index 0000000000000..edc640bf5ec22 --- /dev/null +++ b/src/test/ui/parser/issue-99625-enum-struct-mutually-exclusive.stderr @@ -0,0 +1,8 @@ +error: `enum` and `struct` are mutually exclusive + --> $DIR/issue-99625-enum-struct-mutually-exclusive.rs:3:5 + | +LL | pub enum struct Range { + | ^^^^^^^^^^^ help: replace `enum struct` with: `enum` + +error: aborting due to previous error +