2021-02-09 13:14:32 -06:00
|
|
|
use std::{fmt, iter};
|
|
|
|
|
|
|
|
use ast::make;
|
2021-02-03 01:57:11 -06:00
|
|
|
use either::Either;
|
|
|
|
use hir::{HirDisplay, Local};
|
2021-02-03 11:31:12 -06:00
|
|
|
use ide_db::{
|
|
|
|
defs::{Definition, NameRefClass},
|
2021-02-03 15:27:31 -06:00
|
|
|
search::{FileReference, ReferenceAccess, SearchScope},
|
2021-02-03 11:31:12 -06:00
|
|
|
};
|
|
|
|
use itertools::Itertools;
|
2021-02-03 01:57:11 -06:00
|
|
|
use stdx::format_to;
|
|
|
|
use syntax::{
|
2021-02-03 14:45:03 -06:00
|
|
|
algo::SyntaxRewriter,
|
2021-02-03 01:57:11 -06:00
|
|
|
ast::{
|
|
|
|
self,
|
|
|
|
edit::{AstNodeEdit, IndentLevel},
|
2021-02-03 14:45:03 -06:00
|
|
|
AstNode,
|
2021-02-03 01:57:11 -06:00
|
|
|
},
|
2021-02-04 20:00:53 -06:00
|
|
|
AstToken, Direction, SyntaxElement,
|
2021-02-03 01:57:11 -06:00
|
|
|
SyntaxKind::{self, BLOCK_EXPR, BREAK_EXPR, COMMENT, PATH_EXPR, RETURN_EXPR},
|
2021-02-03 15:27:31 -06:00
|
|
|
SyntaxNode, SyntaxToken, TextRange, TextSize, TokenAtOffset, T,
|
2021-02-03 01:57:11 -06:00
|
|
|
};
|
|
|
|
use test_utils::mark;
|
|
|
|
|
|
|
|
use crate::{
|
|
|
|
assist_context::{AssistContext, Assists},
|
|
|
|
AssistId,
|
|
|
|
};
|
|
|
|
|
|
|
|
// Assist: extract_function
|
|
|
|
//
|
|
|
|
// Extracts selected statements into new function.
|
|
|
|
//
|
|
|
|
// ```
|
|
|
|
// fn main() {
|
|
|
|
// let n = 1;
|
|
|
|
// $0let m = n + 2;
|
|
|
|
// let k = m + n;$0
|
|
|
|
// let g = 3;
|
|
|
|
// }
|
|
|
|
// ```
|
|
|
|
// ->
|
|
|
|
// ```
|
|
|
|
// fn main() {
|
|
|
|
// let n = 1;
|
|
|
|
// fun_name(n);
|
|
|
|
// let g = 3;
|
|
|
|
// }
|
|
|
|
//
|
|
|
|
// fn $0fun_name(n: i32) {
|
|
|
|
// let m = n + 2;
|
|
|
|
// let k = m + n;
|
|
|
|
// }
|
|
|
|
// ```
|
|
|
|
pub(crate) fn extract_function(acc: &mut Assists, ctx: &AssistContext) -> Option<()> {
|
|
|
|
if ctx.frange.range.is_empty() {
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
|
|
|
let node = ctx.covering_element();
|
|
|
|
if node.kind() == COMMENT {
|
|
|
|
mark::hit!(extract_function_in_comment_is_not_applicable);
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
let node = element_to_node(node);
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
let body = extraction_target(&node, ctx.frange.range)?;
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
let vars_used_in_body = vars_used_in_body(ctx, &body);
|
2021-02-04 15:35:28 -06:00
|
|
|
let self_param = self_param_from_usages(ctx, &body, &vars_used_in_body);
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-03 03:27:53 -06:00
|
|
|
let anchor = if self_param.is_some() { Anchor::Method } else { Anchor::Freestanding };
|
2021-02-04 15:35:28 -06:00
|
|
|
let insert_after = scope_for_fn_insertion(&body, anchor)?;
|
2021-02-03 01:57:11 -06:00
|
|
|
let module = ctx.sema.scope(&insert_after).module()?;
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
let vars_defined_in_body_and_outlive = vars_defined_in_body_and_outlive(ctx, &body);
|
|
|
|
let ret_ty = body_return_ty(ctx, &body)?;
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
// FIXME: we compute variables that outlive here just to check `never!` condition
|
|
|
|
// this requires traversing whole `body` (cheap) and finding all references (expensive)
|
|
|
|
// maybe we can move this check to `edit` closure somehow?
|
2021-02-04 15:35:28 -06:00
|
|
|
if stdx::never!(!vars_defined_in_body_and_outlive.is_empty() && !ret_ty.is_unit()) {
|
2021-02-03 11:31:12 -06:00
|
|
|
// We should not have variables that outlive body if we have expression block
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
|
2021-02-09 13:14:32 -06:00
|
|
|
let target_range = body.text_range();
|
2021-02-03 01:57:11 -06:00
|
|
|
|
|
|
|
acc.add(
|
|
|
|
AssistId("extract_function", crate::AssistKind::RefactorExtract),
|
|
|
|
"Extract into function",
|
|
|
|
target_range,
|
|
|
|
move |builder| {
|
2021-02-04 15:35:28 -06:00
|
|
|
let params = extracted_function_params(ctx, &body, &vars_used_in_body);
|
|
|
|
|
2021-02-03 11:31:12 -06:00
|
|
|
let fun = Function {
|
|
|
|
name: "fun_name".to_string(),
|
2021-02-03 14:45:03 -06:00
|
|
|
self_param: self_param.map(|(_, pat)| pat),
|
2021-02-03 11:31:12 -06:00
|
|
|
params,
|
|
|
|
ret_ty,
|
|
|
|
body,
|
2021-02-03 14:45:03 -06:00
|
|
|
vars_defined_in_body_and_outlive,
|
2021-02-03 11:31:12 -06:00
|
|
|
};
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-03 11:31:12 -06:00
|
|
|
builder.replace(target_range, format_replacement(ctx, &fun));
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 20:00:53 -06:00
|
|
|
let new_indent = IndentLevel::from_node(&insert_after);
|
|
|
|
let old_indent = fun.body.indent_level();
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 20:00:53 -06:00
|
|
|
let fn_def = format_function(ctx, module, &fun, old_indent, new_indent);
|
2021-02-03 01:57:11 -06:00
|
|
|
let insert_offset = insert_after.text_range().end();
|
|
|
|
builder.insert(insert_offset, fn_def);
|
|
|
|
},
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
#[derive(Debug)]
|
2021-02-03 01:57:11 -06:00
|
|
|
struct Function {
|
|
|
|
name: String,
|
2021-02-03 11:31:12 -06:00
|
|
|
self_param: Option<ast::SelfParam>,
|
2021-02-03 01:57:11 -06:00
|
|
|
params: Vec<Param>,
|
2021-02-04 15:35:28 -06:00
|
|
|
ret_ty: RetType,
|
2021-02-03 01:57:11 -06:00
|
|
|
body: FunctionBody,
|
2021-02-03 14:45:03 -06:00
|
|
|
vars_defined_in_body_and_outlive: Vec<Local>,
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug)]
|
|
|
|
struct Param {
|
2021-02-04 15:35:28 -06:00
|
|
|
var: Local,
|
2021-02-04 16:18:45 -06:00
|
|
|
ty: hir::Type,
|
2021-02-03 14:45:03 -06:00
|
|
|
has_usages_afterwards: bool,
|
|
|
|
has_mut_inside_body: bool,
|
|
|
|
is_copy: bool,
|
|
|
|
}
|
|
|
|
|
|
|
|
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
|
|
|
|
enum ParamKind {
|
|
|
|
Value,
|
|
|
|
MutValue,
|
|
|
|
SharedRef,
|
|
|
|
MutRef,
|
|
|
|
}
|
|
|
|
|
|
|
|
impl ParamKind {
|
|
|
|
fn is_ref(&self) -> bool {
|
|
|
|
matches!(self, ParamKind::SharedRef | ParamKind::MutRef)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
impl Param {
|
|
|
|
fn kind(&self) -> ParamKind {
|
|
|
|
match (self.has_usages_afterwards, self.has_mut_inside_body, self.is_copy) {
|
|
|
|
(true, true, _) => ParamKind::MutRef,
|
|
|
|
(true, false, false) => ParamKind::SharedRef,
|
|
|
|
(false, true, _) => ParamKind::MutValue,
|
|
|
|
(true, false, true) | (false, false, _) => ParamKind::Value,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn value_prefix(&self) -> &'static str {
|
|
|
|
match self.kind() {
|
2021-02-04 15:35:28 -06:00
|
|
|
ParamKind::Value | ParamKind::MutValue => "",
|
2021-02-03 14:45:03 -06:00
|
|
|
ParamKind::SharedRef => "&",
|
|
|
|
ParamKind::MutRef => "&mut ",
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn type_prefix(&self) -> &'static str {
|
|
|
|
match self.kind() {
|
2021-02-04 15:35:28 -06:00
|
|
|
ParamKind::Value | ParamKind::MutValue => "",
|
2021-02-03 14:45:03 -06:00
|
|
|
ParamKind::SharedRef => "&",
|
|
|
|
ParamKind::MutRef => "&mut ",
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn mut_pattern(&self) -> &'static str {
|
|
|
|
match self.kind() {
|
|
|
|
ParamKind::MutValue => "mut ",
|
|
|
|
_ => "",
|
|
|
|
}
|
|
|
|
}
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
#[derive(Debug)]
|
|
|
|
enum RetType {
|
|
|
|
Expr(hir::Type),
|
|
|
|
Stmt,
|
2021-02-03 15:44:36 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
impl RetType {
|
|
|
|
fn is_unit(&self) -> bool {
|
|
|
|
match self {
|
|
|
|
RetType::Expr(ty) => ty.is_unit(),
|
|
|
|
RetType::Stmt => true,
|
2021-02-03 14:45:03 -06:00
|
|
|
}
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
2021-02-03 14:45:03 -06:00
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
fn as_fn_ret(&self) -> Option<&hir::Type> {
|
|
|
|
match self {
|
|
|
|
RetType::Stmt => None,
|
|
|
|
RetType::Expr(ty) if ty.is_unit() => None,
|
|
|
|
RetType::Expr(ty) => Some(ty),
|
2021-02-03 14:45:03 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Semantically same as `ast::Expr`, but preserves identity when using only part of the Block
|
2021-02-03 01:57:11 -06:00
|
|
|
#[derive(Debug)]
|
|
|
|
enum FunctionBody {
|
|
|
|
Expr(ast::Expr),
|
2021-02-09 13:14:32 -06:00
|
|
|
Span { parent: ast::BlockExpr, text_range: TextRange },
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
impl FunctionBody {
|
|
|
|
fn from_whole_node(node: SyntaxNode) -> Option<Self> {
|
|
|
|
match node.kind() {
|
|
|
|
PATH_EXPR => None,
|
|
|
|
BREAK_EXPR => ast::BreakExpr::cast(node).and_then(|e| e.expr()).map(Self::Expr),
|
|
|
|
RETURN_EXPR => ast::ReturnExpr::cast(node).and_then(|e| e.expr()).map(Self::Expr),
|
|
|
|
BLOCK_EXPR => ast::BlockExpr::cast(node)
|
|
|
|
.filter(|it| it.is_standalone())
|
|
|
|
.map(Into::into)
|
|
|
|
.map(Self::Expr),
|
|
|
|
_ => ast::Expr::cast(node).map(Self::Expr),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-09 13:14:32 -06:00
|
|
|
fn from_range(node: SyntaxNode, text_range: TextRange) -> Option<FunctionBody> {
|
|
|
|
let block = ast::BlockExpr::cast(node)?;
|
|
|
|
Some(Self::Span { parent: block, text_range })
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 20:00:53 -06:00
|
|
|
fn indent_level(&self) -> IndentLevel {
|
|
|
|
match &self {
|
|
|
|
FunctionBody::Expr(expr) => IndentLevel::from_node(expr.syntax()),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, .. } => IndentLevel::from_node(parent.syntax()),
|
2021-02-04 20:00:53 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-03 01:57:11 -06:00
|
|
|
fn tail_expr(&self) -> Option<ast::Expr> {
|
|
|
|
match &self {
|
|
|
|
FunctionBody::Expr(expr) => Some(expr.clone()),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, text_range } => {
|
|
|
|
let tail_expr = parent.tail_expr()?;
|
|
|
|
if text_range.contains_range(tail_expr.syntax().text_range()) {
|
|
|
|
Some(tail_expr)
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn descendants(&self) -> impl Iterator<Item = SyntaxNode> + '_ {
|
|
|
|
match self {
|
|
|
|
FunctionBody::Expr(expr) => Either::Right(expr.syntax().descendants()),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, text_range } => Either::Left(
|
|
|
|
parent
|
|
|
|
.syntax()
|
|
|
|
.descendants()
|
|
|
|
.filter(move |it| text_range.contains_range(it.text_range())),
|
2021-02-03 01:57:11 -06:00
|
|
|
),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
fn text_range(&self) -> TextRange {
|
|
|
|
match self {
|
|
|
|
FunctionBody::Expr(expr) => expr.syntax().text_range(),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent: _, text_range } => *text_range,
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn contains_range(&self, range: TextRange) -> bool {
|
|
|
|
self.text_range().contains_range(range)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn preceedes_range(&self, range: TextRange) -> bool {
|
|
|
|
self.text_range().end() <= range.start()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn contains_node(&self, node: &SyntaxNode) -> bool {
|
|
|
|
self.contains_range(node.text_range())
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
impl HasTokenAtOffset for FunctionBody {
|
2021-02-03 15:27:31 -06:00
|
|
|
fn token_at_offset(&self, offset: TextSize) -> TokenAtOffset<SyntaxToken> {
|
|
|
|
match self {
|
|
|
|
FunctionBody::Expr(expr) => expr.syntax().token_at_offset(offset),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, text_range } => {
|
|
|
|
match parent.syntax().token_at_offset(offset) {
|
|
|
|
TokenAtOffset::None => TokenAtOffset::None,
|
|
|
|
TokenAtOffset::Single(t) => {
|
|
|
|
if text_range.contains_range(t.text_range()) {
|
|
|
|
TokenAtOffset::Single(t)
|
|
|
|
} else {
|
|
|
|
TokenAtOffset::None
|
|
|
|
}
|
|
|
|
}
|
|
|
|
TokenAtOffset::Between(a, b) => {
|
|
|
|
match (
|
|
|
|
text_range.contains_range(a.text_range()),
|
|
|
|
text_range.contains_range(b.text_range()),
|
|
|
|
) {
|
|
|
|
(true, true) => TokenAtOffset::Between(a, b),
|
|
|
|
(true, false) => TokenAtOffset::Single(a),
|
|
|
|
(false, true) => TokenAtOffset::Single(b),
|
|
|
|
(false, false) => TokenAtOffset::None,
|
|
|
|
}
|
|
|
|
}
|
2021-02-03 15:27:31 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
2021-02-03 15:27:31 -06:00
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// node or token's parent
|
2021-02-04 15:35:28 -06:00
|
|
|
fn element_to_node(node: SyntaxElement) -> SyntaxNode {
|
|
|
|
match node {
|
|
|
|
syntax::NodeOrToken::Node(n) => n,
|
|
|
|
syntax::NodeOrToken::Token(t) => t.parent(),
|
2021-02-03 11:31:12 -06:00
|
|
|
}
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
2021-02-03 11:31:12 -06:00
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Try to guess what user wants to extract
|
|
|
|
///
|
|
|
|
/// We have basically have two cases:
|
|
|
|
/// * We want whole node, like `loop {}`, `2 + 2`, `{ let n = 1; }` exprs.
|
|
|
|
/// Then we can use `ast::Expr`
|
|
|
|
/// * We want a few statements for a block. E.g.
|
|
|
|
/// ```rust,no_run
|
|
|
|
/// fn foo() -> i32 {
|
|
|
|
/// let m = 1;
|
|
|
|
/// $0
|
|
|
|
/// let n = 2;
|
|
|
|
/// let k = 3;
|
|
|
|
/// k + n
|
|
|
|
/// $0
|
|
|
|
/// }
|
|
|
|
/// ```
|
|
|
|
///
|
2021-02-04 15:35:28 -06:00
|
|
|
fn extraction_target(node: &SyntaxNode, selection_range: TextRange) -> Option<FunctionBody> {
|
2021-02-04 17:14:32 -06:00
|
|
|
// we have selected exactly the expr node
|
|
|
|
// wrap it before anything else
|
2021-02-04 15:35:28 -06:00
|
|
|
if node.text_range() == selection_range {
|
|
|
|
let body = FunctionBody::from_whole_node(node.clone());
|
|
|
|
if body.is_some() {
|
|
|
|
return body;
|
|
|
|
}
|
2021-02-03 11:31:12 -06:00
|
|
|
}
|
2021-02-03 01:57:11 -06:00
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
// we have selected a few statements in a block
|
|
|
|
// so covering_element returns the whole block
|
2021-02-04 15:35:28 -06:00
|
|
|
if node.kind() == BLOCK_EXPR {
|
2021-02-09 13:14:32 -06:00
|
|
|
let body = FunctionBody::from_range(node.clone(), selection_range);
|
2021-02-04 15:35:28 -06:00
|
|
|
if body.is_some() {
|
|
|
|
return body;
|
|
|
|
}
|
|
|
|
}
|
2021-02-04 17:14:32 -06:00
|
|
|
|
|
|
|
// we have selected single statement
|
|
|
|
// `from_whole_node` failed because (let) statement is not and expression
|
|
|
|
// so we try to expand covering_element to parent and repeat the previous
|
2021-02-04 15:35:28 -06:00
|
|
|
if let Some(parent) = node.parent() {
|
|
|
|
if parent.kind() == BLOCK_EXPR {
|
2021-02-09 13:14:32 -06:00
|
|
|
let body = FunctionBody::from_range(parent, selection_range);
|
2021-02-04 15:35:28 -06:00
|
|
|
if body.is_some() {
|
|
|
|
return body;
|
|
|
|
}
|
|
|
|
}
|
2021-02-03 11:31:12 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
// select the closest containing expr (both ifs are used)
|
|
|
|
std::iter::once(node.clone()).chain(node.ancestors()).find_map(FunctionBody::from_whole_node)
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// list local variables that are referenced in `body`
|
|
|
|
fn vars_used_in_body(ctx: &AssistContext, body: &FunctionBody) -> Vec<Local> {
|
|
|
|
// FIXME: currently usages inside macros are not found
|
2021-02-04 15:35:28 -06:00
|
|
|
body.descendants()
|
|
|
|
.filter_map(ast::NameRef::cast)
|
|
|
|
.filter_map(|name_ref| NameRefClass::classify(&ctx.sema, &name_ref))
|
|
|
|
.map(|name_kind| name_kind.referenced(ctx.db()))
|
|
|
|
.filter_map(|definition| match definition {
|
|
|
|
Definition::Local(local) => Some(local),
|
|
|
|
_ => None,
|
|
|
|
})
|
|
|
|
.unique()
|
|
|
|
.collect()
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// find `self` param, that was not defined inside `body`
|
|
|
|
///
|
|
|
|
/// It should skip `self` params from impls inside `body`
|
2021-02-04 15:35:28 -06:00
|
|
|
fn self_param_from_usages(
|
|
|
|
ctx: &AssistContext,
|
|
|
|
body: &FunctionBody,
|
|
|
|
vars_used_in_body: &[Local],
|
|
|
|
) -> Option<(Local, ast::SelfParam)> {
|
|
|
|
let mut iter = vars_used_in_body
|
|
|
|
.iter()
|
|
|
|
.filter(|var| var.is_self(ctx.db()))
|
|
|
|
.map(|var| (var, var.source(ctx.db())))
|
|
|
|
.filter(|(_, src)| is_defined_before(ctx, body, src))
|
|
|
|
.filter_map(|(&node, src)| match src.value {
|
|
|
|
Either::Right(it) => Some((node, it)),
|
|
|
|
Either::Left(_) => {
|
|
|
|
stdx::never!(false, "Local::is_self returned true, but source is IdentPat");
|
|
|
|
None
|
|
|
|
}
|
|
|
|
});
|
|
|
|
|
|
|
|
let self_param = iter.next();
|
|
|
|
stdx::always!(
|
|
|
|
iter.next().is_none(),
|
2021-02-04 17:14:32 -06:00
|
|
|
"body references two different self params, both defined outside"
|
2021-02-04 15:35:28 -06:00
|
|
|
);
|
|
|
|
|
|
|
|
self_param
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// find variables that should be extracted as params
|
|
|
|
///
|
|
|
|
/// Computes additional info that affects param type and mutability
|
2021-02-04 15:35:28 -06:00
|
|
|
fn extracted_function_params(
|
|
|
|
ctx: &AssistContext,
|
|
|
|
body: &FunctionBody,
|
|
|
|
vars_used_in_body: &[Local],
|
|
|
|
) -> Vec<Param> {
|
|
|
|
vars_used_in_body
|
|
|
|
.iter()
|
|
|
|
.filter(|var| !var.is_self(ctx.db()))
|
|
|
|
.map(|node| (node, node.source(ctx.db())))
|
|
|
|
.filter(|(_, src)| is_defined_before(ctx, body, src))
|
|
|
|
.filter_map(|(&node, src)| {
|
|
|
|
if src.value.is_left() {
|
|
|
|
Some(node)
|
|
|
|
} else {
|
|
|
|
stdx::never!(false, "Local::is_self returned false, but source is SelfParam");
|
|
|
|
None
|
|
|
|
}
|
|
|
|
})
|
|
|
|
.map(|var| {
|
|
|
|
let usages = LocalUsages::find(ctx, var);
|
2021-02-04 16:18:45 -06:00
|
|
|
let ty = var.ty(ctx.db());
|
|
|
|
let is_copy = ty.is_copy(ctx.db());
|
2021-02-04 15:35:28 -06:00
|
|
|
Param {
|
|
|
|
var,
|
2021-02-04 16:18:45 -06:00
|
|
|
ty,
|
2021-02-04 15:35:28 -06:00
|
|
|
has_usages_afterwards: has_usages_after_body(&usages, body),
|
|
|
|
has_mut_inside_body: has_exclusive_usages(ctx, &usages, body),
|
2021-02-04 16:18:45 -06:00
|
|
|
is_copy,
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
})
|
|
|
|
.collect()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn has_usages_after_body(usages: &LocalUsages, body: &FunctionBody) -> bool {
|
|
|
|
usages.iter().any(|reference| body.preceedes_range(reference.range))
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// checks if relevant var is used with `&mut` access inside body
|
2021-02-04 15:35:28 -06:00
|
|
|
fn has_exclusive_usages(ctx: &AssistContext, usages: &LocalUsages, body: &FunctionBody) -> bool {
|
|
|
|
usages
|
|
|
|
.iter()
|
|
|
|
.filter(|reference| body.contains_range(reference.range))
|
|
|
|
.any(|reference| reference_is_exclusive(reference, body, ctx))
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// checks if this reference requires `&mut` access inside body
|
2021-02-04 15:35:28 -06:00
|
|
|
fn reference_is_exclusive(
|
|
|
|
reference: &FileReference,
|
|
|
|
body: &FunctionBody,
|
|
|
|
ctx: &AssistContext,
|
|
|
|
) -> bool {
|
2021-02-04 17:14:32 -06:00
|
|
|
// we directly modify variable with set: `n = 0`, `n += 1`
|
2021-02-04 15:35:28 -06:00
|
|
|
if reference.access == Some(ReferenceAccess::Write) {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
// we take `&mut` reference to variable: `&mut v`
|
2021-02-04 19:35:41 -06:00
|
|
|
let path = match path_element_of_reference(body, reference) {
|
|
|
|
Some(path) => path,
|
|
|
|
None => return false,
|
|
|
|
};
|
|
|
|
|
|
|
|
expr_require_exclusive_access(ctx, &path).unwrap_or(false)
|
|
|
|
}
|
|
|
|
|
|
|
|
/// checks if this expr requires `&mut` access, recurses on field access
|
|
|
|
fn expr_require_exclusive_access(ctx: &AssistContext, expr: &ast::Expr) -> Option<bool> {
|
|
|
|
let parent = expr.syntax().parent()?;
|
|
|
|
|
|
|
|
if let Some(bin_expr) = ast::BinExpr::cast(parent.clone()) {
|
|
|
|
if bin_expr.op_kind()?.is_assignment() {
|
|
|
|
return Some(bin_expr.lhs()?.syntax() == expr.syntax());
|
|
|
|
}
|
|
|
|
return Some(false);
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 19:35:41 -06:00
|
|
|
if let Some(ref_expr) = ast::RefExpr::cast(parent.clone()) {
|
|
|
|
return Some(ref_expr.mut_token().is_some());
|
|
|
|
}
|
|
|
|
|
|
|
|
if let Some(method_call) = ast::MethodCallExpr::cast(parent.clone()) {
|
|
|
|
let func = ctx.sema.resolve_method_call(&method_call)?;
|
|
|
|
let self_param = func.self_param(ctx.db())?;
|
|
|
|
let access = self_param.access(ctx.db());
|
|
|
|
|
|
|
|
return Some(matches!(access, hir::Access::Exclusive));
|
|
|
|
}
|
|
|
|
|
|
|
|
if let Some(field) = ast::FieldExpr::cast(parent) {
|
|
|
|
return expr_require_exclusive_access(ctx, &field.into());
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
2021-02-04 15:35:28 -06:00
|
|
|
|
2021-02-04 19:35:41 -06:00
|
|
|
Some(false)
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Container of local varaible usages
|
|
|
|
///
|
|
|
|
/// Semanticall same as `UsageSearchResult`, but provides more convenient interface
|
2021-02-04 15:35:28 -06:00
|
|
|
struct LocalUsages(ide_db::search::UsageSearchResult);
|
|
|
|
|
|
|
|
impl LocalUsages {
|
|
|
|
fn find(ctx: &AssistContext, var: Local) -> Self {
|
|
|
|
Self(
|
|
|
|
Definition::Local(var)
|
|
|
|
.usages(&ctx.sema)
|
|
|
|
.in_scope(SearchScope::single_file(ctx.frange.file_id))
|
|
|
|
.all(),
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn iter(&self) -> impl Iterator<Item = &FileReference> + '_ {
|
|
|
|
self.0.iter().flat_map(|(_, rs)| rs.iter())
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
trait HasTokenAtOffset {
|
|
|
|
fn token_at_offset(&self, offset: TextSize) -> TokenAtOffset<SyntaxToken>;
|
|
|
|
}
|
|
|
|
|
|
|
|
impl HasTokenAtOffset for SyntaxNode {
|
|
|
|
fn token_at_offset(&self, offset: TextSize) -> TokenAtOffset<SyntaxToken> {
|
|
|
|
SyntaxNode::token_at_offset(&self, offset)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// find relevant `ast::PathExpr` for reference
|
|
|
|
///
|
|
|
|
/// # Preconditions
|
|
|
|
///
|
|
|
|
/// `node` must cover `reference`, that is `node.text_range().contains_range(reference.range)`
|
|
|
|
fn path_element_of_reference(
|
|
|
|
node: &dyn HasTokenAtOffset,
|
|
|
|
reference: &FileReference,
|
|
|
|
) -> Option<ast::Expr> {
|
2021-02-04 15:35:28 -06:00
|
|
|
let token = node.token_at_offset(reference.range.start()).right_biased().or_else(|| {
|
|
|
|
stdx::never!(false, "cannot find token at variable usage: {:?}", reference);
|
|
|
|
None
|
|
|
|
})?;
|
|
|
|
let path = token.ancestors().find_map(ast::Expr::cast).or_else(|| {
|
|
|
|
stdx::never!(false, "cannot find path parent of variable usage: {:?}", token);
|
|
|
|
None
|
|
|
|
})?;
|
|
|
|
stdx::always!(matches!(path, ast::Expr::PathExpr(_)));
|
|
|
|
Some(path)
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// list local variables defined inside `body`
|
2021-02-04 15:35:28 -06:00
|
|
|
fn vars_defined_in_body(body: &FunctionBody, ctx: &AssistContext) -> Vec<Local> {
|
2021-02-04 17:14:32 -06:00
|
|
|
// FIXME: this doesn't work well with macros
|
|
|
|
// see https://github.com/rust-analyzer/rust-analyzer/pull/7535#discussion_r570048550
|
2021-02-04 15:35:28 -06:00
|
|
|
body.descendants()
|
|
|
|
.filter_map(ast::IdentPat::cast)
|
|
|
|
.filter_map(|let_stmt| ctx.sema.to_def(&let_stmt))
|
|
|
|
.unique()
|
|
|
|
.collect()
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// list local variables defined inside `body` that should be returned from extracted function
|
2021-02-04 15:35:28 -06:00
|
|
|
fn vars_defined_in_body_and_outlive(ctx: &AssistContext, body: &FunctionBody) -> Vec<Local> {
|
|
|
|
let mut vars_defined_in_body = vars_defined_in_body(&body, ctx);
|
|
|
|
vars_defined_in_body.retain(|var| var_outlives_body(ctx, body, var));
|
|
|
|
vars_defined_in_body
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// checks if the relevant local was defined before(outside of) body
|
2021-02-04 15:35:28 -06:00
|
|
|
fn is_defined_before(
|
|
|
|
ctx: &AssistContext,
|
|
|
|
body: &FunctionBody,
|
|
|
|
src: &hir::InFile<Either<ast::IdentPat, ast::SelfParam>>,
|
|
|
|
) -> bool {
|
|
|
|
src.file_id.original_file(ctx.db()) == ctx.frange.file_id
|
|
|
|
&& !body.contains_node(&either_syntax(&src.value))
|
|
|
|
}
|
|
|
|
|
|
|
|
fn either_syntax(value: &Either<ast::IdentPat, ast::SelfParam>) -> &SyntaxNode {
|
|
|
|
match value {
|
|
|
|
Either::Left(pat) => pat.syntax(),
|
|
|
|
Either::Right(it) => it.syntax(),
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// checks if local variable is used after(outside of) body
|
2021-02-04 15:35:28 -06:00
|
|
|
fn var_outlives_body(ctx: &AssistContext, body: &FunctionBody, var: &Local) -> bool {
|
2021-02-09 13:14:32 -06:00
|
|
|
let usages = LocalUsages::find(ctx, *var);
|
|
|
|
let has_usages = usages.iter().any(|reference| body.preceedes_range(reference.range));
|
|
|
|
has_usages
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
fn body_return_ty(ctx: &AssistContext, body: &FunctionBody) -> Option<RetType> {
|
|
|
|
match body.tail_expr() {
|
|
|
|
Some(expr) => {
|
|
|
|
let ty = ctx.sema.type_of_expr(&expr)?;
|
|
|
|
Some(RetType::Expr(ty))
|
|
|
|
}
|
|
|
|
None => Some(RetType::Stmt),
|
|
|
|
}
|
|
|
|
}
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Where to put extracted function definition
|
2021-02-04 15:35:28 -06:00
|
|
|
#[derive(Debug)]
|
|
|
|
enum Anchor {
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Extract free function and put right after current top-level function
|
2021-02-04 15:35:28 -06:00
|
|
|
Freestanding,
|
2021-02-04 17:14:32 -06:00
|
|
|
/// Extract method and put right after current function in the impl-block
|
2021-02-04 15:35:28 -06:00
|
|
|
Method,
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// find where to put extracted function definition
|
|
|
|
///
|
|
|
|
/// Function should be put right after returned node
|
2021-02-04 15:35:28 -06:00
|
|
|
fn scope_for_fn_insertion(body: &FunctionBody, anchor: Anchor) -> Option<SyntaxNode> {
|
|
|
|
match body {
|
|
|
|
FunctionBody::Expr(e) => scope_for_fn_insertion_node(e.syntax(), anchor),
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, .. } => scope_for_fn_insertion_node(parent.syntax(), anchor),
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn scope_for_fn_insertion_node(node: &SyntaxNode, anchor: Anchor) -> Option<SyntaxNode> {
|
2021-02-03 01:57:11 -06:00
|
|
|
let mut ancestors = node.ancestors().peekable();
|
|
|
|
let mut last_ancestor = None;
|
|
|
|
while let Some(next_ancestor) = ancestors.next() {
|
|
|
|
match next_ancestor.kind() {
|
|
|
|
SyntaxKind::SOURCE_FILE => break,
|
|
|
|
SyntaxKind::ITEM_LIST => {
|
2021-02-03 03:27:53 -06:00
|
|
|
if !matches!(anchor, Anchor::Freestanding) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if ancestors.peek().map(SyntaxNode::kind) == Some(SyntaxKind::MODULE) {
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
SyntaxKind::ASSOC_ITEM_LIST => {
|
|
|
|
if !matches!(anchor, Anchor::Method) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if ancestors.peek().map(SyntaxNode::kind) == Some(SyntaxKind::IMPL) {
|
2021-02-03 01:57:11 -06:00
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
_ => {}
|
|
|
|
}
|
|
|
|
last_ancestor = Some(next_ancestor);
|
|
|
|
}
|
|
|
|
last_ancestor
|
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
fn format_replacement(ctx: &AssistContext, fun: &Function) -> String {
|
|
|
|
let mut buf = String::new();
|
|
|
|
|
|
|
|
match fun.vars_defined_in_body_and_outlive.as_slice() {
|
|
|
|
[] => {}
|
|
|
|
[var] => format_to!(buf, "let {} = ", var.name(ctx.db()).unwrap()),
|
|
|
|
[v0, vs @ ..] => {
|
|
|
|
buf.push_str("let (");
|
|
|
|
format_to!(buf, "{}", v0.name(ctx.db()).unwrap());
|
|
|
|
for var in vs {
|
|
|
|
format_to!(buf, ", {}", var.name(ctx.db()).unwrap());
|
|
|
|
}
|
|
|
|
buf.push_str(") = ");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
if fun.self_param.is_some() {
|
|
|
|
format_to!(buf, "self.");
|
2021-02-03 03:27:53 -06:00
|
|
|
}
|
2021-02-04 15:35:28 -06:00
|
|
|
format_to!(buf, "{}(", fun.name);
|
|
|
|
format_arg_list_to(&mut buf, fun, ctx);
|
|
|
|
format_to!(buf, ")");
|
|
|
|
|
|
|
|
if fun.ret_ty.is_unit() {
|
|
|
|
format_to!(buf, ";");
|
|
|
|
}
|
|
|
|
|
|
|
|
buf
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
fn format_arg_list_to(buf: &mut String, fun: &Function, ctx: &AssistContext) {
|
|
|
|
let mut it = fun.params.iter();
|
|
|
|
if let Some(param) = it.next() {
|
|
|
|
format_arg_to(buf, ctx, param);
|
|
|
|
}
|
|
|
|
for param in it {
|
|
|
|
buf.push_str(", ");
|
|
|
|
format_arg_to(buf, ctx, param);
|
|
|
|
}
|
2021-02-03 11:31:12 -06:00
|
|
|
}
|
|
|
|
|
2021-02-04 15:35:28 -06:00
|
|
|
fn format_arg_to(buf: &mut String, ctx: &AssistContext, param: &Param) {
|
|
|
|
format_to!(buf, "{}{}", param.value_prefix(), param.var.name(ctx.db()).unwrap());
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_function(
|
|
|
|
ctx: &AssistContext,
|
|
|
|
module: hir::Module,
|
|
|
|
fun: &Function,
|
2021-02-04 20:00:53 -06:00
|
|
|
old_indent: IndentLevel,
|
|
|
|
new_indent: IndentLevel,
|
2021-02-04 15:35:28 -06:00
|
|
|
) -> String {
|
|
|
|
let mut fn_def = String::new();
|
2021-02-04 20:00:53 -06:00
|
|
|
format_to!(fn_def, "\n\n{}fn $0{}(", new_indent, fun.name);
|
2021-02-04 15:35:28 -06:00
|
|
|
format_function_param_list_to(&mut fn_def, ctx, module, fun);
|
|
|
|
fn_def.push(')');
|
|
|
|
format_function_ret_to(&mut fn_def, ctx, module, fun);
|
2021-02-09 13:14:32 -06:00
|
|
|
fn_def.push(' ');
|
2021-02-04 20:00:53 -06:00
|
|
|
format_function_body_to(&mut fn_def, ctx, old_indent, new_indent, fun);
|
2021-02-04 15:35:28 -06:00
|
|
|
|
|
|
|
fn_def
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_function_param_list_to(
|
|
|
|
fn_def: &mut String,
|
|
|
|
ctx: &AssistContext,
|
|
|
|
module: hir::Module,
|
|
|
|
fun: &Function,
|
|
|
|
) {
|
|
|
|
let mut it = fun.params.iter();
|
|
|
|
if let Some(self_param) = &fun.self_param {
|
|
|
|
format_to!(fn_def, "{}", self_param);
|
|
|
|
} else if let Some(param) = it.next() {
|
|
|
|
format_param_to(fn_def, ctx, module, param);
|
|
|
|
}
|
|
|
|
for param in it {
|
|
|
|
fn_def.push_str(", ");
|
|
|
|
format_param_to(fn_def, ctx, module, param);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_param_to(fn_def: &mut String, ctx: &AssistContext, module: hir::Module, param: &Param) {
|
|
|
|
format_to!(
|
|
|
|
fn_def,
|
|
|
|
"{}{}: {}{}",
|
|
|
|
param.mut_pattern(),
|
|
|
|
param.var.name(ctx.db()).unwrap(),
|
|
|
|
param.type_prefix(),
|
2021-02-04 16:18:45 -06:00
|
|
|
format_type(¶m.ty, ctx, module)
|
2021-02-04 15:35:28 -06:00
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_function_ret_to(
|
|
|
|
fn_def: &mut String,
|
|
|
|
ctx: &AssistContext,
|
|
|
|
module: hir::Module,
|
|
|
|
fun: &Function,
|
|
|
|
) {
|
|
|
|
if let Some(ty) = fun.ret_ty.as_fn_ret() {
|
|
|
|
format_to!(fn_def, " -> {}", format_type(ty, ctx, module));
|
|
|
|
} else {
|
|
|
|
match fun.vars_defined_in_body_and_outlive.as_slice() {
|
|
|
|
[] => {}
|
|
|
|
[var] => {
|
|
|
|
format_to!(fn_def, " -> {}", format_type(&var.ty(ctx.db()), ctx, module));
|
|
|
|
}
|
|
|
|
[v0, vs @ ..] => {
|
|
|
|
format_to!(fn_def, " -> ({}", format_type(&v0.ty(ctx.db()), ctx, module));
|
|
|
|
for var in vs {
|
|
|
|
format_to!(fn_def, ", {}", format_type(&var.ty(ctx.db()), ctx, module));
|
|
|
|
}
|
|
|
|
fn_def.push(')');
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn format_function_body_to(
|
|
|
|
fn_def: &mut String,
|
|
|
|
ctx: &AssistContext,
|
2021-02-04 20:00:53 -06:00
|
|
|
old_indent: IndentLevel,
|
|
|
|
new_indent: IndentLevel,
|
2021-02-04 15:35:28 -06:00
|
|
|
fun: &Function,
|
|
|
|
) {
|
2021-02-09 13:14:32 -06:00
|
|
|
let block = match &fun.body {
|
2021-02-04 15:35:28 -06:00
|
|
|
FunctionBody::Expr(expr) => {
|
2021-02-09 13:14:32 -06:00
|
|
|
let expr = rewrite_body_segment(ctx, &fun.params, expr.syntax());
|
|
|
|
let expr = ast::Expr::cast(expr).unwrap();
|
|
|
|
let expr = expr.dedent(old_indent).indent(IndentLevel(1));
|
|
|
|
let block = make::block_expr(Vec::new(), Some(expr));
|
|
|
|
block.indent(new_indent)
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
2021-02-09 13:14:32 -06:00
|
|
|
FunctionBody::Span { parent, text_range } => {
|
|
|
|
let mut elements: Vec<_> = parent
|
|
|
|
.syntax()
|
|
|
|
.children()
|
|
|
|
.filter(|it| text_range.contains_range(it.text_range()))
|
|
|
|
.map(|it| rewrite_body_segment(ctx, &fun.params, &it))
|
|
|
|
.collect();
|
|
|
|
|
|
|
|
let mut tail_expr = match elements.pop() {
|
|
|
|
Some(node) => ast::Expr::cast(node.clone()).or_else(|| {
|
|
|
|
elements.push(node);
|
|
|
|
None
|
|
|
|
}),
|
|
|
|
None => None,
|
|
|
|
};
|
2021-02-04 20:00:53 -06:00
|
|
|
|
2021-02-09 13:14:32 -06:00
|
|
|
if tail_expr.is_none() {
|
|
|
|
match fun.vars_defined_in_body_and_outlive.as_slice() {
|
|
|
|
[] => {}
|
|
|
|
[var] => {
|
|
|
|
tail_expr = Some(path_expr_from_local(ctx, *var));
|
|
|
|
},
|
|
|
|
vars => {
|
|
|
|
let exprs = vars.iter()
|
|
|
|
.map(|var| path_expr_from_local(ctx, *var));
|
|
|
|
let expr = make::expr_tuple(exprs);
|
|
|
|
tail_expr = Some(expr);
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2021-02-09 13:14:32 -06:00
|
|
|
|
|
|
|
let elements = elements.into_iter().filter_map(|node| match ast::Stmt::cast(node) {
|
|
|
|
Some(stmt) => Some(stmt),
|
|
|
|
None => {
|
|
|
|
stdx::never!("block contains non-statement");
|
|
|
|
None
|
|
|
|
}
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
|
|
let block = make::block_expr(elements, tail_expr);
|
|
|
|
block.dedent(old_indent).indent(new_indent)
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
2021-02-09 13:14:32 -06:00
|
|
|
};
|
2021-02-04 15:35:28 -06:00
|
|
|
|
|
|
|
}
|
2021-02-09 13:14:32 -06:00
|
|
|
|
|
|
|
format_to!(fn_def, "{}", block);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn path_expr_from_local(ctx: &AssistContext, var: Local) -> ast::Expr {
|
|
|
|
let name = var.name(ctx.db()).unwrap().to_string();
|
|
|
|
let path = make::path_unqualified(make::path_segment(make::name_ref(&name)));
|
|
|
|
make::expr_path(path)
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
fn format_type(ty: &hir::Type, ctx: &AssistContext, module: hir::Module) -> String {
|
|
|
|
ty.display_source_code(ctx.db(), module.into()).ok().unwrap_or_else(|| "()".to_string())
|
|
|
|
}
|
|
|
|
|
2021-02-09 13:14:32 -06:00
|
|
|
fn rewrite_body_segment(
|
|
|
|
ctx: &AssistContext,
|
|
|
|
params: &[Param],
|
|
|
|
syntax: &SyntaxNode,
|
|
|
|
) -> SyntaxNode {
|
|
|
|
fix_param_usages(ctx, params, syntax)
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
/// change all usages to account for added `&`/`&mut` for some params
|
2021-02-04 15:35:28 -06:00
|
|
|
fn fix_param_usages(ctx: &AssistContext, params: &[Param], syntax: &SyntaxNode) -> SyntaxNode {
|
|
|
|
let mut rewriter = SyntaxRewriter::default();
|
|
|
|
for param in params {
|
|
|
|
if !param.kind().is_ref() {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
let usages = LocalUsages::find(ctx, param.var);
|
|
|
|
let usages = usages
|
|
|
|
.iter()
|
|
|
|
.filter(|reference| syntax.text_range().contains_range(reference.range))
|
2021-02-04 17:14:32 -06:00
|
|
|
.filter_map(|reference| path_element_of_reference(syntax, reference));
|
2021-02-04 15:35:28 -06:00
|
|
|
for path in usages {
|
|
|
|
match path.syntax().ancestors().skip(1).find_map(ast::Expr::cast) {
|
2021-02-04 19:35:41 -06:00
|
|
|
Some(ast::Expr::MethodCallExpr(_)) | Some(ast::Expr::FieldExpr(_)) => {
|
2021-02-04 15:35:28 -06:00
|
|
|
// do nothing
|
|
|
|
}
|
|
|
|
Some(ast::Expr::RefExpr(node))
|
|
|
|
if param.kind() == ParamKind::MutRef && node.mut_token().is_some() =>
|
|
|
|
{
|
|
|
|
rewriter.replace_ast(&node.clone().into(), &node.expr().unwrap());
|
|
|
|
}
|
|
|
|
Some(ast::Expr::RefExpr(node))
|
|
|
|
if param.kind() == ParamKind::SharedRef && node.mut_token().is_none() =>
|
|
|
|
{
|
|
|
|
rewriter.replace_ast(&node.clone().into(), &node.expr().unwrap());
|
|
|
|
}
|
|
|
|
Some(_) | None => {
|
2021-02-09 13:14:32 -06:00
|
|
|
rewriter.replace_ast(&path, &make::expr_prefix(T![*], path.clone()));
|
2021-02-04 15:35:28 -06:00
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
rewriter.rewrite(syntax)
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|
|
|
|
|
|
|
|
#[cfg(test)]
|
|
|
|
mod tests {
|
|
|
|
use crate::tests::{check_assist, check_assist_not_applicable};
|
|
|
|
|
|
|
|
use super::*;
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_binary_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
foo($01 + 1$0);
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
foo(fun_name());
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
1 + 1
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
2021-02-03 03:27:53 -06:00
|
|
|
|
2021-02-03 01:57:11 -06:00
|
|
|
#[test]
|
|
|
|
fn no_args_from_binary_expr_in_module() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
mod bar {
|
|
|
|
fn foo() {
|
|
|
|
foo($01 + 1$0);
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
mod bar {
|
|
|
|
fn foo() {
|
|
|
|
foo(fun_name());
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
1 + 1
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_binary_expr_indented() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0{ 1 + 1 }$0;
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
{ 1 + 1 }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_stmt_with_last_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
let k = 1;
|
|
|
|
$0let m = 1;
|
|
|
|
m + 1$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
let k = 1;
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
let m = 1;
|
|
|
|
m + 1
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_stmt_unit() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let k = 3;
|
|
|
|
$0let m = 1;
|
|
|
|
let n = m + 1;$0
|
|
|
|
let g = 5;
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let k = 3;
|
|
|
|
fun_name();
|
|
|
|
let g = 5;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
let m = 1;
|
|
|
|
let n = m + 1;
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:30:34 -06:00
|
|
|
#[test]
|
|
|
|
fn no_args_if() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0if true { }$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
if true { }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_if_else() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
$0if true { 1 } else { 2 }$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
if true { 1 } else { 2 }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_if_let_else() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
$0if let true = false { 1 } else { 2 }$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
if let true = false { 1 } else { 2 }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_match() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
$0match true {
|
|
|
|
true => 1,
|
|
|
|
false => 2,
|
|
|
|
}$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() -> i32 {
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
match true {
|
|
|
|
true => 1,
|
|
|
|
false => 2,
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_while() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0while true { }$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
while true { }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_for() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0for v in &[0, 1] { }$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
for v in &[0, 1] { }
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-03 01:57:11 -06:00
|
|
|
#[test]
|
|
|
|
fn no_args_from_loop_unit() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0loop {
|
|
|
|
let m = 1;
|
|
|
|
}$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> ! {
|
|
|
|
loop {
|
|
|
|
let m = 1;
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_loop_with_return() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let v = $0loop {
|
|
|
|
let m = 1;
|
|
|
|
break m;
|
|
|
|
}$0;
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let v = fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
loop {
|
|
|
|
let m = 1;
|
|
|
|
break m;
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn no_args_from_match() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let v: i32 = $0match Some(1) {
|
|
|
|
Some(x) => x,
|
|
|
|
None => 0,
|
|
|
|
}$0;
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
let v: i32 = fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
match Some(1) {
|
|
|
|
Some(x) => x,
|
|
|
|
None => 0,
|
|
|
|
}
|
|
|
|
}"#,
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn argument_form_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
$0n+2$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
fun_name(n)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: u32) -> u32 {
|
|
|
|
n+2
|
|
|
|
}",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn argument_used_twice_form_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
$0n+n$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
fun_name(n)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: u32) -> u32 {
|
|
|
|
n+n
|
|
|
|
}",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn two_arguments_form_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
let m = 3;
|
|
|
|
$0n+n*m$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
let m = 3;
|
|
|
|
fun_name(n, m)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: u32, m: u32) -> u32 {
|
|
|
|
n+n*m
|
|
|
|
}",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn argument_and_locals() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
$0let m = 1;
|
|
|
|
n + m$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
let n = 2;
|
|
|
|
fun_name(n)
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: u32) -> u32 {
|
|
|
|
let m = 1;
|
|
|
|
n + m
|
|
|
|
}",
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn in_comment_is_not_applicable() {
|
|
|
|
mark::check!(extract_function_in_comment_is_not_applicable);
|
|
|
|
check_assist_not_applicable(extract_function, r"fn main() { 1 + /* $0comment$0 */ 1; }");
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn part_of_expr_stmt() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
"
|
|
|
|
fn foo() {
|
|
|
|
$01$0 + 1;
|
|
|
|
}",
|
|
|
|
"
|
|
|
|
fn foo() {
|
|
|
|
fun_name() + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
1
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn function_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
$0bar(1 + 1)$0
|
|
|
|
}"#,
|
|
|
|
r#"
|
|
|
|
fn foo() {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
bar(1 + 1)
|
|
|
|
}"#,
|
|
|
|
)
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn extract_from_nested() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let x = true;
|
|
|
|
let tuple = match x {
|
|
|
|
true => ($02 + 2$0, true)
|
|
|
|
_ => (0, false)
|
|
|
|
};
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let x = true;
|
|
|
|
let tuple = match x {
|
|
|
|
true => (fun_name(), true)
|
|
|
|
_ => (0, false)
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
2 + 2
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn param_from_closure() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let lambda = |x: u32| $0x * 2$0;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let lambda = |x: u32| fun_name(x);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(x: u32) -> u32 {
|
|
|
|
x * 2
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn extract_return_stmt() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
$0return 2 + 2$0;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
return fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> u32 {
|
|
|
|
2 + 2
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn does_not_add_extra_whitespace() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
|
|
|
|
|
|
|
|
$0return 2 + 2$0;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() -> u32 {
|
|
|
|
|
|
|
|
|
|
|
|
return fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> u32 {
|
|
|
|
2 + 2
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn break_stmt() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let result = loop {
|
|
|
|
$0break 2 + 2$0;
|
|
|
|
};
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let result = loop {
|
|
|
|
break fun_name();
|
|
|
|
};
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
2 + 2
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn extract_cast() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let v = $00f32 as u32$0;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn main() {
|
|
|
|
let v = fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> u32 {
|
|
|
|
0f32 as u32
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn return_not_applicable() {
|
|
|
|
check_assist_not_applicable(extract_function, r"fn foo() { $0return$0; } ");
|
|
|
|
}
|
2021-02-03 03:27:53 -06:00
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn method_to_freestanding() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct S;
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&self) -> i32 {
|
|
|
|
$01+1$0
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct S;
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&self) -> i32 {
|
|
|
|
fun_name()
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
1+1
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn method_with_reference() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct S { f: i32 };
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&self) -> i32 {
|
|
|
|
$01+self.f$0
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct S { f: i32 };
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&self) -> i32 {
|
|
|
|
self.fun_name()
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(&self) -> i32 {
|
|
|
|
1+self.f
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn method_with_mut() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct S { f: i32 };
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&mut self) {
|
|
|
|
$0self.f += 1;$0
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct S { f: i32 };
|
|
|
|
|
|
|
|
impl S {
|
|
|
|
fn foo(&mut self) {
|
|
|
|
self.fun_name();
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(&mut self) {
|
|
|
|
self.f += 1;
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-03 11:31:12 -06:00
|
|
|
#[test]
|
|
|
|
fn variable_defined_inside_and_used_after_no_ret() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let n = 1;
|
|
|
|
$0let k = n * n;$0
|
|
|
|
let m = k + 1;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let n = 1;
|
|
|
|
let k = fun_name(n);
|
|
|
|
let m = k + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: i32) -> i32 {
|
|
|
|
let k = n * n;
|
|
|
|
k
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn two_variables_defined_inside_and_used_after_no_ret() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let n = 1;
|
|
|
|
$0let k = n * n;
|
|
|
|
let m = k + 2;$0
|
|
|
|
let h = k + m;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let n = 1;
|
|
|
|
let (k, m) = fun_name(n);
|
|
|
|
let h = k + m;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: i32) -> (i32, i32) {
|
|
|
|
let k = n * n;
|
|
|
|
let m = k + 2;
|
|
|
|
(k, m)
|
2021-02-03 14:45:03 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-04 17:14:32 -06:00
|
|
|
#[test]
|
|
|
|
fn nontrivial_patterns_define_variables() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
$0let Counter(n) = Counter(0);$0
|
|
|
|
let m = n;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
let n = fun_name();
|
|
|
|
let m = n;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> i32 {
|
|
|
|
let Counter(n) = Counter(0);
|
|
|
|
n
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn struct_with_two_fields_pattern_define_variables() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct Counter { n: i32, m: i32 };
|
|
|
|
fn foo() {
|
|
|
|
$0let Counter { n, m: k } = Counter { n: 1, m: 2 };$0
|
|
|
|
let h = n + k;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct Counter { n: i32, m: i32 };
|
|
|
|
fn foo() {
|
|
|
|
let (n, k) = fun_name();
|
|
|
|
let h = n + k;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() -> (i32, i32) {
|
|
|
|
let Counter { n, m: k } = Counter { n: 1, m: 2 };
|
|
|
|
(n, k)
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-03 14:45:03 -06:00
|
|
|
#[test]
|
|
|
|
fn mut_var_from_outer_scope() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0n += 1;$0
|
|
|
|
let m = n + 1;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(&mut n);
|
|
|
|
let m = n + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: &mut i32) {
|
|
|
|
*n += 1;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-04 19:35:41 -06:00
|
|
|
#[test]
|
|
|
|
fn mut_field_from_outer_scope() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct C { n: i32 }
|
|
|
|
fn foo() {
|
|
|
|
let mut c = C { n: 0 };
|
|
|
|
$0c.n += 1;$0
|
|
|
|
let m = c.n + 1;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct C { n: i32 }
|
|
|
|
fn foo() {
|
|
|
|
let mut c = C { n: 0 };
|
|
|
|
fun_name(&mut c);
|
|
|
|
let m = c.n + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(c: &mut C) {
|
|
|
|
c.n += 1;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_nested_field_from_outer_scope() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct P { n: i32}
|
|
|
|
struct C { p: P }
|
|
|
|
fn foo() {
|
|
|
|
let mut c = C { p: P { n: 0 } };
|
|
|
|
let mut v = C { p: P { n: 0 } };
|
|
|
|
let u = C { p: P { n: 0 } };
|
|
|
|
$0c.p.n += u.p.n;
|
|
|
|
let r = &mut v.p.n;$0
|
|
|
|
let m = c.p.n + v.p.n + u.p.n;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct P { n: i32}
|
|
|
|
struct C { p: P }
|
|
|
|
fn foo() {
|
|
|
|
let mut c = C { p: P { n: 0 } };
|
|
|
|
let mut v = C { p: P { n: 0 } };
|
|
|
|
let u = C { p: P { n: 0 } };
|
|
|
|
fun_name(&mut c, &u, &mut v);
|
|
|
|
let m = c.p.n + v.p.n + u.p.n;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(c: &mut C, u: &C, v: &mut C) {
|
|
|
|
c.p.n += u.p.n;
|
|
|
|
let r = &mut v.p.n;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
2021-02-03 14:45:03 -06:00
|
|
|
#[test]
|
|
|
|
fn mut_param_many_usages_stmt() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn bar(k: i32) {}
|
|
|
|
trait I: Copy {
|
|
|
|
fn succ(&self) -> Self;
|
|
|
|
fn inc(&mut self) -> Self { let v = self.succ(); *self = v; v }
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) -> Self { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0n += n;
|
|
|
|
bar(n);
|
|
|
|
bar(n+1);
|
|
|
|
bar(n*n);
|
|
|
|
bar(&n);
|
|
|
|
n.inc();
|
|
|
|
let v = &mut n;
|
|
|
|
*v = v.succ();
|
|
|
|
n.succ();$0
|
|
|
|
let m = n + 1;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn bar(k: i32) {}
|
|
|
|
trait I: Copy {
|
|
|
|
fn succ(&self) -> Self;
|
|
|
|
fn inc(&mut self) -> Self { let v = self.succ(); *self = v; v }
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) -> Self { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(&mut n);
|
|
|
|
let m = n + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: &mut i32) {
|
|
|
|
*n += *n;
|
|
|
|
bar(*n);
|
|
|
|
bar(*n+1);
|
|
|
|
bar(*n**n);
|
|
|
|
bar(&*n);
|
|
|
|
n.inc();
|
|
|
|
let v = n;
|
|
|
|
*v = v.succ();
|
|
|
|
n.succ();
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_param_many_usages_expr() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn bar(k: i32) {}
|
|
|
|
trait I: Copy {
|
|
|
|
fn succ(&self) -> Self;
|
|
|
|
fn inc(&mut self) -> Self { let v = self.succ(); *self = v; v }
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) -> Self { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0{
|
|
|
|
n += n;
|
|
|
|
bar(n);
|
|
|
|
bar(n+1);
|
|
|
|
bar(n*n);
|
|
|
|
bar(&n);
|
|
|
|
n.inc();
|
|
|
|
let v = &mut n;
|
|
|
|
*v = v.succ();
|
|
|
|
n.succ();
|
|
|
|
}$0
|
|
|
|
let m = n + 1;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn bar(k: i32) {}
|
|
|
|
trait I: Copy {
|
|
|
|
fn succ(&self) -> Self;
|
|
|
|
fn inc(&mut self) -> Self { let v = self.succ(); *self = v; v }
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) -> Self { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(&mut n);
|
|
|
|
let m = n + 1;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: &mut i32) {
|
|
|
|
{
|
|
|
|
*n += *n;
|
|
|
|
bar(*n);
|
|
|
|
bar(*n+1);
|
|
|
|
bar(*n**n);
|
|
|
|
bar(&*n);
|
|
|
|
n.inc();
|
|
|
|
let v = n;
|
|
|
|
*v = v.succ();
|
|
|
|
n.succ();
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_param_by_value() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0n += 1;$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(n);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(mut n: i32) {
|
|
|
|
n += 1;
|
2021-02-03 15:27:31 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_param_because_of_mut_ref() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0let v = &mut n;
|
|
|
|
*v += 1;$0
|
|
|
|
let k = n;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(&mut n);
|
|
|
|
let k = n;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: &mut i32) {
|
|
|
|
let v = n;
|
|
|
|
*v += 1;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_param_by_value_because_of_mut_ref() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0let v = &mut n;
|
|
|
|
*v += 1;$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(n);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(mut n: i32) {
|
|
|
|
let v = &mut n;
|
|
|
|
*v += 1;
|
2021-02-03 15:44:36 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_method_call() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn inc(&mut self);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn inc(&mut self) { *self += 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0n.inc();$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn inc(&mut self);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn inc(&mut self) { *self += 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(n);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(mut n: i32) {
|
|
|
|
n.inc();
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn shared_method_call() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn succ(&self);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0n.succ();$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn succ(&self);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn succ(&self) { *self + 1 }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(n);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: i32) {
|
|
|
|
n.succ();
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn mut_method_call_with_other_receiver() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn inc(&mut self, n: i32);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn inc(&mut self, n: i32) { *self += n }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
$0let mut m = 2;
|
|
|
|
m.inc(n);$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
trait I {
|
|
|
|
fn inc(&mut self, n: i32);
|
|
|
|
}
|
|
|
|
impl I for i32 {
|
|
|
|
fn inc(&mut self, n: i32) { *self += n }
|
|
|
|
}
|
|
|
|
fn foo() {
|
|
|
|
let mut n = 1;
|
|
|
|
fun_name(n);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(n: i32) {
|
|
|
|
let mut m = 2;
|
|
|
|
m.inc(n);
|
2021-02-04 16:18:45 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn non_copy_without_usages_after() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
let c = Counter(0);
|
|
|
|
$0let n = c.0;$0
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
let c = Counter(0);
|
|
|
|
fun_name(c);
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(c: Counter) {
|
|
|
|
let n = c.0;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn non_copy_used_after() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
let c = Counter(0);
|
|
|
|
$0let n = c.0;$0
|
|
|
|
let m = c.0;
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
struct Counter(i32);
|
|
|
|
fn foo() {
|
|
|
|
let c = Counter(0);
|
|
|
|
fun_name(&c);
|
|
|
|
let m = c.0;
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name(c: &Counter) {
|
2021-02-04 19:35:41 -06:00
|
|
|
let n = c.0;
|
2021-02-04 20:00:53 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn indented_stmts() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
if true {
|
|
|
|
loop {
|
|
|
|
$0let n = 1;
|
|
|
|
let m = 2;$0
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
fn foo() {
|
|
|
|
if true {
|
|
|
|
loop {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
let n = 1;
|
|
|
|
let m = 2;
|
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
|
|
|
|
|
|
|
#[test]
|
|
|
|
fn indented_stmts_inside_mod() {
|
|
|
|
check_assist(
|
|
|
|
extract_function,
|
|
|
|
r"
|
|
|
|
mod bar {
|
|
|
|
fn foo() {
|
|
|
|
if true {
|
|
|
|
loop {
|
|
|
|
$0let n = 1;
|
|
|
|
let m = 2;$0
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}",
|
|
|
|
r"
|
|
|
|
mod bar {
|
|
|
|
fn foo() {
|
|
|
|
if true {
|
|
|
|
loop {
|
|
|
|
fun_name();
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
fn $0fun_name() {
|
|
|
|
let n = 1;
|
|
|
|
let m = 2;
|
|
|
|
}
|
2021-02-03 03:27:53 -06:00
|
|
|
}",
|
|
|
|
);
|
|
|
|
}
|
2021-02-03 01:57:11 -06:00
|
|
|
}
|