Skip to content

Commit

Permalink
Unrolled build for rust-lang#125640
Browse files Browse the repository at this point in the history
Rollup merge of rust-lang#125640 - fmease:plz-no-stringify, r=estebank

Don't suggest turning non-char-literal exprs of ty `char` into string literals

Fixes rust-lang#125595.
Fixes rust-lang#125081.

r? estebank (rust-lang#122217) or compiler
  • Loading branch information
rust-timer authored May 28, 2024
2 parents c0d6003 + 27cdc0d commit 5b7e1ea
Show file tree
Hide file tree
Showing 6 changed files with 66 additions and 11 deletions.
13 changes: 9 additions & 4 deletions compiler/rustc_infer/src/infer/error_reporting/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2102,10 +2102,15 @@ impl<'tcx> TypeErrCtxt<'_, 'tcx> {
// If a string was expected and the found expression is a character literal,
// perhaps the user meant to write `"s"` to specify a string literal.
(ty::Ref(_, r, _), ty::Char) if r.is_str() => {
suggestions.push(TypeErrorAdditionalDiags::MeantStrLiteral {
start: span.with_hi(span.lo() + BytePos(1)),
end: span.with_lo(span.hi() - BytePos(1)),
})
if let Ok(code) = self.tcx.sess().source_map().span_to_snippet(span)
&& code.starts_with("'")
&& code.ends_with("'")
{
suggestions.push(TypeErrorAdditionalDiags::MeantStrLiteral {
start: span.with_hi(span.lo() + BytePos(1)),
end: span.with_lo(span.hi() - BytePos(1)),
});
}
}
// For code `if Some(..) = expr `, the type mismatch may be expected `bool` but found `()`,
// we try to suggest to add the missing `let` for `if let Some(..) = expr`
Expand Down
7 changes: 0 additions & 7 deletions tests/crashes/125081.rs

This file was deleted.

7 changes: 7 additions & 0 deletions tests/ui/inference/str-as-char-butchered.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
// issue: rust-lang/rust#125081

fn main() {
let _: &str = 'β;
//~^ ERROR expected `while`, `for`, `loop` or `{` after a label
//~| ERROR mismatched types
}
22 changes: 22 additions & 0 deletions tests/ui/inference/str-as-char-butchered.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
error: expected `while`, `for`, `loop` or `{` after a label
--> $DIR/str-as-char-butchered.rs:4:21
|
LL | let _: &str = 'β;
| ^ expected `while`, `for`, `loop` or `{` after a label
|
help: add `'` to close the char literal
|
LL | let _: &str = 'β';
| +

error[E0308]: mismatched types
--> $DIR/str-as-char-butchered.rs:4:19
|
LL | let _: &str = 'β;
| ---- ^^ expected `&str`, found `char`
| |
| expected due to this

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.
9 changes: 9 additions & 0 deletions tests/ui/inference/str-as-char-non-lit.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
// Don't suggest double quotes when encountering an expr of type `char` where a `&str`
// is expected if the expr is not a char literal.
// issue: rust-lang/rust#125595

fn main() {
let _: &str = ('a'); //~ ERROR mismatched types
let token = || 'a';
let _: &str = token(); //~ ERROR mismatched types
}
19 changes: 19 additions & 0 deletions tests/ui/inference/str-as-char-non-lit.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
error[E0308]: mismatched types
--> $DIR/str-as-char-non-lit.rs:6:19
|
LL | let _: &str = ('a');
| ---- ^^^^^ expected `&str`, found `char`
| |
| expected due to this

error[E0308]: mismatched types
--> $DIR/str-as-char-non-lit.rs:8:19
|
LL | let _: &str = token();
| ---- ^^^^^^^ expected `&str`, found `char`
| |
| expected due to this

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.

0 comments on commit 5b7e1ea

Please sign in to comment.