From 5c5fedb9454b4fcb237728da7c9e29e981fcdc3a Mon Sep 17 00:00:00 2001 From: Dawer <7803845+iDawer@users.noreply.github.com> Date: Wed, 19 May 2021 23:12:09 +0500 Subject: add_explicit_type respects `@` patterns --- .../ide_assists/src/handlers/add_explicit_type.rs | 25 +++++++++++++++++----- 1 file changed, 20 insertions(+), 5 deletions(-) (limited to 'crates/ide_assists') diff --git a/crates/ide_assists/src/handlers/add_explicit_type.rs b/crates/ide_assists/src/handlers/add_explicit_type.rs index 36589203d..b7617ca3d 100644 --- a/crates/ide_assists/src/handlers/add_explicit_type.rs +++ b/crates/ide_assists/src/handlers/add_explicit_type.rs @@ -1,6 +1,6 @@ use hir::HirDisplay; use syntax::{ - ast::{self, AstNode, LetStmt, NameOwner}, + ast::{self, AstNode, LetStmt}, TextRange, }; @@ -31,9 +31,6 @@ pub(crate) fn add_explicit_type(acc: &mut Assists, ctx: &AssistContext) -> Optio _ => return None, }; let pat_range = pat.syntax().text_range(); - // The binding must have a name - let name = pat.name()?; - let name_range = name.syntax().text_range(); // Assist should only be applicable if cursor is between 'let' and '=' let cursor_in_range = { @@ -74,7 +71,7 @@ pub(crate) fn add_explicit_type(acc: &mut Assists, ctx: &AssistContext) -> Optio builder.replace(ascribed_ty.syntax().text_range(), inferred_type); } None => { - builder.insert(name_range.end(), format!(": {}", inferred_type)); + builder.insert(pat_range.end(), format!(": {}", inferred_type)); } }, ) @@ -243,6 +240,24 @@ struct Test { k: K, t: T } fn main() { let test: Test = Test { t: 23u8, k: 33 }; } +"#, + ); + } + + #[test] + fn type_should_be_added_after_pattern() { + // LetStmt = Attr* 'let' Pat (':' Type)? '=' initializer:Expr ';' + check_assist( + add_explicit_type, + r#" +fn main() { + let $0test @ () = (); +} +"#, + r#" +fn main() { + let test @ (): () = (); +} "#, ); } -- cgit v1.2.3