From 3c17643b3085682a695f0e6d80483edc00d04cb3 Mon Sep 17 00:00:00 2001 From: Jeremy Kolb Date: Mon, 28 Jan 2019 09:26:32 -0500 Subject: Go to Implementation for structs and enums --- crates/ra_hir/src/code_model_api.rs | 18 ++++++++- crates/ra_hir/src/code_model_impl/module.rs | 13 ++++++- crates/ra_hir/src/db.rs | 11 +++++- crates/ra_hir/src/impl_block.rs | 57 ++++++++++++++++++++++++++--- crates/ra_hir/src/source_binder.rs | 24 +++++++++++- crates/ra_hir/src/ty/method_resolution.rs | 6 +-- 6 files changed, 115 insertions(+), 14 deletions(-) (limited to 'crates/ra_hir/src') diff --git a/crates/ra_hir/src/code_model_api.rs b/crates/ra_hir/src/code_model_api.rs index e4008058c..691cd5798 100644 --- a/crates/ra_hir/src/code_model_api.rs +++ b/crates/ra_hir/src/code_model_api.rs @@ -10,12 +10,13 @@ use crate::{ nameres::{ModuleScope, lower::ImportId}, db::HirDatabase, expr::BodySyntaxMapping, - ty::InferenceResult, + ty::{InferenceResult}, adt::{EnumVariantId, StructFieldId, VariantDef}, generics::GenericParams, docs::{Documentation, Docs, docs_from_ast}, module_tree::ModuleId, ids::{FunctionId, StructId, EnumId, AstItemDef, ConstId, StaticId, TraitId, TypeId}, + impl_block::ImplId, }; /// hir::Crate describes a single crate. It's the main interface with which @@ -23,7 +24,7 @@ use crate::{ /// root module. #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)] pub struct Crate { - pub(crate) crate_id: CrateId, + pub crate_id: CrateId, } #[derive(Debug)] @@ -126,6 +127,11 @@ impl Module { self.import_source_impl(db, import) } + /// Returns the syntax of the impl block in this module + pub fn impl_source(&self, db: &impl HirDatabase, impl_id: ImplId) -> TreeArc { + self.impl_source_impl(db, impl_id) + } + /// Returns the crate this module is part of. pub fn krate(&self, db: &impl HirDatabase) -> Option { self.krate_impl(db) @@ -272,6 +278,10 @@ impl Struct { pub fn generic_params(&self, db: &impl HirDatabase) -> Arc { db.generic_params((*self).into()) } + + pub fn ty(&self, db: &impl HirDatabase) -> Ty { + db.type_for_def((*self).into()) + } } impl Docs for Struct { @@ -317,6 +327,10 @@ impl Enum { pub fn generic_params(&self, db: &impl HirDatabase) -> Arc { db.generic_params((*self).into()) } + + pub fn ty(&self, db: &impl HirDatabase) -> Ty { + db.type_for_def((*self).into()) + } } impl Docs for Enum { diff --git a/crates/ra_hir/src/code_model_impl/module.rs b/crates/ra_hir/src/code_model_impl/module.rs index 418d59c91..c6f85ac82 100644 --- a/crates/ra_hir/src/code_model_impl/module.rs +++ b/crates/ra_hir/src/code_model_impl/module.rs @@ -5,6 +5,7 @@ use crate::{ Module, ModuleSource, Problem, Crate, Name, module_tree::ModuleId, + impl_block::ImplId, nameres::{lower::ImportId}, db::HirDatabase, }; @@ -51,11 +52,21 @@ impl Module { db: &impl HirDatabase, import: ImportId, ) -> TreeArc { - let source_map = db.lower_module_source_map(self.clone()); + let source_map = db.lower_module_source_map(*self); let (_, source) = self.definition_source(db); source_map.get(&source, import) } + pub(crate) fn impl_source_impl( + &self, + db: &impl HirDatabase, + impl_id: ImplId, + ) -> TreeArc { + let source_map = db.impls_in_module_source_map(*self); + let (_, source) = self.definition_source(db); + source_map.get(&source, impl_id) + } + pub(crate) fn krate_impl(&self, _db: &impl HirDatabase) -> Option { Some(Crate::new(self.krate)) } diff --git a/crates/ra_hir/src/db.rs b/crates/ra_hir/src/db.rs index 3f76b769d..70d9de212 100644 --- a/crates/ra_hir/src/db.rs +++ b/crates/ra_hir/src/db.rs @@ -14,7 +14,7 @@ use crate::{ nameres::{ItemMap, lower::{LoweredModule, ImportSourceMap}}, ty::{InferenceResult, Ty, method_resolution::CrateImplBlocks, TypableDef}, adt::{StructData, EnumData}, - impl_block::ModuleImplBlocks, + impl_block::{ModuleImplBlocks, ImplSourceMap}, generics::{GenericParams, GenericDef}, ids::SourceFileItemId, }; @@ -73,9 +73,18 @@ pub trait HirDatabase: SourceDatabase + AsRef { #[salsa::invoke(crate::module_tree::ModuleTree::module_tree_query)] fn module_tree(&self, crate_id: CrateId) -> Arc; + #[salsa::invoke(crate::impl_block::impls_in_module_with_source_map_query)] + fn impls_in_module_with_source_map( + &self, + module: Module, + ) -> (Arc, Arc); + #[salsa::invoke(crate::impl_block::impls_in_module)] fn impls_in_module(&self, module: Module) -> Arc; + #[salsa::invoke(crate::impl_block::impls_in_module_source_map_query)] + fn impls_in_module_source_map(&self, module: Module) -> Arc; + #[salsa::invoke(crate::ty::method_resolution::CrateImplBlocks::impls_in_crate_query)] fn impls_in_crate(&self, krate: Crate) -> Arc; diff --git a/crates/ra_hir/src/impl_block.rs b/crates/ra_hir/src/impl_block.rs index 222e47349..5fc26324a 100644 --- a/crates/ra_hir/src/impl_block.rs +++ b/crates/ra_hir/src/impl_block.rs @@ -1,8 +1,10 @@ use std::sync::Arc; use rustc_hash::FxHashMap; -use ra_arena::{Arena, RawId, impl_arena_id}; -use ra_syntax::ast::{self, AstNode}; +use ra_arena::{Arena, RawId, impl_arena_id, map::ArenaMap}; +use ra_syntax::{ + AstPtr, SourceFile, TreeArc, +ast::{self, AstNode}}; use crate::{ Const, Type, @@ -14,6 +16,26 @@ use crate::{ use crate::code_model_api::{Module, ModuleSource}; +#[derive(Debug, Default, PartialEq, Eq)] +pub struct ImplSourceMap { + map: ArenaMap>, +} + +impl ImplSourceMap { + fn insert(&mut self, impl_id: ImplId, impl_block: &ast::ImplBlock) { + self.map.insert(impl_id, AstPtr::new(impl_block)) + } + + pub fn get(&self, source: &ModuleSource, impl_id: ImplId) -> TreeArc { + let file = match source { + ModuleSource::SourceFile(file) => &*file, + ModuleSource::Module(m) => m.syntax().ancestors().find_map(SourceFile::cast).unwrap(), + }; + + self.map[impl_id].to_node(file).to_owned() + } +} + #[derive(Debug, Clone, PartialEq, Eq)] pub struct ImplBlock { module_impl_blocks: Arc, @@ -39,6 +61,10 @@ impl ImplBlock { } } + pub fn id(&self) -> ImplId { + self.impl_id + } + fn impl_data(&self) -> &ImplData { &self.module_impl_blocks.impls[self.impl_id] } @@ -148,7 +174,7 @@ impl ModuleImplBlocks { } } - fn collect(&mut self, db: &impl HirDatabase, module: Module) { + fn collect(&mut self, db: &impl HirDatabase, module: Module, source_map: &mut ImplSourceMap) { let (file_id, module_source) = module.definition_source(db); let file_id: HirFileId = file_id.into(); let node = match &module_source { @@ -165,12 +191,31 @@ impl ModuleImplBlocks { for &impl_item in &self.impls[id].items { self.impls_by_def.insert(impl_item, id); } + + source_map.insert(id, impl_block_ast); } } } -pub(crate) fn impls_in_module(db: &impl HirDatabase, module: Module) -> Arc { +pub(crate) fn impls_in_module_with_source_map_query( + db: &impl HirDatabase, + module: Module, +) -> (Arc, Arc) { + let mut source_map = ImplSourceMap::default(); + let mut result = ModuleImplBlocks::new(); - result.collect(db, module); - Arc::new(result) + result.collect(db, module, &mut source_map); + + (Arc::new(result), Arc::new(source_map)) +} + +pub(crate) fn impls_in_module(db: &impl HirDatabase, module: Module) -> Arc { + db.impls_in_module_with_source_map(module).0 +} + +pub(crate) fn impls_in_module_source_map_query( + db: &impl HirDatabase, + module: Module, +) -> Arc { + db.impls_in_module_with_source_map(module).1 } diff --git a/crates/ra_hir/src/source_binder.rs b/crates/ra_hir/src/source_binder.rs index f523f0647..589efd023 100644 --- a/crates/ra_hir/src/source_binder.rs +++ b/crates/ra_hir/src/source_binder.rs @@ -13,7 +13,7 @@ use ra_syntax::{ }; use crate::{ - HirDatabase, Function, ModuleDef, + HirDatabase, Function, ModuleDef, Struct, Enum, AsName, Module, HirFileId, ids::{LocationCtx, SourceFileItemId}, }; @@ -128,6 +128,28 @@ pub fn function_from_child_node( function_from_source(db, file_id, fn_def) } +pub fn struct_from_module( + db: &impl HirDatabase, + module: Module, + struct_def: &ast::StructDef, +) -> Struct { + let (file_id, _) = module.definition_source(db); + let file_id = file_id.into(); + let ctx = LocationCtx::new(db, module, file_id); + Struct { + id: ctx.to_def(struct_def), + } +} + +pub fn enum_from_module(db: &impl HirDatabase, module: Module, enum_def: &ast::EnumDef) -> Enum { + let (file_id, _) = module.definition_source(db); + let file_id = file_id.into(); + let ctx = LocationCtx::new(db, module, file_id); + Enum { + id: ctx.to_def(enum_def), + } +} + pub fn macro_symbols(db: &impl HirDatabase, file_id: FileId) -> Vec<(SmolStr, TextRange)> { let module = match module_from_file_id(db, file_id) { Some(it) => it, diff --git a/crates/ra_hir/src/ty/method_resolution.rs b/crates/ra_hir/src/ty/method_resolution.rs index 9a571c2aa..d70a24582 100644 --- a/crates/ra_hir/src/ty/method_resolution.rs +++ b/crates/ra_hir/src/ty/method_resolution.rs @@ -44,7 +44,7 @@ impl CrateImplBlocks { &'a self, db: &'a impl HirDatabase, ty: &Ty, - ) -> impl Iterator + 'a { + ) -> impl Iterator + 'a { let fingerprint = TyFingerprint::for_impl(ty); fingerprint .and_then(|f| self.impls.get(&f)) @@ -56,7 +56,7 @@ impl CrateImplBlocks { module_id: *module_id, }; let module_impl_blocks = db.impls_in_module(module); - ImplBlock::from_id(module_impl_blocks, *impl_id) + (module, ImplBlock::from_id(module_impl_blocks, *impl_id)) }) } @@ -152,7 +152,7 @@ impl Ty { }; let impls = db.impls_in_crate(krate); - for impl_block in impls.lookup_impl_blocks(db, &derefed_ty) { + for (_, impl_block) in impls.lookup_impl_blocks(db, &derefed_ty) { for item in impl_block.items() { match item { ImplItem::Method(f) => { -- cgit v1.2.3