From 0f42a71806ad62cd042dd40ab42501180fb72999 Mon Sep 17 00:00:00 2001 From: Jesse Bakker Date: Tue, 15 Dec 2020 16:25:57 +0100 Subject: Parenthesize composite if condition before inverting in invert-if assist --- crates/assists/src/utils.rs | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'crates/assists/src/utils.rs') diff --git a/crates/assists/src/utils.rs b/crates/assists/src/utils.rs index 01f5c291f..f2cacf7c8 100644 --- a/crates/assists/src/utils.rs +++ b/crates/assists/src/utils.rs @@ -212,6 +212,10 @@ fn invert_special_case(expr: &ast::Expr) -> Option { ast::Expr::BinExpr(bin) => match bin.op_kind()? { ast::BinOp::NegatedEqualityTest => bin.replace_op(T![==]).map(|it| it.into()), ast::BinOp::EqualityTest => bin.replace_op(T![!=]).map(|it| it.into()), + // Parenthesize composite boolean expressions before prefixing `!` + ast::BinOp::BooleanAnd | ast::BinOp::BooleanOr => { + Some(make::expr_prefix(T![!], make::expr_paren(expr.clone()))) + } _ => None, }, ast::Expr::MethodCallExpr(mce) => { -- cgit v1.2.3