2019-03-25 17:53:57 -05:00
|
|
|
use hir::db::HirDatabase;
|
|
|
|
use ra_syntax::ast::{AstNode, BinExpr, BinOp};
|
|
|
|
|
2019-07-04 15:05:17 -05:00
|
|
|
use crate::{Assist, AssistCtx, AssistId};
|
2019-03-25 17:53:57 -05:00
|
|
|
|
2019-10-26 11:08:13 -05:00
|
|
|
// Assist: flip_binexpr
|
|
|
|
//
|
|
|
|
// Flips operands of a binary expression.
|
|
|
|
//
|
|
|
|
// ```
|
|
|
|
// fn main() {
|
|
|
|
// let _ = 90 +<|> 2;
|
|
|
|
// }
|
|
|
|
// ```
|
|
|
|
// ->
|
|
|
|
// ```
|
|
|
|
// fn main() {
|
|
|
|
// let _ = 2 + 90;
|
|
|
|
// }
|
|
|
|
// ```
|
2019-03-25 17:53:57 -05:00
|
|
|
pub(crate) fn flip_binexpr(mut ctx: AssistCtx<impl HirDatabase>) -> Option<Assist> {
|
2019-10-27 03:48:40 -05:00
|
|
|
let expr = ctx.find_node_at_offset::<BinExpr>()?;
|
2019-07-19 03:24:41 -05:00
|
|
|
let lhs = expr.lhs()?.syntax().clone();
|
|
|
|
let rhs = expr.rhs()?.syntax().clone();
|
2019-07-20 04:58:27 -05:00
|
|
|
let op_range = expr.op_token()?.text_range();
|
2019-03-26 17:12:46 -05:00
|
|
|
// The assist should be applied only if the cursor is on the operator
|
2019-03-25 17:53:57 -05:00
|
|
|
let cursor_in_range = ctx.frange.range.is_subrange(&op_range);
|
2019-03-26 17:12:46 -05:00
|
|
|
if !cursor_in_range {
|
2019-03-25 17:53:57 -05:00
|
|
|
return None;
|
|
|
|
}
|
2019-03-26 17:12:46 -05:00
|
|
|
let action: FlipAction = expr.op_kind()?.into();
|
|
|
|
// The assist should not be applied for certain operators
|
|
|
|
if let FlipAction::DontFlip = action {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
2019-03-25 17:53:57 -05:00
|
|
|
ctx.add_action(AssistId("flip_binexpr"), "flip binary expression", |edit| {
|
|
|
|
edit.target(op_range);
|
2019-03-26 17:12:46 -05:00
|
|
|
if let FlipAction::FlipAndReplaceOp(new_op) = action {
|
2019-03-25 17:53:57 -05:00
|
|
|
edit.replace(op_range, new_op);
|
|
|
|
}
|
2019-07-20 04:58:27 -05:00
|
|
|
edit.replace(lhs.text_range(), rhs.text());
|
|
|
|
edit.replace(rhs.text_range(), lhs.text());
|
2019-03-25 17:53:57 -05:00
|
|
|
});
|
|
|
|
|
|
|
|
ctx.build()
|
|
|
|
}
|
|
|
|
|
2019-03-26 17:12:46 -05:00
|
|
|
enum FlipAction {
|
|
|
|
// Flip the expression
|
|
|
|
Flip,
|
|
|
|
// Flip the expression and replace the operator with this string
|
|
|
|
FlipAndReplaceOp(&'static str),
|
|
|
|
// Do not flip the expression
|
|
|
|
DontFlip,
|
|
|
|
}
|
|
|
|
|
|
|
|
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,
|
|
|
|
BinOp::GreaterTest => FlipAction::FlipAndReplaceOp("<"),
|
|
|
|
BinOp::GreaterEqualTest => FlipAction::FlipAndReplaceOp("<="),
|
|
|
|
BinOp::LesserTest => FlipAction::FlipAndReplaceOp(">"),
|
|
|
|
BinOp::LesserEqualTest => FlipAction::FlipAndReplaceOp(">="),
|
|
|
|
_ => FlipAction::Flip,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2019-03-25 17:53:57 -05:00
|
|
|
#[cfg(test)]
|
|
|
|
mod tests {
|
|
|
|
use super::*;
|
|
|
|
|
2019-07-04 15:05:17 -05:00
|
|
|
use crate::helpers::{check_assist, check_assist_not_applicable, check_assist_target};
|
2019-03-25 17:53:57 -05:00
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_target_is_the_op() {
|
|
|
|
check_assist_target(flip_binexpr, "fn f() { let res = 1 ==<|> 2; }", "==")
|
2019-03-25 17:53:57 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_not_applicable_for_assignment() {
|
|
|
|
check_assist_not_applicable(flip_binexpr, "fn f() { let mut _x = 1; _x +=<|> 2 }")
|
2019-03-25 17:53:57 -05:00
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_works_for_eq() {
|
2019-03-25 17:53:57 -05:00
|
|
|
check_assist(
|
|
|
|
flip_binexpr,
|
2019-03-26 17:12:46 -05:00
|
|
|
"fn f() { let res = 1 ==<|> 2; }",
|
|
|
|
"fn f() { let res = 2 ==<|> 1; }",
|
2019-03-25 17:53:57 -05:00
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_works_for_gt() {
|
2019-03-25 17:53:57 -05:00
|
|
|
check_assist(
|
|
|
|
flip_binexpr,
|
2019-03-26 17:12:46 -05:00
|
|
|
"fn f() { let res = 1 ><|> 2; }",
|
|
|
|
"fn f() { let res = 2 <<|> 1; }",
|
2019-03-25 17:53:57 -05:00
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_works_for_lteq() {
|
2019-03-25 17:53:57 -05:00
|
|
|
check_assist(
|
|
|
|
flip_binexpr,
|
|
|
|
"fn f() { let res = 1 <=<|> 2; }",
|
|
|
|
"fn f() { let res = 2 >=<|> 1; }",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_works_for_complex_expr() {
|
2019-03-25 17:53:57 -05:00
|
|
|
check_assist(
|
|
|
|
flip_binexpr,
|
|
|
|
"fn f() { let res = (1 + 1) ==<|> (2 + 2); }",
|
|
|
|
"fn f() { let res = (2 + 2) ==<|> (1 + 1); }",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
2019-03-26 17:12:46 -05:00
|
|
|
fn flip_binexpr_works_inside_match() {
|
2019-03-25 17:53:57 -05:00
|
|
|
check_assist(
|
|
|
|
flip_binexpr,
|
|
|
|
r#"
|
|
|
|
fn dyn_eq(&self, other: &dyn Diagnostic) -> bool {
|
|
|
|
match other.downcast_ref::<Self>() {
|
|
|
|
None => false,
|
|
|
|
Some(it) => it ==<|> self,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
"#,
|
|
|
|
r#"
|
|
|
|
fn dyn_eq(&self, other: &dyn Diagnostic) -> bool {
|
|
|
|
match other.downcast_ref::<Self>() {
|
|
|
|
None => false,
|
|
|
|
Some(it) => self ==<|> it,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
}
|