Skip to content

Commit

Permalink
Remove notes, increase S/N ratio
Browse files Browse the repository at this point in the history
  • Loading branch information
estebank committed Mar 7, 2021
1 parent 63fb294 commit aa7ac6e
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 21 deletions.
29 changes: 11 additions & 18 deletions compiler/rustc_ast_lowering/src/expr.rs
Expand Up @@ -366,24 +366,17 @@ impl<'hir> LoweringContext<'_, 'hir> {
fn if_let_expr_with_parens(&mut self, cond: &Expr, paren: &Expr) {
let start = cond.span.until(paren.span);
let end = paren.span.shrink_to_hi().until(cond.span.shrink_to_hi());
let mut err = self.sess.struct_span_err(
vec![start, end],
"invalid parentheses around `let` expression in `if let`",
);
if self.sess.opts.unstable_features.is_nightly_build() {
err.note(
"only supported directly without parentheses in conditions of `if`- and \
`while`-expressions, as well as in `let` chains within parentheses",
);
} else {
err.note("variable declaration using `let` is a statement, not a condition");
}
err.multipart_suggestion(
"`if let` needs to be written without parentheses",
vec![(start, String::new()), (end, String::new())],
rustc_errors::Applicability::MachineApplicable,
);
err.emit();
self.sess
.struct_span_err(
vec![start, end],
"invalid parentheses around `let` expression in `if let`",
)
.multipart_suggestion(
"`if let` needs to be written without parentheses",
vec![(start, String::new()), (end, String::new())],
rustc_errors::Applicability::MachineApplicable,
)
.emit();
// Ideally, we'd remove the feature gating of a `let` expression since we are already
// complaining about it here, but `feature_gate::check_crate` has already run by now:
// self.sess.parse_sess.gated_spans.ungate_last(sym::let_chains, paren.span);
Expand Down
3 changes: 0 additions & 3 deletions src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr
Expand Up @@ -333,7 +333,6 @@ error: invalid parentheses around `let` expression in `if let`
LL | if (let 0 = 1) {}
| ^ ^
|
= note: only supported directly without parentheses in conditions of `if`- and `while`-expressions, as well as in `let` chains within parentheses
help: `if let` needs to be written without parentheses
|
LL | if let 0 = 1 {}
Expand All @@ -345,7 +344,6 @@ error: invalid parentheses around `let` expression in `if let`
LL | if (((let 0 = 1))) {}
| ^^^ ^^^
|
= note: only supported directly without parentheses in conditions of `if`- and `while`-expressions, as well as in `let` chains within parentheses
help: `if let` needs to be written without parentheses
|
LL | if let 0 = 1 {}
Expand Down Expand Up @@ -581,7 +579,6 @@ error: invalid parentheses around `let` expression in `if let`
LL | use_expr!((let 0 = 1));
| ^ ^
|
= note: only supported directly without parentheses in conditions of `if`- and `while`-expressions, as well as in `let` chains within parentheses
help: `if let` needs to be written without parentheses
|
LL | use_expr!(let 0 = 1);
Expand Down

0 comments on commit aa7ac6e

Please sign in to comment.