Skip to content

Commit 44b19e5

Browse files
committed
rc and cr more clear error message
1 parent b8005bf commit 44b19e5

File tree

5 files changed

+41
-2
lines changed

5 files changed

+41
-2
lines changed

compiler/rustc_parse/messages.ftl

+1
Original file line numberDiff line numberDiff line change
@@ -893,6 +893,7 @@ parse_unknown_prefix = prefix `{$prefix}` is unknown
893893
.label = unknown prefix
894894
.note = prefixed identifiers and literals are reserved since Rust 2021
895895
.suggestion_br = use `br` for a raw byte string
896+
.suggestion_cr = use `cr` for a raw C-string
896897
.suggestion_str = if you meant to write a string literal, use double quotes
897898
.suggestion_whitespace = consider inserting whitespace here
898899

compiler/rustc_parse/src/errors.rs

+7
Original file line numberDiff line numberDiff line change
@@ -2140,6 +2140,13 @@ pub(crate) enum UnknownPrefixSugg {
21402140
style = "verbose"
21412141
)]
21422142
UseBr(#[primary_span] Span),
2143+
#[suggestion(
2144+
parse_suggestion_cr,
2145+
code = "cr",
2146+
applicability = "maybe-incorrect",
2147+
style = "verbose"
2148+
)]
2149+
UseCr(#[primary_span] Span),
21432150
#[suggestion(
21442151
parse_suggestion_whitespace,
21452152
code = " ",

compiler/rustc_parse/src/lexer/mod.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,6 @@ impl<'psess, 'src> Lexer<'psess, 'src> {
256256
let lit_start = start + BytePos(prefix_len);
257257
self.pos = lit_start;
258258
self.cursor = Cursor::new(&str_before[prefix_len as usize..]);
259-
260259
self.report_unknown_prefix(start);
261260
let prefix_span = self.mk_sp(start, lit_start);
262261
return (Token::new(self.ident(start), prefix_span), preceded_by_whitespace);
@@ -789,13 +788,14 @@ impl<'psess, 'src> Lexer<'psess, 'src> {
789788
fn report_unknown_prefix(&self, start: BytePos) {
790789
let prefix_span = self.mk_sp(start, self.pos);
791790
let prefix = self.str_from_to(start, self.pos);
792-
793791
let expn_data = prefix_span.ctxt().outer_expn_data();
794792

795793
if expn_data.edition.at_least_rust_2021() {
796794
// In Rust 2021, this is a hard error.
797795
let sugg = if prefix == "rb" {
798796
Some(errors::UnknownPrefixSugg::UseBr(prefix_span))
797+
} else if prefix == "rc" {
798+
Some(errors::UnknownPrefixSugg::UseCr(prefix_span))
799799
} else if expn_data.is_root() {
800800
if self.cursor.first() == '\''
801801
&& let Some(start) = self.last_lifetime
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
// `rc` and `cr` are easy to confuse; check that we issue a suggestion to help.
2+
3+
//@ edition:2021
4+
5+
fn main() {
6+
rc"abc";
7+
//~^ ERROR: prefix `rc` is unknown
8+
//~| HELP: use `cr` for a raw C-string
9+
//~| ERROR: expected one of
10+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
error: prefix `rc` is unknown
2+
--> $DIR/raw-c-string-prefix.rs:6:5
3+
|
4+
LL | rc"abc";
5+
| ^^ unknown prefix
6+
|
7+
= note: prefixed identifiers and literals are reserved since Rust 2021
8+
help: use `cr` for a raw C-string
9+
|
10+
LL - rc"abc";
11+
LL + cr"abc";
12+
|
13+
14+
error: expected one of `!`, `.`, `::`, `;`, `?`, `{`, `}`, or an operator, found `"abc"`
15+
--> $DIR/raw-c-string-prefix.rs:6:7
16+
|
17+
LL | rc"abc";
18+
| ^^^^^ expected one of 8 possible tokens
19+
20+
error: aborting due to 2 previous errors
21+

0 commit comments

Comments
 (0)