Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit 21fdd54

Browse files
terrarier2111terrarier2111
committed
Improves parser diagnostics, fixes rust-lang#93867 
1 parent d201c81 commit 21fdd54

22 files changed

+135
-33
lines changed

‎compiler/rustc_parse/src/parser/diagnostics.rs

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ use std::ops::{Deref, DerefMut};
2929

3030
use std::mem::take;
3131

32+
use crate::parser;
3233
use tracing::{debug, trace};
3334

3435
const TURBOFISH_SUGGESTION_STR: &str =
@@ -400,6 +401,35 @@ impl<'a> Parser<'a> {
400401
.map(|x| TokenType::Token(x.clone()))
401402
.chain(inedible.iter().map(|x| TokenType::Token(x.clone())))
402403
.chain(self.expected_tokens.iter().cloned())
404+
.filter_map(|token| {
405+
// filter out suggestions which suggest the same token which was found and deemed incorrect
406+
fn is_ident_eq_keyword(found: &TokenKind, expected: &TokenType) -> bool {
407+
if let TokenKind::Ident(current_sym, _) = found {
408+
if let TokenType::Keyword(suggested_sym) = expected {
409+
return current_sym == suggested_sym;
410+
}
411+
}
412+
false
413+
}
414+
if token != parser::TokenType::Token(self.token.kind.clone()) {
415+
let eq = is_ident_eq_keyword(&self.token.kind, &token);
416+
// if the suggestion is a keyword and the found token is an ident,
417+
// the content of which are equal to the suggestion's content,
418+
// we can remove that suggestion (see the return None statement below)
419+
420+
// if this isn't the case however, and the suggestion is a token the
421+
// content of which is the same as the found token's, we remove it as well
422+
if !eq {
423+
if let TokenType::Token(kind) = &token {
424+
if kind == &self.token.kind {
425+
return None;
426+
}
427+
}
428+
return Some(token);
429+
}
430+
}
431+
return None;
432+
})
403433
.collect::<Vec<_>>();
404434
expected.sort_by_cached_key(|x| x.to_string());
405435
expected.dedup();

‎compiler/rustc_parse/src/parser/expr.rs

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -977,12 +977,26 @@ impl<'a> Parser<'a> {
977977

978978
fn parse_dot_or_call_expr_with_(&mut self, mut e: P<Expr>, lo: Span) -> PResult<'a, P<Expr>> {
979979
loop {
980-
if self.eat(&token::Question) {
980+
let has_question = if self.prev_token.kind == TokenKind::Ident(kw::Return, false) {
981+
// we are using noexpect here because we don't expect a `?` directly after a `return`
982+
// which could be suggested otherwise
983+
self.eat_noexpect(&token::Question)
984+
} else {
985+
self.eat(&token::Question)
986+
};
987+
if has_question {
981988
// `expr?`
982989
e = self.mk_expr(lo.to(self.prev_token.span), ExprKind::Try(e), AttrVec::new());
983990
continue;
984991
}
985-
if self.eat(&token::Dot) {
992+
let has_dot = if self.prev_token.kind == TokenKind::Ident(kw::Return, false) {
993+
// we are using noexpect here because we don't expect a `.` directly after a `return`
994+
// which could be suggested otherwise
995+
self.eat_noexpect(&token::Dot)
996+
} else {
997+
self.eat(&token::Dot)
998+
};
999+
if has_dot {
9861000
// expr.f
9871001
e = self.parse_dot_suffix_expr(lo, e)?;
9881002
continue;
@@ -1536,9 +1550,13 @@ impl<'a> Parser<'a> {
15361550
self.parse_for_expr(label, lo, attrs)
15371551
} else if self.eat_keyword(kw::Loop) {
15381552
self.parse_loop_expr(label, lo, attrs)
1539-
} else if self.check(&token::OpenDelim(Delimiter::Brace)) || self.token.is_whole_block() {
1553+
} else if self.check_noexpect(&token::OpenDelim(Delimiter::Brace))
1554+
|| self.token.is_whole_block()
1555+
{
15401556
self.parse_block_expr(label, lo, BlockCheckMode::Default, attrs)
1541-
} else if !ate_colon && (self.check(&TokenKind::Comma) || self.check(&TokenKind::Gt)) {
1557+
} else if !ate_colon
1558+
&& (self.check_noexpect(&TokenKind::Comma) || self.check_noexpect(&TokenKind::Gt))
1559+
{
15421560
// We're probably inside of a `Path<'a>` that needs a turbofish
15431561
let msg = "expected `while`, `for`, `loop` or `{` after a label";
15441562
self.struct_span_err(self.token.span, msg).span_label(self.token.span, msg).emit();

‎compiler/rustc_parse/src/parser/mod.rs

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -548,6 +548,22 @@ impl<'a> Parser<'a> {
548548
is_present
549549
}
550550

551+
fn check_noexpect(&self, tok: &TokenKind) -> bool {
552+
self.token == *tok
553+
}
554+
555+
/// Consumes a token 'tok' if it exists. Returns whether the given token was present.
556+
///
557+
/// the main purpose of this function is to reduce the cluttering of the suggestions list
558+
/// which using the normal eat method could introduce in some cases.
559+
pub fn eat_noexpect(&mut self, tok: &TokenKind) -> bool {
560+
let is_present = self.check_noexpect(tok);
561+
if is_present {
562+
self.bump()
563+
}
564+
is_present
565+
}
566+
551567
/// Consumes a token 'tok' if it exists. Returns whether the given token was present.
552568
pub fn eat(&mut self, tok: &TokenKind) -> bool {
553569
let is_present = self.check(tok);

‎compiler/rustc_parse/src/parser/path.rs

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ use super::ty::{AllowPlus, RecoverQPath, RecoverReturnSign};
22
use super::{Parser, Restrictions, TokenType};
33
use crate::maybe_whole;
44
use rustc_ast::ptr::P;
5-
use rustc_ast::token::{self, Delimiter, Token};
5+
use rustc_ast::token::{self, Delimiter, Token, TokenKind};
66
use rustc_ast::{
77
self as ast, AngleBracketedArg, AngleBracketedArgs, AnonConst, AssocConstraint,
88
AssocConstraintKind, BlockCheckMode, GenericArg, GenericArgs, Generics, ParenthesizedArgs,
@@ -96,7 +96,7 @@ impl<'a> Parser<'a> {
9696
/// ^ help: use double colon
9797
/// ```
9898
fn recover_colon_before_qpath_proj(&mut self) -> bool {
99-
if self.token.kind != token::Colon
99+
if !self.check_noexpect(&TokenKind::Colon)
100100
|| self.look_ahead(1, |t| !t.is_ident() || t.is_reserved_ident())
101101
{
102102
return false;
@@ -478,7 +478,7 @@ impl<'a> Parser<'a> {
478478
while let Some(arg) = self.parse_angle_arg(ty_generics)? {
479479
args.push(arg);
480480
if !self.eat(&token::Comma) {
481-
if self.token.kind == token::Semi
481+
if self.check_noexpect(&TokenKind::Semi)
482482
&& self.look_ahead(1, |t| t.is_ident() || t.is_lifetime())
483483
{
484484
// Add `>` to the list of expected tokens.
@@ -517,7 +517,11 @@ impl<'a> Parser<'a> {
517517
let arg = self.parse_generic_arg(ty_generics)?;
518518
match arg {
519519
Some(arg) => {
520-
if self.check(&token::Colon) | self.check(&token::Eq) {
520+
// we are using noexpect here because we first want to find out if either `=` or `:`
521+
// is present and then use that info to push the other token onto the tokens list
522+
let separated =
523+
self.check_noexpect(&token::Colon) || self.check_noexpect(&token::Eq);
524+
if separated && (self.check(&token::Colon) | self.check(&token::Eq)) {
521525
let arg_span = arg.span();
522526
let (binder, ident, gen_args) = match self.get_ident_from_generic_arg(&arg) {
523527
Ok(ident_gen_args) => ident_gen_args,
@@ -553,6 +557,14 @@ impl<'a> Parser<'a> {
553557
AssocConstraint { id: ast::DUMMY_NODE_ID, ident, gen_args, kind, span };
554558
Ok(Some(AngleBracketedArg::Constraint(constraint)))
555559
} else {
560+
// we only want to suggest `:` and `=` in contexts where the previous token
561+
// is an ident and the current token or the next token is an ident
562+
if self.prev_token.is_ident()
563+
&& (self.token.is_ident() || self.look_ahead(1, |token| token.is_ident()))
564+
{
565+
self.check(&token::Colon);
566+
self.check(&token::Eq);
567+
}
556568
Ok(Some(AngleBracketedArg::Arg(arg)))
557569
}
558570
}

‎compiler/rustc_parse/src/parser/stmt.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,10 @@ impl<'a> Parser<'a> {
262262
if let Ok(snip) = self.span_to_snippet(pat.span) {
263263
err.span_label(pat.span, format!("while parsing the type for `{}`", snip));
264264
}
265-
let err = if self.check(&token::Eq) {
265+
// we use noexpect here because we don't actually expect Eq to be here
266+
// but we are still checking for it in order to be able to handle it if
267+
// it is there
268+
let err = if self.check_noexpect(&token::Eq) {
266269
err.emit();
267270
None
268271
} else {

‎src/test/ui/parser/can-begin-expr-check.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,5 @@ pub fn main() {
1616
return break as ();
1717
}
1818

19-
return enum; //~ ERROR expected one of `.`, `;`, `?`, `}`, or an operator, found keyword `enum`
19+
return enum; //~ ERROR expected one of `;`, `}`, or an operator, found keyword `enum`
2020
}

‎src/test/ui/parser/can-begin-expr-check.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
error: expected one of `.`, `;`, `?`, `}`, or an operator, found keyword `enum`
1+
error: expected one of `;`, `}`, or an operator, found keyword `enum`
22
--> $DIR/can-begin-expr-check.rs:19:12
33
|
44
LL | return enum;
5-
| ^^^^ expected one of `.`, `;`, `?`, `}`, or an operator
5+
| ^^^^ expected one of `;`, `}`, or an operator
66

77
error: aborting due to previous error
88

‎src/test/ui/parser/duplicate-visibility.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ fn main() {}
22

33
extern "C" { //~ NOTE while parsing this item list starting here
44
pub pub fn foo();
5-
//~^ ERROR expected one of `(`, `async`, `const`, `default`, `extern`, `fn`, `pub`, `unsafe`, or `use`, found keyword `pub`
6-
//~| NOTE expected one of 9 possible tokens
5+
//~^ ERROR expected one of `(`, `async`, `const`, `default`, `extern`, `fn`, `unsafe`, or `use`, found keyword `pub`
6+
//~| NOTE expected one of 8 possible tokens
77
//~| HELP there is already a visibility modifier, remove one
88
//~| NOTE explicit visibility first seen here
99
} //~ NOTE the item list ends here

‎src/test/ui/parser/duplicate-visibility.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
1-
error: expected one of `(`, `async`, `const`, `default`, `extern`, `fn`, `pub`, `unsafe`, or `use`, found keyword `pub`
1+
error: expected one of `(`, `async`, `const`, `default`, `extern`, `fn`, `unsafe`, or `use`, found keyword `pub`
22
--> $DIR/duplicate-visibility.rs:4:9
33
|
44
LL | extern "C" {
55
| - while parsing this item list starting here
66
LL | pub pub fn foo();
77
| ^^^
88
| |
9-
| expected one of 9 possible tokens
9+
| expected one of 8 possible tokens
1010
| help: there is already a visibility modifier, remove one
1111
...
1212
LL | }

‎src/test/ui/parser/issues/issue-20616-2.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ type Type_1_<'a, T> = &'a T;
99
//type Type_1<'a T> = &'a T; // error: expected `,` or `>` after lifetime name, found `T`
1010

1111

12-
type Type_2 = Type_1_<'static ()>; //~ error: expected one of `,`, `:`, `=`, or `>`, found `(`
12+
type Type_2 = Type_1_<'static ()>; //~ error: expected one of `,` or `>`, found `(`
1313

1414

1515
//type Type_3<T> = Box<T,,>; // error: expected type, found `,`

‎src/test/ui/parser/issues/issue-20616-2.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
error: expected one of `,`, `:`, `=`, or `>`, found `(`
1+
error: expected one of `,` or `>`, found `(`
22
--> $DIR/issue-20616-2.rs:12:31
33
|
44
LL | type Type_2 = Type_1_<'static ()>;
5-
| ^ expected one of `,`, `:`, `=`, or `>`
5+
| ^ expected one of `,` or `>`
66
|
77
help: you might have meant to end the type parameters here
88
|

‎src/test/ui/parser/issues/issue-62660.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ struct Foo;
55

66
impl Foo {
77
pub fn foo(_: i32, self: Box<Self) {}
8-
//~^ ERROR expected one of `!`, `(`, `+`, `,`, `::`, `:`, `<`, `=`, or `>`, found `)`
8+
//~^ ERROR expected one of `!`, `(`, `+`, `,`, `::`, `<`, or `>`, found `)`
99
}
1010

1111
fn main() {}

‎src/test/ui/parser/issues/issue-62660.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
error: expected one of `!`, `(`, `+`, `,`, `::`, `:`, `<`, `=`, or `>`, found `)`
1+
error: expected one of `!`, `(`, `+`, `,`, `::`, `<`, or `>`, found `)`
22
--> $DIR/issue-62660.rs:7:38
33
|
44
LL | pub fn foo(_: i32, self: Box<Self) {}
5-
| ^ expected one of 9 possible tokens
5+
| ^ expected one of 7 possible tokens
66
|
77
help: you might have meant to end the type parameters here
88
|

‎src/test/ui/parser/issues/issue-84117.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,4 @@ fn main() {
66
//~| ERROR expected one of `!`, `.`, `::`, `;`, `?`, `else`, `{`, or an operator, found `,`
77
//~| ERROR expected one of `!`, `.`, `::`, `;`, `?`, `else`, `{`, or an operator, found `,`
88
}
9-
//~^ ERROR expected one of `,`, `:`, `=`, or `>`, found `}`
9+
//~^ ERROR expected one of `,` or `>`, found `}`

‎src/test/ui/parser/issues/issue-84117.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,11 @@ error: expected one of `!`, `.`, `::`, `;`, `?`, `else`, `{`, or an operator, fo
2121
LL | let outer_local:e_outer<&str, { let inner_local:e_inner<&str, }
2222
| ^ expected one of 8 possible tokens
2323

24-
error: expected one of `,`, `:`, `=`, or `>`, found `}`
24+
error: expected one of `,` or `>`, found `}`
2525
--> $DIR/issue-84117.rs:8:1
2626
|
2727
LL | let outer_local:e_outer<&str, { let inner_local:e_inner<&str, }
28-
| ----------- while parsing the type for `outer_local` - expected one of `,`, `:`, `=`, or `>`
28+
| ----------- while parsing the type for `outer_local` - expected one of `,` or `>`
2929
...
3030
LL | }
3131
| ^ unexpected token

‎src/test/ui/parser/issues/issue-93282.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,11 @@ error: expected `while`, `for`, `loop` or `{` after a label
44
LL | f<'a,>
55
| ^ expected `while`, `for`, `loop` or `{` after a label
66

7-
error: expected one of `.`, `:`, `;`, `?`, `for`, `loop`, `while`, `{`, `}`, or an operator, found `,`
7+
error: expected one of `.`, `:`, `;`, `?`, `for`, `loop`, `while`, `}`, or an operator, found `,`
88
--> $DIR/issue-93282.rs:2:9
99
|
1010
LL | f<'a,>
11-
| ^ expected one of 10 possible tokens
11+
| ^ expected one of 9 possible tokens
1212
|
1313
help: use `::<...>` instead of `<...>` to specify lifetime, type, or const arguments
1414
|

‎src/test/ui/parser/issues/issue-93867.rs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
pub struct Entry<'a, K, V> {
2+
k: &'a mut K,
3+
v: V,
4+
}
5+
6+
pub fn entry<'a, K, V>() -> Entry<'a K, V> {
7+
// ^ missing comma
8+
//~^^ expected one of `,` or `>`, found `K`
9+
unimplemented!()
10+
}

‎src/test/ui/parser/issues/issue-93867.stderr

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
error: expected one of `,` or `>`, found `K`
2+
--> $DIR/issue-93867.rs:6:38
3+
|
4+
LL | pub fn entry<'a, K, V>() -> Entry<'a K, V> {
5+
| ^ expected one of `,` or `>`
6+
|
7+
help: you might have meant to end the type parameters here
8+
|
9+
LL | pub fn entry<'a, K, V>() -> Entry<'a> K, V> {
10+
| +
11+
12+
error: aborting due to previous error
13+

‎src/test/ui/parser/lifetime-semicolon.fixed

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ struct Foo<'a, 'b> {
55
}
66

77
fn foo<'a, 'b>(_x: &mut Foo<'a, 'b>) {}
8-
//~^ ERROR expected one of `,`, `:`, `=`, or `>`, found `;`
8+
//~^ ERROR expected one of `,` or `>`, found `;`
99

1010
fn main() {}

‎src/test/ui/parser/lifetime-semicolon.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ struct Foo<'a, 'b> {
55
}
66

77
fn foo<'a, 'b>(_x: &mut Foo<'a; 'b>) {}
8-
//~^ ERROR expected one of `,`, `:`, `=`, or `>`, found `;`
8+
//~^ ERROR expected one of `,` or `>`, found `;`
99

1010
fn main() {}

‎src/test/ui/parser/lifetime-semicolon.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
error: expected one of `,`, `:`, `=`, or `>`, found `;`
1+
error: expected one of `,` or `>`, found `;`
22
--> $DIR/lifetime-semicolon.rs:7:31
33
|
44
LL | fn foo<'a, 'b>(_x: &mut Foo<'a; 'b>) {}
5-
| ^ expected one of `,`, `:`, `=`, or `>`
5+
| ^ expected one of `,` or `>`
66
|
77
help: use a comma to separate type parameters
88
|

‎src/test/ui/parser/require-parens-for-chained-comparison.stderr

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,11 +59,11 @@ error: expected `while`, `for`, `loop` or `{` after a label
5959
LL | let _ = f<'_, i8>();
6060
| ^ expected `while`, `for`, `loop` or `{` after a label
6161

62-
error: expected one of `.`, `:`, `;`, `?`, `else`, `for`, `loop`, `while`, `{`, or an operator, found `,`
62+
error: expected one of `.`, `:`, `;`, `?`, `else`, `for`, `loop`, `while`, or an operator, found `,`
6363
--> $DIR/require-parens-for-chained-comparison.rs:22:17
6464
|
6565
LL | let _ = f<'_, i8>();
66-
| ^ expected one of 10 possible tokens
66+
| ^ expected one of 9 possible tokens
6767
|
6868
help: use `::<...>` instead of `<...>` to specify lifetime, type, or const arguments
6969
|

0 commit comments

Comments
 (0)