4 files changed, 64 insertions, 32 deletions

diff --git a/crates/ra_ide_api/src/completion/complete_path.rs b/crates/ra_ide_api/src/completion/complete_path.rs
index 23dece73c..a58fdc036 100644
--- a/crates/ra_ide_api/src/completion/complete_path.rs
+++ b/crates/ra_ide_api/src/completion/complete_path.rs
@@ -50,7 +50,7 @@ pub(super) fn complete_path(acc: &mut Completions, ctx: &CompletionContext) {
                 hir::ModuleDef::TypeAlias(a) => a.ty(ctx.db),
                 _ => unreachable!(),
             };
-            let krate = ctx.module.and_then(|m| m.krate(ctx.db));
+            let krate = ctx.module.map(|m| m.krate());
             if let Some(krate) = krate {
                 ty.iterate_impl_items(ctx.db, krate, |item| {
                     match item {
@@ -67,7 +67,7 @@ pub(super) fn complete_path(acc: &mut Completions, ctx: &CompletionContext) {
                 });
             }
         }
-        _ => return,
+        _ => {}
     };
 }
 
diff --git a/crates/ra_ide_api/src/completion/complete_postfix.rs b/crates/ra_ide_api/src/completion/complete_postfix.rs
index 555cecb73..60ed3518b 100644
--- a/crates/ra_ide_api/src/completion/complete_postfix.rs
+++ b/crates/ra_ide_api/src/completion/complete_postfix.rs
@@ -9,16 +9,14 @@ use crate::{
 };
 use hir::{Ty, TypeCtor};
 use ra_syntax::{ast::AstNode, TextRange, TextUnit};
-use ra_text_edit::TextEditBuilder;
+use ra_text_edit::TextEdit;
 
 fn postfix_snippet(ctx: &CompletionContext, label: &str, detail: &str, snippet: &str) -> Builder {
     let edit = {
         let receiver_range =
             ctx.dot_receiver.as_ref().expect("no receiver available").syntax().text_range();
         let delete_range = TextRange::from_to(receiver_range.start(), ctx.source_range().end());
-        let mut builder = TextEditBuilder::default();
-        builder.replace(delete_range, snippet.to_string());
-        builder.finish()
+        TextEdit::replace(delete_range, snippet.to_string())
     };
     CompletionItem::new(CompletionKind::Postfix, ctx.source_range(), label)
         .detail(detail)
diff --git a/crates/ra_ide_api/src/completion/completion_item.rs b/crates/ra_ide_api/src/completion/completion_item.rs
index 3e6933bc1..5c9c44704 100644
--- a/crates/ra_ide_api/src/completion/completion_item.rs
+++ b/crates/ra_ide_api/src/completion/completion_item.rs
@@ -4,7 +4,7 @@ use std::fmt;
 
 use hir::Documentation;
 use ra_syntax::TextRange;
-use ra_text_edit::{TextEdit, TextEditBuilder};
+use ra_text_edit::TextEdit;
 
 /// `CompletionItem` describes a single completion variant in the editor pop-up.
 /// It is basically a POD with various properties. To construct a
@@ -192,12 +192,10 @@ impl Builder {
         let label = self.label;
         let text_edit = match self.text_edit {
             Some(it) => it,
-            None => {
-                let mut builder = TextEditBuilder::default();
-                builder
-                    .replace(self.source_range, self.insert_text.unwrap_or_else(|| label.clone()));
-                builder.finish()
-            }
+            None => TextEdit::replace(
+                self.source_range,
+                self.insert_text.unwrap_or_else(|| label.clone()),
+            ),
         };
 
         CompletionItem {
diff --git a/crates/ra_ide_api/src/completion/presentation.rs b/crates/ra_ide_api/src/completion/presentation.rs
index aed4ce6d4..65bb639ed 100644
--- a/crates/ra_ide_api/src/completion/presentation.rs
+++ b/crates/ra_ide_api/src/completion/presentation.rs
@@ -136,7 +136,7 @@ impl Completions {
         for (idx, s) in docs.match_indices(&macro_name) {
             let (before, after) = (&docs[..idx], &docs[idx + s.len()..]);
             // Ensure to match the full word
-            if after.starts_with("!")
+            if after.starts_with('!')
                 && before
                     .chars()
                     .rev()
@@ -164,27 +164,32 @@ impl Completions {
         name: Option<String>,
         macro_: hir::MacroDef,
     ) {
+        let name = match name {
+            Some(it) => it,
+            None => return,
+        };
+
         let ast_node = macro_.source(ctx.db).ast;
-        if let Some(name) = name {
-            let detail = macro_label(&ast_node);
+        let detail = macro_label(&ast_node);
+
+        let docs = macro_.docs(ctx.db);
+        let macro_declaration = format!("{}!", name);
+
+        let mut builder =
+            CompletionItem::new(CompletionKind::Reference, ctx.source_range(), &macro_declaration)
+                .kind(CompletionItemKind::Macro)
+                .set_documentation(docs.clone())
+                .detail(detail);
 
-            let docs = macro_.docs(ctx.db);
+        builder = if ctx.use_item_syntax.is_some() {
+            builder.insert_text(name)
+        } else {
             let macro_braces_to_insert =
                 self.guess_macro_braces(&name, docs.as_ref().map_or("", |s| s.as_str()));
-            let macro_declaration = name + "!";
-
-            let builder = CompletionItem::new(
-                CompletionKind::Reference,
-                ctx.source_range(),
-                &macro_declaration,
-            )
-            .kind(CompletionItemKind::Macro)
-            .set_documentation(docs)
-            .detail(detail)
-            .insert_snippet(macro_declaration + macro_braces_to_insert);
+            builder.insert_snippet(macro_declaration + macro_braces_to_insert)
+        };
 
-            self.add(builder);
-        }
+        self.add(builder);
     }
 
     fn add_function_with_name(
@@ -220,7 +225,7 @@ impl Completions {
                 } else {
                     (format!("{}($0)", data.name()), format!("{}(…)", name))
                 };
-            builder = builder.lookup_by(name.clone()).label(label).insert_snippet(snippet);
+            builder = builder.lookup_by(name).label(label).insert_snippet(snippet);
         }
 
         self.add(builder)
@@ -281,10 +286,11 @@ fn has_non_default_type_params(def: hir::GenericDef, db: &db::RootDatabase) -> b
 
 #[cfg(test)]
 mod tests {
-    use crate::completion::{do_completion, CompletionItem, CompletionKind};
     use insta::assert_debug_snapshot;
     use test_utils::covers;
 
+    use crate::completion::{do_completion, CompletionItem, CompletionKind};
+
     fn do_reference_completion(code: &str) -> Vec<CompletionItem> {
         do_completion(code, CompletionKind::Reference)
     }
@@ -576,4 +582,34 @@ mod tests {
         "###
         );
     }
+
+    #[test]
+    fn dont_insert_macro_call_braces_in_use() {
+        assert_debug_snapshot!(
+            do_reference_completion(
+                r"
+                //- /main.rs
+                use foo::<|>;
+
+                //- /foo/lib.rs
+                #[macro_export]
+                macro_rules frobnicate {
+                    () => ()
+                }
+                "
+            ),
+            @r###"
+        [
+            CompletionItem {
+                label: "frobnicate!",
+                source_range: [9; 9),
+                delete: [9; 9),
+                insert: "frobnicate",
+                kind: Macro,
+                detail: "#[macro_export]\nmacro_rules! frobnicate",
+            },
+        ]
+        "###
+        )
+    }
 }