Skip to content

Commit

Permalink
Add suggestion for removing invalid path separator :: in function d…
Browse files Browse the repository at this point in the history
…efinition.

for example: `fn invalid_path_separator::<T>() {}`

fixes: rust-lang#130791
  • Loading branch information
surechen committed Sep 27, 2024
1 parent 1b5aa96 commit 67e682d
Show file tree
Hide file tree
Showing 6 changed files with 46 additions and 0 deletions.
3 changes: 3 additions & 0 deletions compiler/rustc_parse/messages.ftl
Original file line number Diff line number Diff line change
Expand Up @@ -415,6 +415,9 @@ parse_invalid_meta_item = expected unsuffixed literal, found `{$token}`
parse_invalid_offset_of = offset_of expects dot-separated field and variant names
parse_invalid_path_sep_in_fn_definition = invalid path separator in function definition
.suggestion = remove invalid path separator
parse_invalid_unicode_escape = invalid unicode character escape
.label = invalid escape
.help = unicode escape must {$surrogate ->
Expand Down
8 changes: 8 additions & 0 deletions compiler/rustc_parse/src/errors.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1755,6 +1755,14 @@ pub(crate) struct MissingFnParams {
pub span: Span,
}

#[derive(Diagnostic)]
#[diag(parse_invalid_path_sep_in_fn_definition)]
pub(crate) struct InvalidPathSepInFnDefinition {
#[primary_span]
#[suggestion(code = "", applicability = "machine-applicable", style = "verbose")]
pub span: Span,
}

#[derive(Diagnostic)]
#[diag(parse_missing_trait_in_trait_impl)]
pub(crate) struct MissingTraitInTraitImpl {
Expand Down
7 changes: 7 additions & 0 deletions compiler/rustc_parse/src/parser/generics.rs
Original file line number Diff line number Diff line change
Expand Up @@ -269,6 +269,13 @@ impl<'a> Parser<'a> {
/// | ( < lifetimes , typaramseq ( , )? > )
/// where typaramseq = ( typaram ) | ( typaram , typaramseq )
pub(super) fn parse_generics(&mut self) -> PResult<'a, ast::Generics> {
// invalid path separator `::` in function definition
// for example `fn invalid_path_separator::<T>() {}`
if self.eat_path_sep() {
self.dcx()
.emit_err(errors::InvalidPathSepInFnDefinition { span: self.prev_token.span });
}

let span_lo = self.token.span;
let (params, span) = if self.eat_lt() {
let params = self.parse_generic_params()?;
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
//@ run-rustfix

#[allow(dead_code)]
fn invalid_path_separator<T>() {}
//~^ ERROR invalid path separator in function definition

fn main() {}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
//@ run-rustfix

#[allow(dead_code)]
fn invalid_path_separator::<T>() {}
//~^ ERROR invalid path separator in function definition

fn main() {}
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
error: invalid path separator in function definition
--> $DIR/invalid-path-sep-in-fn-definition-issue-130791.rs:4:26
|
LL | fn invalid_path_separator::<T>() {}
| ^^
|
help: remove invalid path separator
|
LL - fn invalid_path_separator::<T>() {}
LL + fn invalid_path_separator<T>() {}
|

error: aborting due to 1 previous error

0 comments on commit 67e682d

Please sign in to comment.