move guard to arm body assist.
This commit is contained in:
parent
43524a39cd
commit
f83e452b1e
crates/ra_assists/src
@ -101,6 +101,7 @@ mod split_import;
|
||||
mod remove_dbg;
|
||||
pub mod auto_import;
|
||||
mod add_missing_impl_members;
|
||||
mod move_guard_to_arm_body;
|
||||
|
||||
fn all_assists<DB: HirDatabase>() -> &'static [fn(AssistCtx<DB>) -> Option<Assist>] {
|
||||
&[
|
||||
@ -120,6 +121,7 @@ fn all_assists<DB: HirDatabase>() -> &'static [fn(AssistCtx<DB>) -> Option<Assis
|
||||
add_missing_impl_members::add_missing_impl_members,
|
||||
add_missing_impl_members::add_missing_default_members,
|
||||
inline_local_variable::inline_local_varialbe,
|
||||
move_guard_to_arm_body::move_guard_to_arm_body,
|
||||
]
|
||||
}
|
||||
|
||||
|
99
crates/ra_assists/src/move_guard_to_arm_body.rs
Normal file
99
crates/ra_assists/src/move_guard_to_arm_body.rs
Normal file
@ -0,0 +1,99 @@
|
||||
use hir::{
|
||||
db::HirDatabase,
|
||||
};
|
||||
use ra_syntax::{
|
||||
TextUnit,
|
||||
SyntaxElement,
|
||||
ast::{MatchArm, AstNode, AstToken},
|
||||
ast,
|
||||
};
|
||||
|
||||
use crate::{AssistCtx, Assist, AssistId};
|
||||
|
||||
pub(crate) fn move_guard_to_arm_body(mut ctx: AssistCtx<impl HirDatabase>) -> Option<Assist> {
|
||||
let match_arm = ctx.node_at_offset::<MatchArm>()?;
|
||||
let guard = match_arm.guard()?;
|
||||
let space_before_guard = guard.syntax().prev_sibling_or_token();
|
||||
|
||||
let guard_conditions = guard.expr()?;
|
||||
let arm_expr = match_arm.expr()?;
|
||||
let buf = format!("if {} {{ {} }}",
|
||||
guard_conditions.syntax().text(),
|
||||
arm_expr.syntax().text());
|
||||
|
||||
ctx.add_action(AssistId("move_guard_to_arm_body"), "move guard to arm body", |edit| {
|
||||
edit.target(guard.syntax().range());
|
||||
let offseting_amount = match space_before_guard {
|
||||
Some(SyntaxElement::Token(tok)) => {
|
||||
if let Some(_) = ast::Whitespace::cast(tok) {
|
||||
let ele = space_before_guard.unwrap().range();
|
||||
edit.delete(ele);
|
||||
ele.len()
|
||||
} else {
|
||||
TextUnit::from(0)
|
||||
}
|
||||
},
|
||||
_ => TextUnit::from(0)
|
||||
};
|
||||
|
||||
edit.delete(guard.syntax().range());
|
||||
edit.replace_node_and_indent(arm_expr.syntax(), buf);
|
||||
edit.set_cursor(
|
||||
arm_expr.syntax().range().start() +
|
||||
TextUnit::from(3) -
|
||||
offseting_amount);
|
||||
});
|
||||
ctx.build()
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use super::*;
|
||||
|
||||
use crate::helpers::{ check_assist, check_assist_target };
|
||||
|
||||
#[test]
|
||||
fn move_guard_to_arm_body_target() {
|
||||
check_assist_target(
|
||||
move_guard_to_arm_body,
|
||||
r#"
|
||||
fn f() {
|
||||
let t = 'a';
|
||||
let chars = "abcd";
|
||||
match t {
|
||||
'\r' <|>if chars.clone().next() == Some('\n') => false,
|
||||
_ => true
|
||||
}
|
||||
}
|
||||
"#,
|
||||
r#"if chars.clone().next() == Some('\n')"#,
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn move_guard_to_arm_body_works() {
|
||||
check_assist(
|
||||
move_guard_to_arm_body,
|
||||
r#"
|
||||
fn f() {
|
||||
let t = 'a';
|
||||
let chars = "abcd";
|
||||
match t {
|
||||
'\r' <|>if chars.clone().next() == Some('\n') => false,
|
||||
_ => true
|
||||
}
|
||||
}
|
||||
"#,
|
||||
r#"
|
||||
fn f() {
|
||||
let t = 'a';
|
||||
let chars = "abcd";
|
||||
match t {
|
||||
'\r' => if chars.clone().next() == Some('\n') { <|>false },
|
||||
_ => true
|
||||
}
|
||||
}
|
||||
"#
|
||||
);
|
||||
}
|
||||
}
|
Loading…
x
Reference in New Issue
Block a user