Skip to content

Commit 5507091

Browse files
bors[bot]matklad
andauthored
Merge #6087
6087: Microoptimize r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
2 parents eb79c20 + 6ff7896 commit 5507091

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

crates/assists/src/handlers/invert_if.rs

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,14 @@ pub(crate) fn invert_if(acc: &mut Assists, ctx: &AssistContext) -> Option<()> {
4949
ast::ElseBranch::IfExpr(_) => return None,
5050
};
5151

52-
let cond_range = cond.syntax().text_range();
53-
let flip_cond = invert_boolean_expression(cond);
54-
let else_node = else_block.syntax();
55-
let else_range = else_node.text_range();
56-
let then_range = then_node.text_range();
5752
acc.add(AssistId("invert_if", AssistKind::RefactorRewrite), "Invert if", if_range, |edit| {
58-
edit.replace(cond_range, flip_cond.syntax().text());
53+
let flip_cond = invert_boolean_expression(cond.clone());
54+
edit.replace_ast(cond, flip_cond);
55+
56+
let else_node = else_block.syntax();
57+
let else_range = else_node.text_range();
58+
let then_range = then_node.text_range();
59+
5960
edit.replace(else_range, then_node.text());
6061
edit.replace(then_range, else_node.text());
6162
})

0 commit comments

Comments
 (0)