Make ImplData's fields public

diff --git a/crates/ra_hir/src/code_model.rs b/crates/ra_hir/src/code_model.rs
index f426f8c9fa8edeace57a5a3b51403649484e8d7a..3daf7488ed413011b7f187c9da55631a873305f5 100644 (file)
--- a/crates/ra_hir/src/code_model.rs
+++ b/crates/ra_hir/src/code_model.rs
@@ -10,9 +10,9 @@
     adt::VariantData,
     body::scope::ExprScopes,
     builtin_type::BuiltinType,
+    data::TraitData,
     nameres::per_ns::PerNs,
     resolver::{HasResolver, TypeNs},
-    traits::TraitData,
     type_ref::TypeRef,
     ContainerId, CrateModuleId, HasModule, ImplId, LocalEnumVariantId, LocalStructFieldId, Lookup,
     ModuleId, UnionId,

diff --git a/crates/ra_hir/src/impl_block.rs b/crates/ra_hir/src/impl_block.rs
index 774fa1d96404a8e9d97e4a06c06b588584becdc2..334eeebacd9a9f2428bbb4e670e2d8ae1f3473de 100644 (file)
--- a/crates/ra_hir/src/impl_block.rs
+++ b/crates/ra_hir/src/impl_block.rs
@@ -18,11 +18,11 @@ fn source(self, db: &(impl DefDatabase + AstDatabase)) -> Source<ast::ImplBlock>
 
 impl ImplBlock {
     pub fn target_trait(&self, db: &impl DefDatabase) -> Option<TypeRef> {
-        db.impl_data(self.id).target_trait().cloned()
+        db.impl_data(self.id).target_trait.clone()
     }
 
     pub fn target_type(&self, db: &impl DefDatabase) -> TypeRef {
-        db.impl_data(self.id).target_type().clone()
+        db.impl_data(self.id).target_type.clone()
     }
 
     pub fn target_ty(&self, db: &impl HirDatabase) -> Ty {
@@ -35,11 +35,11 @@ pub fn target_trait_ref(&self, db: &impl HirDatabase) -> Option<TraitRef> {
     }
 
     pub fn items(&self, db: &impl DefDatabase) -> Vec<AssocItem> {
-        db.impl_data(self.id).items().iter().map(|it| (*it).into()).collect()
+        db.impl_data(self.id).items.iter().map(|it| (*it).into()).collect()
     }
 
     pub fn is_negative(&self, db: &impl DefDatabase) -> bool {
-        db.impl_data(self.id).is_negative()
+        db.impl_data(self.id).is_negative
     }
 
     pub fn module(&self, db: &impl DefDatabase) -> Module {

diff --git a/crates/ra_hir/src/ty/infer.rs b/crates/ra_hir/src/ty/infer.rs
index 41a51283d67363f90de2734e6b34a7a2a3d28d5f..6e07ab86e89c868eb60e92b2782762890bf27a4c 100644 (file)
--- a/crates/ra_hir/src/ty/infer.rs
+++ b/crates/ra_hir/src/ty/infer.rs
@@ -22,7 +22,7 @@
 use rustc_hash::FxHashMap;
 
 use hir_def::{
-    function::FunctionData,
+    data::FunctionData,
     path::known,
     resolver::{HasResolver, Resolver, TypeNs},
     type_ref::{Mutability, TypeRef},

diff --git a/crates/ra_hir_def/src/data.rs b/crates/ra_hir_def/src/data.rs
index b73e0d0a7c855e86cea2abafd13eae448fb13585..ba47629db33bd2151144c7995be6578e9ccc36a9 100644 (file)
--- a/crates/ra_hir_def/src/data.rs
+++ b/crates/ra_hir_def/src/data.rs
@@ -1,3 +1,5 @@
+//! Contains basic data about various HIR declarations.
+
 use std::sync::Arc;
 
 use hir_expand::{
@@ -135,10 +137,10 @@ pub fn associated_types(&self) -> impl Iterator<Item = TypeAliasId> + '_ {
 
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct ImplData {
-    target_trait: Option<TypeRef>,
-    target_type: TypeRef,
-    items: Vec<AssocItemId>,
-    negative: bool,
+    pub target_trait: Option<TypeRef>,
+    pub target_type: TypeRef,
+    pub items: Vec<AssocItemId>,
+    pub is_negative: bool,
 }
 
 impl ImplData {
@@ -148,7 +150,7 @@ pub(crate) fn impl_data_query(db: &impl DefDatabase2, id: ImplId) -> Arc<ImplDat
 
         let target_trait = src.value.target_trait().map(TypeRef::from_ast);
         let target_type = TypeRef::from_ast_opt(src.value.target_type());
-        let negative = src.value.is_negative();
+        let is_negative = src.value.is_negative();
 
         let items = if let Some(item_list) = src.value.item_list() {
             item_list
@@ -184,23 +186,7 @@ pub(crate) fn impl_data_query(db: &impl DefDatabase2, id: ImplId) -> Arc<ImplDat
             Vec::new()
         };
 
-        let res = ImplData { target_trait, target_type, items, negative };
+        let res = ImplData { target_trait, target_type, items, is_negative };
         Arc::new(res)
     }
-
-    pub fn target_trait(&self) -> Option<&TypeRef> {
-        self.target_trait.as_ref()
-    }
-
-    pub fn target_type(&self) -> &TypeRef {
-        &self.target_type
-    }
-
-    pub fn items(&self) -> &[AssocItemId] {
-        &self.items
-    }
-
-    pub fn is_negative(&self) -> bool {
-        self.negative
-    }
 }