From 1755b57e1a568176bc0bda144889460e7d603cd5 Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Sat, 8 May 2021 23:09:36 +0300 Subject: internal: pull_assignment_up uses mutable trees --- .../ide_assists/src/handlers/pull_assignment_up.rs | 137 ++++++++++----------- 1 file changed, 68 insertions(+), 69 deletions(-) (limited to 'crates/ide_assists') diff --git a/crates/ide_assists/src/handlers/pull_assignment_up.rs b/crates/ide_assists/src/handlers/pull_assignment_up.rs index 543b1dfe9..602c3813e 100644 --- a/crates/ide_assists/src/handlers/pull_assignment_up.rs +++ b/crates/ide_assists/src/handlers/pull_assignment_up.rs @@ -1,6 +1,6 @@ use syntax::{ - ast::{self, edit::AstNodeEdit, make}, - AstNode, + ast::{self, make}, + ted, AstNode, }; use crate::{ @@ -44,96 +44,95 @@ pub(crate) fn pull_assignment_up(acc: &mut Assists, ctx: &AssistContext) -> Opti return None; } - let name_expr = assign_expr.lhs()?; - - let old_stmt: ast::Expr; - let new_stmt: ast::Expr; + let mut collector = AssignmentsCollector { + sema: &ctx.sema, + common_lhs: assign_expr.lhs()?, + assignments: Vec::new(), + }; - if let Some(if_expr) = ctx.find_node_at_offset::() { - new_stmt = exprify_if(&if_expr, &ctx.sema, &name_expr)?.indent(if_expr.indent_level()); - old_stmt = if_expr.into(); + let tgt: ast::Expr = if let Some(if_expr) = ctx.find_node_at_offset::() { + collector.collect_if(&if_expr)?; + if_expr.into() } else if let Some(match_expr) = ctx.find_node_at_offset::() { - new_stmt = exprify_match(&match_expr, &ctx.sema, &name_expr)?; - old_stmt = match_expr.into() + collector.collect_match(&match_expr)?; + match_expr.into() } else { return None; }; - let expr_stmt = make::expr_stmt(new_stmt); - acc.add( AssistId("pull_assignment_up", AssistKind::RefactorExtract), "Pull assignment up", - old_stmt.syntax().text_range(), + tgt.syntax().text_range(), move |edit| { - edit.replace(old_stmt.syntax().text_range(), format!("{} = {};", name_expr, expr_stmt)); + let assignments: Vec<_> = collector + .assignments + .into_iter() + .map(|(stmt, rhs)| (edit.make_ast_mut(stmt), rhs.clone_for_update())) + .collect(); + + let tgt = edit.make_ast_mut(tgt); + + for (stmt, rhs) in assignments { + ted::replace(stmt.syntax(), rhs.syntax()); + } + let assign_expr = make::expr_assignment(collector.common_lhs, tgt.clone()); + let assign_stmt = make::expr_stmt(assign_expr); + + ted::replace(tgt.syntax(), assign_stmt.syntax().clone_for_update()); }, ) } -fn exprify_match( - match_expr: &ast::MatchExpr, - sema: &hir::Semantics, - name: &ast::Expr, -) -> Option { - let new_arm_list = match_expr - .match_arm_list()? - .arms() - .map(|arm| { - if let ast::Expr::BlockExpr(block) = arm.expr()? { - let new_block = exprify_block(&block, sema, name)?.indent(block.indent_level()); - Some(arm.replace_descendant(block, new_block)) - } else { - None - } - }) - .collect::>>()?; - let new_arm_list = match_expr - .match_arm_list()? - .replace_descendants(match_expr.match_arm_list()?.arms().zip(new_arm_list)); - Some(make::expr_match(match_expr.expr()?, new_arm_list)) +struct AssignmentsCollector<'a> { + sema: &'a hir::Semantics<'a, ide_db::RootDatabase>, + common_lhs: ast::Expr, + assignments: Vec<(ast::ExprStmt, ast::Expr)>, } -fn exprify_if( - statement: &ast::IfExpr, - sema: &hir::Semantics, - name: &ast::Expr, -) -> Option { - let then_branch = exprify_block(&statement.then_branch()?, sema, name)?; - let else_branch = match statement.else_branch()? { - ast::ElseBranch::Block(block) => ast::ElseBranch::Block(exprify_block(&block, sema, name)?), - ast::ElseBranch::IfExpr(expr) => { - cov_mark::hit!(test_pull_assignment_up_chained_if); - ast::ElseBranch::IfExpr(ast::IfExpr::cast( - exprify_if(&expr, sema, name)?.syntax().to_owned(), - )?) +impl<'a> AssignmentsCollector<'a> { + fn collect_match(&mut self, match_expr: &ast::MatchExpr) -> Option<()> { + for arm in match_expr.match_arm_list()?.arms() { + match arm.expr()? { + ast::Expr::BlockExpr(block) => self.collect_block(&block)?, + // TODO: Handle this while we are at it? + _ => return None, + } } - }; - Some(make::expr_if(statement.condition()?, then_branch, Some(else_branch))) -} -fn exprify_block( - block: &ast::BlockExpr, - sema: &hir::Semantics, - name: &ast::Expr, -) -> Option { - if block.tail_expr().is_some() { - return None; + Some(()) } + fn collect_if(&mut self, if_expr: &ast::IfExpr) -> Option<()> { + let then_branch = if_expr.then_branch()?; + self.collect_block(&then_branch)?; + + match if_expr.else_branch()? { + ast::ElseBranch::Block(block) => self.collect_block(&block), + ast::ElseBranch::IfExpr(expr) => { + cov_mark::hit!(test_pull_assignment_up_chained_if); + self.collect_if(&expr) + } + } + } + fn collect_block(&mut self, block: &ast::BlockExpr) -> Option<()> { + if block.tail_expr().is_some() { + return None; + } - let mut stmts: Vec<_> = block.statements().collect(); - let stmt = stmts.pop()?; - - if let ast::Stmt::ExprStmt(stmt) = stmt { - if let ast::Expr::BinExpr(expr) = stmt.expr()? { - if expr.op_kind()? == ast::BinOp::Assignment && is_equivalent(sema, &expr.lhs()?, name) - { - // The last statement in the block is an assignment to the name we want - return Some(make::block_expr(stmts, Some(expr.rhs()?))); + let last_stmt = block.statements().last()?; + if let ast::Stmt::ExprStmt(stmt) = last_stmt { + if let ast::Expr::BinExpr(expr) = stmt.expr()? { + if expr.op_kind()? == ast::BinOp::Assignment + && is_equivalent(self.sema, &expr.lhs()?, &self.common_lhs) + { + self.assignments.push((stmt, expr.rhs()?)); + return Some(()); + } } } + + None } - None } fn is_equivalent( -- cgit v1.2.3