From 233f01c9ba555e5d06f336cb0ff64e7a83e4a23a Mon Sep 17 00:00:00 2001 From: Aleksey Kladov Date: Wed, 6 May 2020 12:51:28 +0200 Subject: Move target to AssistLabel Target is used for assists sorting, so we need it before we compute the action. --- crates/ra_assists/src/handlers/add_function.rs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'crates/ra_assists/src/handlers/add_function.rs') diff --git a/crates/ra_assists/src/handlers/add_function.rs b/crates/ra_assists/src/handlers/add_function.rs index cb2afc863..76c0f9783 100644 --- a/crates/ra_assists/src/handlers/add_function.rs +++ b/crates/ra_assists/src/handlers/add_function.rs @@ -57,9 +57,9 @@ pub(crate) fn add_function(ctx: AssistCtx) -> Option { let function_builder = FunctionBuilder::from_call(&ctx, &call, &path, target_module)?; - ctx.add_assist(AssistId("add_function"), "Add function", |edit| { - edit.target(call.syntax().text_range()); - + let target = call.syntax().text_range(); + // TODO: assert here? + ctx.add_assist(AssistId("add_function"), "Add function", target, |edit| { if let Some(function_template) = function_builder.render() { edit.set_file(function_template.file); edit.set_cursor(function_template.cursor_offset); -- cgit v1.2.3