diff options
| author | Esteban Küber <esteban@kuber.com.ar> | 2021-03-07 15:03:46 -0800 |
|---|---|---|
| committer | Esteban Küber <esteban@kuber.com.ar> | 2021-03-07 15:03:46 -0800 |
| commit | aa7ac6e957747c0fc2edae126162bdfd8faff69c (patch) | |
| tree | 8f0c16367efd250fe3eaeef6bc676e1c0b640cfe | |
| parent | 63fb294a7455b76ecfbef6831410e5362ee9d514 (diff) | |
| download | rust-aa7ac6e957747c0fc2edae126162bdfd8faff69c.tar.gz rust-aa7ac6e957747c0fc2edae126162bdfd8faff69c.zip | |
Remove notes, increase S/N ratio
| -rw-r--r-- | compiler/rustc_ast_lowering/src/expr.rs | 29 | ||||
| -rw-r--r-- | src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr | 3 |
2 files changed, 11 insertions, 21 deletions
diff --git a/compiler/rustc_ast_lowering/src/expr.rs b/compiler/rustc_ast_lowering/src/expr.rs index 63fc4e0ceda..c9a5541585f 100644 --- a/compiler/rustc_ast_lowering/src/expr.rs +++ b/compiler/rustc_ast_lowering/src/expr.rs @@ -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); diff --git a/src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr b/src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr index 96131b38db9..7364f62c922 100644 --- a/src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr +++ b/src/test/ui/rfc-2497-if-let-chains/feature-gate.stderr @@ -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 {} @@ -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 {} @@ -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); |
