Merge #2747

2747: BinOp helper to detect assignment r=matklad a=kjeremy



Co-authored-by: Jeremy Kolb <[email protected]>

1 files changed, 1 insertions, 11 deletions

diff --git a/crates/ra_assists/src/assists/flip_binexpr.rs b/crates/ra_assists/src/assists/flip_binexpr.rs
index 386045eb0..2d91b2a7e 100644
--- a/crates/ra_assists/src/assists/flip_binexpr.rs
+++ b/crates/ra_assists/src/assists/flip_binexpr.rs
@@ -56,17 +56,7 @@ enum FlipAction {
 impl From<BinOp> for FlipAction {
     fn from(op_kind: BinOp) -> Self {
         match op_kind {
-            BinOp::Assignment => FlipAction::DontFlip,
-            BinOp::AddAssign => FlipAction::DontFlip,
-            BinOp::DivAssign => FlipAction::DontFlip,
-            BinOp::MulAssign => FlipAction::DontFlip,
-            BinOp::RemAssign => FlipAction::DontFlip,
-            BinOp::ShrAssign => FlipAction::DontFlip,
-            BinOp::ShlAssign => FlipAction::DontFlip,
-            BinOp::SubAssign => FlipAction::DontFlip,
-            BinOp::BitOrAssign => FlipAction::DontFlip,
-            BinOp::BitAndAssign => FlipAction::DontFlip,
-            BinOp::BitXorAssign => FlipAction::DontFlip,
+            kind if kind.is_assignment() => FlipAction::DontFlip,
             BinOp::GreaterTest => FlipAction::FlipAndReplaceOp("<"),
             BinOp::GreaterEqualTest => FlipAction::FlipAndReplaceOp("<="),
             BinOp::LesserTest => FlipAction::FlipAndReplaceOp(">"),