From 19fbf2c16b5c1f39e23c720a2655cfdb49c25135 Mon Sep 17 00:00:00 2001
From: Ekaterina Babshukova <ekaterina.babshukova@yandex.ru>
Date: Sat, 12 Oct 2019 20:40:49 +0300
Subject: remove `unreachable!()`

---
 crates/ra_ide_api/src/references/classify.rs        | 20 ++++++++++----------
 crates/ra_ide_api/src/references/name_definition.rs |  8 ++++++--
 2 files changed, 16 insertions(+), 12 deletions(-)

(limited to 'crates/ra_ide_api/src/references')

diff --git a/crates/ra_ide_api/src/references/classify.rs b/crates/ra_ide_api/src/references/classify.rs
index 2ba1cf71c..5cb194c0e 100644
--- a/crates/ra_ide_api/src/references/classify.rs
+++ b/crates/ra_ide_api/src/references/classify.rs
@@ -32,32 +32,32 @@ pub(crate) fn classify_name(
                 let ast = hir::ModuleSource::Module(it);
                 let src = hir::Source { file_id, ast };
                 let def = hir::Module::from_definition(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::StructDef(it) => {
                 let src = hir::Source { file_id, ast: it };
                 let def = hir::Struct::from_source(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::EnumDef(it) => {
                 let src = hir::Source { file_id, ast: it };
                 let def = hir::Enum::from_source(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::TraitDef(it) => {
                 let src = hir::Source { file_id, ast: it };
                 let def = hir::Trait::from_source(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::StaticDef(it) => {
                 let src = hir::Source { file_id, ast: it };
                 let def = hir::Static::from_source(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::EnumVariant(it) => {
                 let src = hir::Source { file_id, ast: it };
                 let def = hir::EnumVariant::from_source(db, src)?;
-                Some(from_module_def(db, def.into()))
+                Some(from_module_def(db, def.into(), None))
             },
             ast::FnDef(it) => {
                 let src = hir::Source { file_id, ast: it };
@@ -65,7 +65,7 @@ pub(crate) fn classify_name(
                 if parent.parent().and_then(ast::ItemList::cast).is_some() {
                     Some(from_assoc_item(db, def.into()))
                 } else {
-                    Some(from_module_def(db, def.into()))
+                    Some(from_module_def(db, def.into(), None))
                 }
             },
             ast::ConstDef(it) => {
@@ -74,7 +74,7 @@ pub(crate) fn classify_name(
                 if parent.parent().and_then(ast::ItemList::cast).is_some() {
                     Some(from_assoc_item(db, def.into()))
                 } else {
-                    Some(from_module_def(db, def.into()))
+                    Some(from_module_def(db, def.into(), None))
                 }
             },
             ast::TypeAliasDef(it) => {
@@ -83,7 +83,7 @@ pub(crate) fn classify_name(
                 if parent.parent().and_then(ast::ItemList::cast).is_some() {
                     Some(from_assoc_item(db, def.into()))
                 } else {
-                    Some(from_module_def(db, def.into()))
+                    Some(from_module_def(db, def.into(), None))
                 }
             },
             _ => None,
@@ -143,7 +143,7 @@ pub(crate) fn classify_name_ref(
     let path = name_ref.syntax().ancestors().find_map(ast::Path::cast)?;
     let resolved = analyzer.resolve_path(db, &path)?;
     match resolved {
-        Def(def) => Some(from_module_def(db, def)),
+        Def(def) => Some(from_module_def(db, def, Some(container))),
         AssocItem(item) => Some(from_assoc_item(db, item)),
         LocalBinding(Either::A(pat)) => from_pat(db, file_id, pat),
         LocalBinding(Either::B(par)) => {
diff --git a/crates/ra_ide_api/src/references/name_definition.rs b/crates/ra_ide_api/src/references/name_definition.rs
index 19702eba0..58baf3686 100644
--- a/crates/ra_ide_api/src/references/name_definition.rs
+++ b/crates/ra_ide_api/src/references/name_definition.rs
@@ -77,7 +77,11 @@ pub(super) fn from_struct_field(db: &RootDatabase, field: StructField) -> NameDe
     NameDefinition { item, container, visibility }
 }
 
-pub(super) fn from_module_def(db: &RootDatabase, def: ModuleDef) -> NameDefinition {
+pub(super) fn from_module_def(
+    db: &RootDatabase,
+    def: ModuleDef,
+    module: Option<Module>,
+) -> NameDefinition {
     let item = NameKind::Def(def);
     let (container, visibility) = match def {
         ModuleDef::Module(it) => {
@@ -98,7 +102,7 @@ pub(super) fn from_module_def(db: &RootDatabase, def: ModuleDef) -> NameDefiniti
         ModuleDef::Adt(Adt::Struct(it)) => (it.module(db), it.source(db).ast.visibility()),
         ModuleDef::Adt(Adt::Union(it)) => (it.module(db), it.source(db).ast.visibility()),
         ModuleDef::Adt(Adt::Enum(it)) => (it.module(db), it.source(db).ast.visibility()),
-        ModuleDef::BuiltinType(..) => unreachable!(),
+        ModuleDef::BuiltinType(..) => (module.unwrap(), None),
     };
     NameDefinition { item, container, visibility }
 }
-- 
cgit v1.2.3