From 83e49200d82dccda54bbf376bba5a9c75da14cab Mon Sep 17 00:00:00 2001 From: Lukas Wirth Date: Tue, 19 Jan 2021 20:21:56 +0100 Subject: Add LifetimeParam and ConstParam to CompletionItemKind --- .../completion/src/completions/unqualified_path.rs | 2 +- crates/completion/src/item.rs | 24 +++++++++++++--------- crates/completion/src/render.rs | 6 +++++- 3 files changed, 20 insertions(+), 12 deletions(-) (limited to 'crates/completion') diff --git a/crates/completion/src/completions/unqualified_path.rs b/crates/completion/src/completions/unqualified_path.rs index 809e1645a..860c774c1 100644 --- a/crates/completion/src/completions/unqualified_path.rs +++ b/crates/completion/src/completions/unqualified_path.rs @@ -241,7 +241,7 @@ fn main() { check( r#"fn quux() { $0 }"#, expect![[r#" - tp C + cp C fn quux() fn quux() "#]], ); diff --git a/crates/completion/src/item.rs b/crates/completion/src/item.rs index 4147853e7..ecf581be2 100644 --- a/crates/completion/src/item.rs +++ b/crates/completion/src/item.rs @@ -117,24 +117,26 @@ pub enum CompletionScore { #[derive(Debug, Clone, Copy, PartialEq, Eq)] pub enum CompletionItemKind { - Snippet, - Keyword, - Module, - Function, + Attribute, + Binding, BuiltinType, - Struct, + Const, + ConstParam, Enum, EnumVariant, - Binding, Field, + Function, + Keyword, + LifetimeParam, + Macro, + Method, + Module, + Snippet, Static, - Const, + Struct, Trait, TypeAlias, - Method, TypeParam, - Macro, - Attribute, UnresolvedReference, } @@ -146,11 +148,13 @@ impl CompletionItemKind { CompletionItemKind::Binding => "bn", CompletionItemKind::BuiltinType => "bt", CompletionItemKind::Const => "ct", + CompletionItemKind::ConstParam => "cp", CompletionItemKind::Enum => "en", CompletionItemKind::EnumVariant => "ev", CompletionItemKind::Field => "fd", CompletionItemKind::Function => "fn", CompletionItemKind::Keyword => "kw", + CompletionItemKind::LifetimeParam => "lt", CompletionItemKind::Macro => "ma", CompletionItemKind::Method => "me", CompletionItemKind::Module => "md", diff --git a/crates/completion/src/render.rs b/crates/completion/src/render.rs index 4f622d28a..68556c8ce 100644 --- a/crates/completion/src/render.rs +++ b/crates/completion/src/render.rs @@ -208,7 +208,11 @@ impl<'a> Render<'a> { ScopeDef::ModuleDef(Trait(..)) => CompletionItemKind::Trait, ScopeDef::ModuleDef(TypeAlias(..)) => CompletionItemKind::TypeAlias, ScopeDef::ModuleDef(BuiltinType(..)) => CompletionItemKind::BuiltinType, - ScopeDef::GenericParam(..) => CompletionItemKind::TypeParam, + ScopeDef::GenericParam(param) => match param { + hir::GenericParam::TypeParam(_) => CompletionItemKind::TypeParam, + hir::GenericParam::LifetimeParam(_) => CompletionItemKind::LifetimeParam, + hir::GenericParam::ConstParam(_) => CompletionItemKind::ConstParam, + }, ScopeDef::Local(..) => CompletionItemKind::Binding, // (does this need its own kind?) ScopeDef::AdtSelfType(..) | ScopeDef::ImplSelfType(..) => CompletionItemKind::TypeParam, -- cgit v1.2.3 From f2cb7dbcb71d81336c95dc7ae1301ba2a79ef707 Mon Sep 17 00:00:00 2001 From: Lukas Wirth Date: Wed, 20 Jan 2021 18:38:12 +0100 Subject: Partially unify SymbolKind and CompletionItemKind --- crates/completion/src/completions/fn_param.rs | 4 +- crates/completion/src/completions/mod_.rs | 10 ++-- crates/completion/src/completions/record.rs | 8 ++- crates/completion/src/completions/trait_impl.rs | 8 +-- crates/completion/src/item.rs | 70 ++++++++++++------------- crates/completion/src/render.rs | 47 +++++++++-------- crates/completion/src/render/const_.rs | 5 +- crates/completion/src/render/enum_variant.rs | 5 +- crates/completion/src/render/function.rs | 3 +- crates/completion/src/render/macro_.rs | 5 +- crates/completion/src/render/type_alias.rs | 5 +- 11 files changed, 90 insertions(+), 80 deletions(-) (limited to 'crates/completion') diff --git a/crates/completion/src/completions/fn_param.rs b/crates/completion/src/completions/fn_param.rs index 5505c3559..38e33a93e 100644 --- a/crates/completion/src/completions/fn_param.rs +++ b/crates/completion/src/completions/fn_param.rs @@ -6,7 +6,7 @@ use syntax::{ match_ast, AstNode, }; -use crate::{CompletionContext, CompletionItem, CompletionKind, Completions}; +use crate::{CompletionContext, CompletionItem, CompletionItemKind, CompletionKind, Completions}; /// Complete repeated parameters, both name and type. For example, if all /// functions in a file have a `spam: &mut Spam` parameter, a completion with @@ -58,7 +58,7 @@ pub(crate) fn complete_fn_param(acc: &mut Completions, ctx: &CompletionContext) }) .for_each(|(label, lookup)| { CompletionItem::new(CompletionKind::Magic, ctx.source_range(), label) - .kind(crate::CompletionItemKind::Binding) + .kind(CompletionItemKind::Binding) .lookup_by(lookup) .add_to(acc) }); diff --git a/crates/completion/src/completions/mod_.rs b/crates/completion/src/completions/mod_.rs index 00e951ca9..352fc7c77 100644 --- a/crates/completion/src/completions/mod_.rs +++ b/crates/completion/src/completions/mod_.rs @@ -3,11 +3,13 @@ use std::iter; use hir::{Module, ModuleSource}; -use ide_db::base_db::{SourceDatabaseExt, VfsPath}; -use ide_db::RootDatabase; +use ide_db::{ + base_db::{SourceDatabaseExt, VfsPath}, + RootDatabase, SymbolKind, +}; use rustc_hash::FxHashSet; -use crate::{CompletionItem, CompletionItemKind}; +use crate::CompletionItem; use crate::{context::CompletionContext, item::CompletionKind, Completions}; @@ -79,7 +81,7 @@ pub(crate) fn complete_mod(acc: &mut Completions, ctx: &CompletionContext) -> Op label.push(';'); } CompletionItem::new(CompletionKind::Magic, ctx.source_range(), &label) - .kind(CompletionItemKind::Module) + .kind(SymbolKind::Module) .add_to(acc) }); diff --git a/crates/completion/src/completions/record.rs b/crates/completion/src/completions/record.rs index bb6354ded..0a7927eb8 100644 --- a/crates/completion/src/completions/record.rs +++ b/crates/completion/src/completions/record.rs @@ -1,10 +1,8 @@ //! Complete fields in record literals and patterns. -use ide_db::helpers::FamousDefs; +use ide_db::{helpers::FamousDefs, SymbolKind}; use syntax::ast::Expr; -use crate::{ - item::CompletionKind, CompletionContext, CompletionItem, CompletionItemKind, Completions, -}; +use crate::{item::CompletionKind, CompletionContext, CompletionItem, Completions}; pub(crate) fn complete_record(acc: &mut Completions, ctx: &CompletionContext) -> Option<()> { let missing_fields = match (ctx.record_pat_syntax.as_ref(), ctx.record_lit_syntax.as_ref()) { @@ -31,7 +29,7 @@ pub(crate) fn complete_record(acc: &mut Completions, ctx: &CompletionContext) -> "..Default::default()", ) .insert_text(completion_text) - .kind(CompletionItemKind::Field) + .kind(SymbolKind::Field) .build(), ); } diff --git a/crates/completion/src/completions/trait_impl.rs b/crates/completion/src/completions/trait_impl.rs index 135ae49dc..f258ad9c3 100644 --- a/crates/completion/src/completions/trait_impl.rs +++ b/crates/completion/src/completions/trait_impl.rs @@ -32,7 +32,7 @@ //! ``` use hir::{self, HasAttrs, HasSource}; -use ide_db::traits::get_missing_assoc_items; +use ide_db::{traits::get_missing_assoc_items, SymbolKind}; use syntax::{ ast::{self, edit, Impl}, display::function_declaration, @@ -152,7 +152,7 @@ fn add_function_impl( let completion_kind = if func.self_param(ctx.db).is_some() { CompletionItemKind::Method } else { - CompletionItemKind::Function + CompletionItemKind::SymbolKind(SymbolKind::Function) }; let range = TextRange::new(fn_def_node.text_range().start(), ctx.source_range().end()); @@ -188,7 +188,7 @@ fn add_type_alias_impl( CompletionItem::new(CompletionKind::Magic, ctx.source_range(), snippet.clone()) .text_edit(TextEdit::replace(range, snippet)) .lookup_by(alias_name) - .kind(CompletionItemKind::TypeAlias) + .kind(SymbolKind::TypeAlias) .set_documentation(type_alias.docs(ctx.db)) .add_to(acc); } @@ -211,7 +211,7 @@ fn add_const_impl( CompletionItem::new(CompletionKind::Magic, ctx.source_range(), snippet.clone()) .text_edit(TextEdit::replace(range, snippet)) .lookup_by(const_name) - .kind(CompletionItemKind::Const) + .kind(SymbolKind::Const) .set_documentation(const_.docs(ctx.db)) .add_to(acc); } diff --git a/crates/completion/src/item.rs b/crates/completion/src/item.rs index ecf581be2..178e05c46 100644 --- a/crates/completion/src/item.rs +++ b/crates/completion/src/item.rs @@ -3,11 +3,14 @@ use std::fmt; use hir::{Documentation, ModPath, Mutability}; -use ide_db::helpers::{ - insert_use::{self, ImportScope, MergeBehavior}, - mod_path_to_ast, SnippetCap, +use ide_db::{ + helpers::{ + insert_use::{self, ImportScope, MergeBehavior}, + mod_path_to_ast, SnippetCap, + }, + SymbolKind, }; -use stdx::assert_never; +use stdx::{assert_never, impl_from}; use syntax::{algo, TextRange}; use text_edit::TextEdit; @@ -117,53 +120,50 @@ pub enum CompletionScore { #[derive(Debug, Clone, Copy, PartialEq, Eq)] pub enum CompletionItemKind { + SymbolKind(SymbolKind), + BuiltinType, Attribute, Binding, - BuiltinType, - Const, - ConstParam, - Enum, - EnumVariant, - Field, - Function, Keyword, - LifetimeParam, - Macro, Method, - Module, Snippet, - Static, - Struct, - Trait, - TypeAlias, - TypeParam, UnresolvedReference, } +impl_from!(SymbolKind for CompletionItemKind); + impl CompletionItemKind { #[cfg(test)] pub(crate) fn tag(&self) -> &'static str { match self { + CompletionItemKind::SymbolKind(kind) => match kind { + SymbolKind::Const => "ct", + SymbolKind::ConstParam => "cp", + SymbolKind::Enum => "en", + SymbolKind::Field => "fd", + SymbolKind::Function => "fn", + SymbolKind::Impl => "im", + SymbolKind::Label => "lb", + SymbolKind::LifetimeParam => "lt", + SymbolKind::Local => "lc", + SymbolKind::Macro => "ma", + SymbolKind::Module => "md", + SymbolKind::SelfParam => "sp", + SymbolKind::Static => "sc", + SymbolKind::Struct => "st", + SymbolKind::Trait => "tt", + SymbolKind::TypeAlias => "ta", + SymbolKind::TypeParam => "tp", + SymbolKind::Union => "un", + SymbolKind::ValueParam => "vp", + SymbolKind::Variant => "ev", + }, CompletionItemKind::Attribute => "at", CompletionItemKind::Binding => "bn", CompletionItemKind::BuiltinType => "bt", - CompletionItemKind::Const => "ct", - CompletionItemKind::ConstParam => "cp", - CompletionItemKind::Enum => "en", - CompletionItemKind::EnumVariant => "ev", - CompletionItemKind::Field => "fd", - CompletionItemKind::Function => "fn", CompletionItemKind::Keyword => "kw", - CompletionItemKind::LifetimeParam => "lt", - CompletionItemKind::Macro => "ma", CompletionItemKind::Method => "me", - CompletionItemKind::Module => "md", CompletionItemKind::Snippet => "sn", - CompletionItemKind::Static => "sc", - CompletionItemKind::Struct => "st", - CompletionItemKind::Trait => "tt", - CompletionItemKind::TypeAlias => "ta", - CompletionItemKind::TypeParam => "tp", CompletionItemKind::UnresolvedReference => "??", } } @@ -386,8 +386,8 @@ impl Builder { self.insert_text_format = InsertTextFormat::Snippet; self.insert_text(snippet) } - pub(crate) fn kind(mut self, kind: CompletionItemKind) -> Builder { - self.kind = Some(kind); + pub(crate) fn kind(mut self, kind: impl Into) -> Builder { + self.kind = Some(kind.into()); self } pub(crate) fn text_edit(mut self, edit: TextEdit) -> Builder { diff --git a/crates/completion/src/render.rs b/crates/completion/src/render.rs index 68556c8ce..8f66f6e59 100644 --- a/crates/completion/src/render.rs +++ b/crates/completion/src/render.rs @@ -13,7 +13,7 @@ mod builder_ext; use hir::{ AsAssocItem, Documentation, HasAttrs, HirDisplay, ModuleDef, Mutability, ScopeDef, Type, }; -use ide_db::{helpers::SnippetCap, RootDatabase}; +use ide_db::{helpers::SnippetCap, RootDatabase, SymbolKind}; use syntax::TextRange; use test_utils::mark; @@ -146,7 +146,7 @@ impl<'a> Render<'a> { self.ctx.source_range(), name.to_string(), ) - .kind(CompletionItemKind::Field) + .kind(SymbolKind::Field) .detail(ty.display(self.ctx.db()).to_string()) .set_documentation(field.docs(self.ctx.db())) .set_deprecated(is_deprecated); @@ -160,7 +160,7 @@ impl<'a> Render<'a> { fn add_tuple_field(&mut self, field: usize, ty: &Type) -> CompletionItem { CompletionItem::new(CompletionKind::Reference, self.ctx.source_range(), field.to_string()) - .kind(CompletionItemKind::Field) + .kind(SymbolKind::Field) .detail(ty.display(self.ctx.db()).to_string()) .build() } @@ -187,7 +187,7 @@ impl<'a> Render<'a> { if self.ctx.completion.is_pat_binding_or_const | self.ctx.completion.is_irrefutable_pat_binding => { - CompletionItemKind::EnumVariant + CompletionItemKind::SymbolKind(SymbolKind::Variant) } ScopeDef::ModuleDef(Variant(var)) => { let item = render_variant(self.ctx, import_to_add, Some(local_name), *var, None); @@ -198,24 +198,29 @@ impl<'a> Render<'a> { return item; } - ScopeDef::ModuleDef(Module(..)) => CompletionItemKind::Module, - ScopeDef::ModuleDef(Adt(hir::Adt::Struct(_))) => CompletionItemKind::Struct, - // FIXME: add CompletionItemKind::Union - ScopeDef::ModuleDef(Adt(hir::Adt::Union(_))) => CompletionItemKind::Struct, - ScopeDef::ModuleDef(Adt(hir::Adt::Enum(_))) => CompletionItemKind::Enum, - ScopeDef::ModuleDef(Const(..)) => CompletionItemKind::Const, - ScopeDef::ModuleDef(Static(..)) => CompletionItemKind::Static, - ScopeDef::ModuleDef(Trait(..)) => CompletionItemKind::Trait, - ScopeDef::ModuleDef(TypeAlias(..)) => CompletionItemKind::TypeAlias, + ScopeDef::ModuleDef(Module(..)) => CompletionItemKind::SymbolKind(SymbolKind::Module), + ScopeDef::ModuleDef(Adt(adt)) => CompletionItemKind::SymbolKind(match adt { + hir::Adt::Struct(_) => SymbolKind::Struct, + // FIXME: add CompletionItemKind::Union + hir::Adt::Union(_) => SymbolKind::Struct, + hir::Adt::Enum(_) => SymbolKind::Enum, + }), + ScopeDef::ModuleDef(Const(..)) => CompletionItemKind::SymbolKind(SymbolKind::Const), + ScopeDef::ModuleDef(Static(..)) => CompletionItemKind::SymbolKind(SymbolKind::Static), + ScopeDef::ModuleDef(Trait(..)) => CompletionItemKind::SymbolKind(SymbolKind::Trait), + ScopeDef::ModuleDef(TypeAlias(..)) => { + CompletionItemKind::SymbolKind(SymbolKind::TypeAlias) + } ScopeDef::ModuleDef(BuiltinType(..)) => CompletionItemKind::BuiltinType, - ScopeDef::GenericParam(param) => match param { - hir::GenericParam::TypeParam(_) => CompletionItemKind::TypeParam, - hir::GenericParam::LifetimeParam(_) => CompletionItemKind::LifetimeParam, - hir::GenericParam::ConstParam(_) => CompletionItemKind::ConstParam, - }, - ScopeDef::Local(..) => CompletionItemKind::Binding, - // (does this need its own kind?) - ScopeDef::AdtSelfType(..) | ScopeDef::ImplSelfType(..) => CompletionItemKind::TypeParam, + ScopeDef::GenericParam(param) => CompletionItemKind::SymbolKind(match param { + hir::GenericParam::TypeParam(_) => SymbolKind::TypeParam, + hir::GenericParam::LifetimeParam(_) => SymbolKind::LifetimeParam, + hir::GenericParam::ConstParam(_) => SymbolKind::ConstParam, + }), + ScopeDef::Local(..) => CompletionItemKind::SymbolKind(SymbolKind::Local), + ScopeDef::AdtSelfType(..) | ScopeDef::ImplSelfType(..) => { + CompletionItemKind::SymbolKind(SymbolKind::SelfParam) + } ScopeDef::Unknown => { let item = CompletionItem::new( CompletionKind::Reference, diff --git a/crates/completion/src/render/const_.rs b/crates/completion/src/render/const_.rs index e46452d4e..5010b642a 100644 --- a/crates/completion/src/render/const_.rs +++ b/crates/completion/src/render/const_.rs @@ -1,13 +1,14 @@ //! Renderer for `const` fields. use hir::HasSource; +use ide_db::SymbolKind; use syntax::{ ast::{Const, NameOwner}, display::const_label, }; use crate::{ - item::{CompletionItem, CompletionItemKind, CompletionKind}, + item::{CompletionItem, CompletionKind}, render::RenderContext, }; @@ -36,7 +37,7 @@ impl<'a> ConstRender<'a> { let detail = self.detail(); let item = CompletionItem::new(CompletionKind::Reference, self.ctx.source_range(), name) - .kind(CompletionItemKind::Const) + .kind(SymbolKind::Const) .set_documentation(self.ctx.docs(self.const_)) .set_deprecated( self.ctx.is_deprecated(self.const_) diff --git a/crates/completion/src/render/enum_variant.rs b/crates/completion/src/render/enum_variant.rs index 89fb49773..adcddebd1 100644 --- a/crates/completion/src/render/enum_variant.rs +++ b/crates/completion/src/render/enum_variant.rs @@ -1,11 +1,12 @@ //! Renderer for `enum` variants. use hir::{HasAttrs, HirDisplay, ModPath, StructKind}; +use ide_db::SymbolKind; use itertools::Itertools; use test_utils::mark; use crate::{ - item::{CompletionItem, CompletionItemKind, CompletionKind, ImportEdit}, + item::{CompletionItem, CompletionKind, ImportEdit}, render::{builder_ext::Params, RenderContext}, }; @@ -60,7 +61,7 @@ impl<'a> EnumRender<'a> { self.ctx.source_range(), self.qualified_name.clone(), ) - .kind(CompletionItemKind::EnumVariant) + .kind(SymbolKind::Variant) .set_documentation(self.variant.docs(self.ctx.db())) .set_deprecated(self.ctx.is_deprecated(self.variant)) .add_import(import_to_add) diff --git a/crates/completion/src/render/function.rs b/crates/completion/src/render/function.rs index 8f4c66211..2d616b1fb 100644 --- a/crates/completion/src/render/function.rs +++ b/crates/completion/src/render/function.rs @@ -1,6 +1,7 @@ //! Renderer for function calls. use hir::{HasSource, Type}; +use ide_db::SymbolKind; use syntax::{ast::Fn, display::function_declaration}; use test_utils::mark; @@ -105,7 +106,7 @@ impl<'a> FunctionRender<'a> { if self.func.self_param(self.ctx.db()).is_some() { CompletionItemKind::Method } else { - CompletionItemKind::Function + SymbolKind::Function.into() } } } diff --git a/crates/completion/src/render/macro_.rs b/crates/completion/src/render/macro_.rs index f893e420a..a4535786f 100644 --- a/crates/completion/src/render/macro_.rs +++ b/crates/completion/src/render/macro_.rs @@ -1,11 +1,12 @@ //! Renderer for macro invocations. use hir::{Documentation, HasSource}; +use ide_db::SymbolKind; use syntax::display::macro_label; use test_utils::mark; use crate::{ - item::{CompletionItem, CompletionItemKind, CompletionKind, ImportEdit}, + item::{CompletionItem, CompletionKind, ImportEdit}, render::RenderContext, }; @@ -41,7 +42,7 @@ impl<'a> MacroRender<'a> { fn render(&self, import_to_add: Option) -> Option { let mut builder = CompletionItem::new(CompletionKind::Reference, self.ctx.source_range(), &self.label()) - .kind(CompletionItemKind::Macro) + .kind(SymbolKind::Macro) .set_documentation(self.docs.clone()) .set_deprecated(self.ctx.is_deprecated(self.macro_)) .add_import(import_to_add) diff --git a/crates/completion/src/render/type_alias.rs b/crates/completion/src/render/type_alias.rs index 29287143a..bd97c3692 100644 --- a/crates/completion/src/render/type_alias.rs +++ b/crates/completion/src/render/type_alias.rs @@ -1,13 +1,14 @@ //! Renderer for type aliases. use hir::HasSource; +use ide_db::SymbolKind; use syntax::{ ast::{NameOwner, TypeAlias}, display::type_label, }; use crate::{ - item::{CompletionItem, CompletionItemKind, CompletionKind}, + item::{CompletionItem, CompletionKind}, render::RenderContext, }; @@ -36,7 +37,7 @@ impl<'a> TypeAliasRender<'a> { let detail = self.detail(); let item = CompletionItem::new(CompletionKind::Reference, self.ctx.source_range(), name) - .kind(CompletionItemKind::TypeAlias) + .kind(SymbolKind::TypeAlias) .set_documentation(self.ctx.docs(self.type_alias)) .set_deprecated( self.ctx.is_deprecated(self.type_alias) -- cgit v1.2.3 From 08b822b25914bea7405a30383fe43c7235d14346 Mon Sep 17 00:00:00 2001 From: Lukas Wirth Date: Wed, 20 Jan 2021 18:46:14 +0100 Subject: Update completions test output --- .../completion/src/completions/unqualified_path.rs | 36 ++++++------ crates/completion/src/item.rs | 2 +- crates/completion/src/render.rs | 64 ++++++++++++++++------ 3 files changed, 66 insertions(+), 36 deletions(-) (limited to 'crates/completion') diff --git a/crates/completion/src/completions/unqualified_path.rs b/crates/completion/src/completions/unqualified_path.rs index 860c774c1..a289efc34 100644 --- a/crates/completion/src/completions/unqualified_path.rs +++ b/crates/completion/src/completions/unqualified_path.rs @@ -165,8 +165,8 @@ fn quux(x: i32) { } "#, expect![[r#" - bn y i32 - bn x i32 + lc y i32 + lc x i32 fn quux(…) fn quux(x: i32) "#]], ); @@ -187,8 +187,8 @@ fn quux() { } "#, expect![[r#" - bn b i32 - bn a + lc b i32 + lc a fn quux() fn quux() "#]], ); @@ -203,7 +203,7 @@ fn quux() { } "#, expect![[r#" - bn x + lc x fn quux() fn quux() "#]], ); @@ -263,7 +263,7 @@ fn main() { check( r#"struct S { x: $0}"#, expect![[r#" - tp Self + sp Self tp T st S<…> "#]], @@ -275,7 +275,7 @@ fn main() { check( r#"enum X { Y($0) }"#, expect![[r#" - tp Self + sp Self en X "#]], ); @@ -378,8 +378,8 @@ fn foo() { "#, // FIXME: should be only one bar here expect![[r#" - bn bar i32 - bn bar i32 + lc bar i32 + lc bar i32 fn foo() fn foo() "#]], ); @@ -390,8 +390,8 @@ fn foo() { check( r#"impl S { fn foo(&self) { $0 } }"#, expect![[r#" - bn self &{unknown} - tp Self + lc self &{unknown} + sp Self "#]], ); } @@ -575,8 +575,8 @@ fn quux(x: i32) { } "#, expect![[r#" - bn y i32 - bn x i32 + lc y i32 + lc x i32 fn quux(…) fn quux(x: i32) ma m!(…) macro_rules! m "#]], @@ -594,8 +594,8 @@ fn quux(x: i32) { } ", expect![[r#" - bn y i32 - bn x i32 + lc y i32 + lc x i32 fn quux(…) fn quux(x: i32) ma m!(…) macro_rules! m "#]], @@ -613,8 +613,8 @@ fn quux(x: i32) { } "#, expect![[r#" - bn y i32 - bn x i32 + lc y i32 + lc x i32 fn quux(…) fn quux(x: i32) ma m!(…) macro_rules! m "#]], @@ -750,7 +750,7 @@ struct MyStruct {} impl My$0 "#, expect![[r#" - tp Self + sp Self tt MyTrait st MyStruct "#]], diff --git a/crates/completion/src/item.rs b/crates/completion/src/item.rs index 178e05c46..d2e6a6aeb 100644 --- a/crates/completion/src/item.rs +++ b/crates/completion/src/item.rs @@ -121,9 +121,9 @@ pub enum CompletionScore { #[derive(Debug, Clone, Copy, PartialEq, Eq)] pub enum CompletionItemKind { SymbolKind(SymbolKind), - BuiltinType, Attribute, Binding, + BuiltinType, Keyword, Method, Snippet, diff --git a/crates/completion/src/render.rs b/crates/completion/src/render.rs index 8f66f6e59..fa594b5e5 100644 --- a/crates/completion/src/render.rs +++ b/crates/completion/src/render.rs @@ -409,7 +409,9 @@ fn main() { Foo::Fo$0 } source_range: 54..56, delete: 54..56, insert: "Foo", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), detail: "{ x: i32, y: i32 }", }, ] @@ -432,7 +434,9 @@ fn main() { Foo::Fo$0 } source_range: 46..48, delete: 46..48, insert: "Foo($0)", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), lookup: "Foo", detail: "(i32, i32)", trigger_call_info: true, @@ -457,7 +461,9 @@ fn main() { Foo::Fo$0 } source_range: 35..37, delete: 35..37, insert: "Foo", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), detail: "()", }, ] @@ -481,7 +487,9 @@ fn main() { let _: m::Spam = S$0 } source_range: 75..76, delete: 75..76, insert: "Spam::Bar($0)", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), lookup: "Spam::Bar", detail: "(i32)", trigger_call_info: true, @@ -491,14 +499,18 @@ fn main() { let _: m::Spam = S$0 } source_range: 75..76, delete: 75..76, insert: "m", - kind: Module, + kind: SymbolKind( + Module, + ), }, CompletionItem { label: "m::Spam::Foo", source_range: 75..76, delete: 75..76, insert: "m::Spam::Foo", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), lookup: "Spam::Foo", detail: "()", }, @@ -507,7 +519,9 @@ fn main() { let _: m::Spam = S$0 } source_range: 75..76, delete: 75..76, insert: "main()$0", - kind: Function, + kind: SymbolKind( + Function, + ), lookup: "main", detail: "fn main()", }, @@ -534,7 +548,9 @@ fn main() { som$0 } source_range: 127..130, delete: 127..130, insert: "main()$0", - kind: Function, + kind: SymbolKind( + Function, + ), lookup: "main", detail: "fn main()", }, @@ -543,7 +559,9 @@ fn main() { som$0 } source_range: 127..130, delete: 127..130, insert: "something_deprecated()$0", - kind: Function, + kind: SymbolKind( + Function, + ), lookup: "something_deprecated", detail: "fn something_deprecated()", deprecated: true, @@ -553,7 +571,9 @@ fn main() { som$0 } source_range: 127..130, delete: 127..130, insert: "something_else_deprecated()$0", - kind: Function, + kind: SymbolKind( + Function, + ), lookup: "something_else_deprecated", detail: "fn something_else_deprecated()", deprecated: true, @@ -574,7 +594,9 @@ fn foo() { A { the$0 } } source_range: 57..60, delete: 57..60, insert: "the_field", - kind: Field, + kind: SymbolKind( + Field, + ), detail: "u32", deprecated: true, }, @@ -614,7 +636,9 @@ impl S { source_range: 94..94, delete: 94..94, insert: "foo", - kind: Field, + kind: SymbolKind( + Field, + ), detail: "{unknown}", documentation: Documentation( "Field docs", @@ -645,7 +669,9 @@ use self::E::*; source_range: 10..12, delete: 10..12, insert: "E", - kind: Enum, + kind: SymbolKind( + Enum, + ), documentation: Documentation( "enum docs", ), @@ -655,7 +681,9 @@ use self::E::*; source_range: 10..12, delete: 10..12, insert: "V", - kind: EnumVariant, + kind: SymbolKind( + Variant, + ), detail: "()", documentation: Documentation( "variant docs", @@ -666,7 +694,9 @@ use self::E::*; source_range: 10..12, delete: 10..12, insert: "my", - kind: Module, + kind: SymbolKind( + Module, + ), documentation: Documentation( "mod docs", ), @@ -892,7 +922,7 @@ struct WorldSnapshot { _f: () }; fn go(world: &WorldSnapshot) { go(w$0) } "#, expect![[r#" - bn world [type+name] + lc world [type+name] st WorldSnapshot [] fn go(…) [] "#]], @@ -909,7 +939,7 @@ fn f(foo: &Foo) { f(foo, w$0) } expect![[r#" st Foo [] fn f(…) [] - bn foo [] + lc foo [] "#]], ); } -- cgit v1.2.3