Merge #831

831: tweak postfix completions r=matklad a=matklad



Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

crates/ra_ide_api/src/completion/complete_postfix.rs

@@ -15,7 +15,7 @@ use ra_syntax::{
 };
 use ra_text_edit::TextEditBuilder;
 
-fn postfix_snippet(ctx: &CompletionContext, label: &str, snippet: &str) -> Builder {
+fn postfix_snippet(ctx: &CompletionContext, label: &str, detail: &str, snippet: &str) -> Builder {
     let replace_range = ctx.source_range();
     let receiver_range = ctx.dot_receiver.expect("no receiver available").syntax().range();
     let delete_range = TextRange::from_to(receiver_range.start(), replace_range.start());
@@ -23,22 +23,33 @@ fn postfix_snippet(ctx: &CompletionContext, label: &str, snippet: &str) -> Build
     builder.delete(delete_range);
     CompletionItem::new(CompletionKind::Postfix, replace_range, label)
         .snippet(snippet)
+        .detail(detail)
         .text_edit(builder.finish())
 }
 
 pub(super) fn complete_postfix(acc: &mut Completions, ctx: &CompletionContext) {
     if let Some(dot_receiver) = ctx.dot_receiver {
         let receiver_text = dot_receiver.syntax().text().to_string();
-        postfix_snippet(ctx, "not", &format!("!{}", receiver_text)).add_to(acc);
+        postfix_snippet(ctx, "not", "!expr", &format!("!{}", receiver_text)).add_to(acc);
-        postfix_snippet(ctx, "if", &format!("if {} {{$0}}", receiver_text)).add_to(acc);
+        postfix_snippet(ctx, "ref", "&expr", &format!("&{}", receiver_text)).add_to(acc);
+        postfix_snippet(ctx, "mref", "&mut expr", &format!("&mut {}", receiver_text)).add_to(acc);
+        postfix_snippet(ctx, "if", "if expr {}", &format!("if {} {{$0}}", receiver_text))
+            .add_to(acc);
         postfix_snippet(
             ctx,
             "match",
+            "match expr {}",
             &format!("match {} {{\n${{1:_}} => {{$0\\}},\n}}", receiver_text),
         )
         .add_to(acc);
-        postfix_snippet(ctx, "while", &format!("while {} {{\n$0\n}}", receiver_text)).add_to(acc);
+        postfix_snippet(
-        postfix_snippet(ctx, "dbg", &format!("dbg!({})", receiver_text)).add_to(acc);
+            ctx,
+            "while",
+            "while expr {}",
+            &format!("while {} {{\n$0\n}}", receiver_text),
+        )
+        .add_to(acc);
+        postfix_snippet(ctx, "dbg", "dbg!(expr)", &format!("dbg!({})", receiver_text)).add_to(acc);
     }
 }
 

crates/ra_ide_api/src/completion/snapshots/completion_item__postfix_completion_works_for_trivial_path_expression.snap

@@ -1,15 +1,17 @@
 ---
-created: "2019-02-03T11:38:42.897384636+00:00"
+created: "2019-02-14T17:38:20.322102467Z"
-creator: insta@0.5.3
+creator: insta@0.6.2
-expression: kind_completions
 source: crates/ra_ide_api/src/completion/completion_item.rs
+expression: kind_completions
 ---
 [
     CompletionItem {
         completion_kind: Postfix,
         label: "dbg",
         kind: None,
-        detail: None,
+        detail: Some(
+            "dbg!(expr)"
+        ),
         documentation: None,
         lookup: None,
         insert_text: Some(
@@ -32,7 +34,9 @@ source: crates/ra_ide_api/src/completion/completion_item.rs
         completion_kind: Postfix,
         label: "if",
         kind: None,
-        detail: None,
+        detail: Some(
+            "if expr {}"
+        ),
         documentation: None,
         lookup: None,
         insert_text: Some(
@@ -55,7 +59,9 @@ source: crates/ra_ide_api/src/completion/completion_item.rs
         completion_kind: Postfix,
         label: "match",
         kind: None,
-        detail: None,
+        detail: Some(
+            "match expr {}"
+        ),
         documentation: None,
         lookup: None,
         insert_text: Some(
@@ -74,11 +80,38 @@ source: crates/ra_ide_api/src/completion/completion_item.rs
             }
         )
     },
+    CompletionItem {
+        completion_kind: Postfix,
+        label: "mref",
+        kind: None,
+        detail: Some(
+            "&mut expr"
+        ),
+        documentation: None,
+        lookup: None,
+        insert_text: Some(
+            "&mut bar"
+        ),
+        insert_text_format: Snippet,
+        source_range: [76; 76),
+        text_edit: Some(
+            TextEdit {
+                atoms: [
+                    AtomTextEdit {
+                        delete: [72; 76),
+                        insert: ""
+                    }
+                ]
+            }
+        )
+    },
     CompletionItem {
         completion_kind: Postfix,
         label: "not",
         kind: None,
-        detail: None,
+        detail: Some(
+            "!expr"
+        ),
         documentation: None,
         lookup: None,
         insert_text: Some(
@@ -97,11 +130,38 @@ source: crates/ra_ide_api/src/completion/completion_item.rs
             }
         )
     },
+    CompletionItem {
+        completion_kind: Postfix,
+        label: "ref",
+        kind: None,
+        detail: Some(
+            "&expr"
+        ),
+        documentation: None,
+        lookup: None,
+        insert_text: Some(
+            "&bar"
+        ),
+        insert_text_format: Snippet,
+        source_range: [76; 76),
+        text_edit: Some(
+            TextEdit {
+                atoms: [
+                    AtomTextEdit {
+                        delete: [72; 76),
+                        insert: ""
+                    }
+                ]
+            }
+        )
+    },
     CompletionItem {
         completion_kind: Postfix,
         label: "while",
         kind: None,
-        detail: None,
+        detail: Some(
+            "while expr {}"
+        ),
         documentation: None,
         lookup: None,
         insert_text: Some(