Try to recover from path sep error in parser

This commit is contained in:
yukang 2025-02-11 15:26:21 +08:00
parent d8810e3e2d
commit 0aa2e6b606
10 changed files with 92 additions and 40 deletions

View file

@ -743,9 +743,6 @@ parse_single_colon_import_path = expected `::`, found `:`
.suggestion = use double colon .suggestion = use double colon
.note = import paths are delimited using `::` .note = import paths are delimited using `::`
parse_single_colon_struct_type = found single colon in a struct field type path
.suggestion = write a path separator here
parse_static_with_generics = static items may not have generic parameters parse_static_with_generics = static items may not have generic parameters
parse_struct_literal_body_without_path = parse_struct_literal_body_without_path =

View file

@ -3071,14 +3071,6 @@ pub(crate) struct BadItemKind {
pub help: bool, pub help: bool,
} }
#[derive(Diagnostic)]
#[diag(parse_single_colon_struct_type)]
pub(crate) struct SingleColonStructType {
#[primary_span]
#[suggestion(code = "::", applicability = "maybe-incorrect", style = "verbose")]
pub span: Span,
}
#[derive(Diagnostic)] #[derive(Diagnostic)]
#[diag(parse_macro_rules_missing_bang)] #[diag(parse_macro_rules_missing_bang)]
pub(crate) struct MacroRulesMissingBang { pub(crate) struct MacroRulesMissingBang {

View file

@ -2043,9 +2043,6 @@ impl<'a> Parser<'a> {
} }
self.expect_field_ty_separator()?; self.expect_field_ty_separator()?;
let ty = self.parse_ty()?; let ty = self.parse_ty()?;
if self.token == token::Colon && self.look_ahead(1, |t| *t != token::Colon) {
self.dcx().emit_err(errors::SingleColonStructType { span: self.token.span });
}
let default = if self.token == token::Eq { let default = if self.token == token::Eq {
self.bump(); self.bump();
let const_expr = self.parse_expr_anon_const()?; let const_expr = self.parse_expr_anon_const()?;

View file

@ -246,8 +246,19 @@ impl<'a> Parser<'a> {
segments.push(segment); segments.push(segment);
if self.is_import_coupler() || !self.eat_path_sep() { if self.is_import_coupler() || !self.eat_path_sep() {
if style == PathStyle::Expr let ok_for_recovery = self.may_recover()
&& self.may_recover() && match style {
PathStyle::Expr => true,
PathStyle::Type if let Some((ident, _)) = self.prev_token.ident() => {
self.token == token::Colon
&& ident.as_str().chars().all(|c| c.is_lowercase())
&& self.token.span.lo() == self.prev_token.span.hi()
&& self
.look_ahead(1, |token| self.token.span.hi() == token.span.lo())
}
_ => false,
};
if ok_for_recovery
&& self.token == token::Colon && self.token == token::Colon
&& self.look_ahead(1, |token| token.is_ident() && !token.is_reserved_ident()) && self.look_ahead(1, |token| token.is_ident() && !token.is_reserved_ident())
{ {

View file

@ -1,8 +1,6 @@
error: path separator must be a double colon error: path separator must be a double colon
--> $DIR/single-colon-path-not-const-generics.rs:8:18 --> $DIR/single-colon-path-not-const-generics.rs:8:18
| |
LL | pub struct Foo {
| --- while parsing this struct
LL | a: Vec<foo::bar:A>, LL | a: Vec<foo::bar:A>,
| ^ | ^
| |

View file

@ -0,0 +1,15 @@
//@ run-rustfix
use std::fmt;
struct Hello;
impl fmt::Display for Hello {
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { //~ ERROR path separator must be a double colon
write!(f, "hello")
}
}
fn main() {
let _ = Hello;
}

View file

@ -0,0 +1,15 @@
//@ run-rustfix
use std::fmt;
struct Hello;
impl fmt::Display for Hello {
fn fmt(&self, f: &mut fmt:Formatter) -> fmt::Result { //~ ERROR path separator must be a double colon
write!(f, "hello")
}
}
fn main() {
let _ = Hello;
}

View file

@ -0,0 +1,14 @@
error: path separator must be a double colon
--> $DIR/argument-list-from-path-sep-error-129273.rs:8:30
|
LL | fn fmt(&self, f: &mut fmt:Formatter) -> fmt::Result {
| ^
|
= note: if you meant to annotate an expression with a type, the type ascription syntax has been removed, see issue #101728 <https://github.com/rust-lang/rust/issues/101728>
help: use a double colon instead
|
LL | fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
| +
error: aborting due to 1 previous error

View file

@ -7,14 +7,14 @@ mod foo {
struct Foo { struct Foo {
a: foo:A, a: foo:A,
//~^ ERROR found single colon in a struct field type path //~^ ERROR path separator must be a double colon
//~| expected `,`, or `}`, found `:` //~| ERROR struct `A` is private
} }
struct Bar { struct Bar {
b: foo::bar:B, b: foo::bar:B,
//~^ ERROR found single colon in a struct field type path //~^ ERROR path separator must be a double colon
//~| expected `,`, or `}`, found `:` //~| ERROR module `bar` is private
} }
fn main() {} fn main() {}

View file

@ -1,40 +1,53 @@
error: found single colon in a struct field type path error: path separator must be a double colon
--> $DIR/struct-field-type-including-single-colon.rs:9:11 --> $DIR/struct-field-type-including-single-colon.rs:9:11
| |
LL | a: foo:A, LL | a: foo:A,
| ^ | ^
| |
help: write a path separator here = note: if you meant to annotate an expression with a type, the type ascription syntax has been removed, see issue #101728 <https://github.com/rust-lang/rust/issues/101728>
help: use a double colon instead
| |
LL | a: foo::A, LL | a: foo::A,
| + | +
error: expected `,`, or `}`, found `:` error: path separator must be a double colon
--> $DIR/struct-field-type-including-single-colon.rs:9:11
|
LL | struct Foo {
| --- while parsing this struct
LL | a: foo:A,
| ^
error: found single colon in a struct field type path
--> $DIR/struct-field-type-including-single-colon.rs:15:16 --> $DIR/struct-field-type-including-single-colon.rs:15:16
| |
LL | b: foo::bar:B, LL | b: foo::bar:B,
| ^ | ^
| |
help: write a path separator here = note: if you meant to annotate an expression with a type, the type ascription syntax has been removed, see issue #101728 <https://github.com/rust-lang/rust/issues/101728>
help: use a double colon instead
| |
LL | b: foo::bar::B, LL | b: foo::bar::B,
| + | +
error: expected `,`, or `}`, found `:` error[E0603]: struct `A` is private
--> $DIR/struct-field-type-including-single-colon.rs:15:16 --> $DIR/struct-field-type-including-single-colon.rs:9:12
|
LL | a: foo:A,
| ^ private struct
|
note: the struct `A` is defined here
--> $DIR/struct-field-type-including-single-colon.rs:2:5
|
LL | struct A;
| ^^^^^^^^^
error[E0603]: module `bar` is private
--> $DIR/struct-field-type-including-single-colon.rs:15:13
| |
LL | struct Bar {
| --- while parsing this struct
LL | b: foo::bar:B, LL | b: foo::bar:B,
| ^ | ^^^ - struct `B` is not publicly re-exported
| |
| private module
|
note: the module `bar` is defined here
--> $DIR/struct-field-type-including-single-colon.rs:3:5
|
LL | mod bar {
| ^^^^^^^
error: aborting due to 4 previous errors error: aborting due to 4 previous errors
For more information about this error, try `rustc --explain E0603`.