1
Fork 0

Auto merge of #88547 - notriddle:notriddle/is-expr-delims-necessary, r=davidtwco

fix(rustc_lint): better detect when parens are necessary

Fixes #88519
This commit is contained in:
bors 2021-09-04 09:46:58 +00:00
commit 72a51c39c6
2 changed files with 104 additions and 7 deletions

View file

@ -461,13 +461,16 @@ trait UnusedDelimLint {
let lhs_needs_parens = {
let mut innermost = inner;
loop {
if let ExprKind::Binary(_, lhs, _rhs) = &innermost.kind {
innermost = lhs;
if !classify::expr_requires_semi_to_be_stmt(innermost) {
break true;
}
} else {
break false;
innermost = match &innermost.kind {
ExprKind::Binary(_, lhs, _rhs) => lhs,
ExprKind::Call(fn_, _params) => fn_,
ExprKind::Cast(expr, _ty) => expr,
ExprKind::Type(expr, _ty) => expr,
ExprKind::Index(base, _subscript) => base,
_ => break false,
};
if !classify::expr_requires_semi_to_be_stmt(innermost) {
break true;
}
}
};