From 9be1ab7ff948d89334a8acbc309c8235d4ab374f Mon Sep 17 00:00:00 2001
From: Kirill Bulatov <mail4score@gmail.com>
Date: Mon, 27 Jan 2020 14:42:45 +0200
Subject: Code review fixes

---
 crates/ra_assists/src/assists/auto_import.rs | 10 +++++-----
 crates/ra_assists/src/lib.rs                 |  5 +++--
 2 files changed, 8 insertions(+), 7 deletions(-)

(limited to 'crates/ra_assists/src')

diff --git a/crates/ra_assists/src/assists/auto_import.rs b/crates/ra_assists/src/assists/auto_import.rs
index 4c3793ac7..9163cc662 100644
--- a/crates/ra_assists/src/assists/auto_import.rs
+++ b/crates/ra_assists/src/assists/auto_import.rs
@@ -1,4 +1,4 @@
-use hir::{db::HirDatabase, AsName};
+use hir::db::HirDatabase;
 use ra_syntax::{
     ast::{self, AstNode},
     SmolStr, SyntaxElement,
@@ -48,7 +48,7 @@ pub(crate) fn auto_import<F: ImportsLocator>(
         return None;
     }
 
-    let name_to_import = &find_applicable_name_ref(ctx.covering_element())?.as_name();
+    let name_to_import = &find_applicable_name_ref(ctx.covering_element())?.syntax().to_string();
     let proposed_imports = imports_locator
         .find_imports(&name_to_import.to_string())
         .into_iter()
@@ -64,7 +64,7 @@ pub(crate) fn auto_import<F: ImportsLocator>(
     ctx.add_assist_group(AssistId("auto_import"), "auto import", || {
         proposed_imports
             .into_iter()
-            .map(|import| import_to_action(import, &position, &path_to_import))
+            .map(|import| import_to_action(import, &position, &path_to_import.syntax()))
             .collect()
     })
 }
@@ -84,12 +84,12 @@ fn find_applicable_name_ref(element: SyntaxElement) -> Option<ast::NameRef> {
     }
 }
 
-fn import_to_action(import: String, position: &SyntaxNode, path: &ast::Path) -> ActionBuilder {
+fn import_to_action(import: String, position: &SyntaxNode, anchor: &SyntaxNode) -> ActionBuilder {
     let mut action_builder = ActionBuilder::default();
     action_builder.label(format!("Import `{}`", &import));
     auto_import_text_edit(
         position,
-        &path.syntax().clone(),
+        anchor,
         &[SmolStr::new(import)],
         action_builder.text_edit_builder(),
     );
diff --git a/crates/ra_assists/src/lib.rs b/crates/ra_assists/src/lib.rs
index 724bce191..625ebc4a2 100644
--- a/crates/ra_assists/src/lib.rs
+++ b/crates/ra_assists/src/lib.rs
@@ -234,6 +234,7 @@ mod helpers {
     use crate::{test_db::TestDB, Assist, AssistCtx, ImportsLocator};
     use std::sync::Arc;
 
+    // FIXME remove the `ModuleDefId` reexport from `ra_hir` when this gets removed.
     pub(crate) struct TestImportsLocator {
         db: Arc<TestDB>,
         test_file_id: FileId,
@@ -248,13 +249,13 @@ mod helpers {
     impl ImportsLocator for TestImportsLocator {
         fn find_imports(&mut self, name_to_import: &str) -> Vec<hir::ModuleDef> {
             let crate_def_map = self.db.crate_def_map(self.db.test_crate());
-            let mut findings = vec![];
+            let mut findings = Vec::new();
 
             let mut module_ids_to_process =
                 crate_def_map.modules_for_file(self.test_file_id).collect::<Vec<_>>();
 
             while !module_ids_to_process.is_empty() {
-                let mut more_ids_to_process = vec![];
+                let mut more_ids_to_process = Vec::new();
                 for local_module_id in module_ids_to_process.drain(..) {
                     for (name, namespace_data) in
                         crate_def_map[local_module_id].scope.entries_without_primitives()
-- 
cgit v1.2.3