From 83e49200d82dccda54bbf376bba5a9c75da14cab Mon Sep 17 00:00:00 2001
From: Lukas Wirth <lukastw97@gmail.com>
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/src')

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<const C: usize>() { $0 }"#,
             expect![[r#"
-                tp C
+                cp C
                 fn quux() fn quux<const C: usize>()
             "#]],
         );
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