Rename ResolvedVisibility -> Visibility

2 files changed, 22 insertions, 27 deletions

diff --git a/crates/ra_hir_def/src/nameres/collector.rs b/crates/ra_hir_def/src/nameres/collector.rs
index f4678d145..63beaedc5 100644
--- a/crates/ra_hir_def/src/nameres/collector.rs
+++ b/crates/ra_hir_def/src/nameres/collector.rs
@@ -24,7 +24,7 @@ use crate::{
     },
     path::{ModPath, PathKind},
     per_ns::PerNs,
-    visibility::ResolvedVisibility,
+    visibility::Visibility,
     AdtId, AstId, ConstLoc, ContainerId, EnumLoc, EnumVariantId, FunctionLoc, ImplLoc, Intern,
     LocalModuleId, ModuleDefId, ModuleId, StaticLoc, StructLoc, TraitLoc, TypeAliasLoc, UnionLoc,
 };
@@ -109,7 +109,7 @@ struct MacroDirective {
 struct DefCollector<'a, DB> {
     db: &'a DB,
     def_map: CrateDefMap,
-    glob_imports: FxHashMap<LocalModuleId, Vec<(LocalModuleId, ResolvedVisibility)>>,
+    glob_imports: FxHashMap<LocalModuleId, Vec<(LocalModuleId, Visibility)>>,
     unresolved_imports: Vec<ImportDirective>,
     resolved_imports: Vec<ImportDirective>,
     unexpanded_macros: Vec<MacroDirective>,
@@ -217,8 +217,8 @@ where
         if export {
             self.update(
                 self.def_map.root,
-                &[(name, PerNs::macros(macro_, ResolvedVisibility::Public))],
-                ResolvedVisibility::Public,
+                &[(name, PerNs::macros(macro_, Visibility::Public))],
+                Visibility::Public,
             );
         }
     }
@@ -358,7 +358,7 @@ where
         let vis = self
             .def_map
             .resolve_visibility(self.db, module_id, &directive.import.visibility)
-            .unwrap_or(ResolvedVisibility::Public);
+            .unwrap_or(Visibility::Public);
 
         if import.is_glob {
             log::debug!("glob import: {:?}", import);
@@ -461,12 +461,7 @@ where
         }
     }
 
-    fn update(
-        &mut self,
-        module_id: LocalModuleId,
-        resolutions: &[(Name, PerNs)],
-        vis: ResolvedVisibility,
-    ) {
+    fn update(&mut self, module_id: LocalModuleId, resolutions: &[(Name, PerNs)], vis: Visibility) {
         self.update_recursive(module_id, resolutions, vis, 0)
     }
 
@@ -476,7 +471,7 @@ where
         resolutions: &[(Name, PerNs)],
         // All resolutions are imported with this visibility; the visibilies in
         // the `PerNs` values are ignored and overwritten
-        vis: ResolvedVisibility,
+        vis: Visibility,
         depth: usize,
     ) {
         if depth > 100 {
@@ -749,7 +744,7 @@ where
             .def_collector
             .def_map
             .resolve_visibility(self.def_collector.db, self.module_id, visibility)
-            .unwrap_or(ResolvedVisibility::Public);
+            .unwrap_or(Visibility::Public);
         let modules = &mut self.def_collector.def_map.modules;
         let res = modules.alloc(ModuleData::default());
         modules[res].parent = Some(self.module_id);
@@ -825,7 +820,7 @@ where
             .def_collector
             .def_map
             .resolve_visibility(self.def_collector.db, self.module_id, vis)
-            .unwrap_or(ResolvedVisibility::Public);
+            .unwrap_or(Visibility::Public);
         self.def_collector.update(self.module_id, &[(name, PerNs::from_def(def, vis))], vis)
     }
 
diff --git a/crates/ra_hir_def/src/nameres/path_resolution.rs b/crates/ra_hir_def/src/nameres/path_resolution.rs
index 8a6256eee..fd6422d60 100644
--- a/crates/ra_hir_def/src/nameres/path_resolution.rs
+++ b/crates/ra_hir_def/src/nameres/path_resolution.rs
@@ -21,7 +21,7 @@ use crate::{
     nameres::{BuiltinShadowMode, CrateDefMap},
     path::{ModPath, PathKind},
     per_ns::PerNs,
-    visibility::{RawVisibility, ResolvedVisibility},
+    visibility::{RawVisibility, Visibility},
     AdtId, CrateId, EnumVariantId, LocalModuleId, ModuleDefId, ModuleId,
 };
 
@@ -64,7 +64,7 @@ impl CrateDefMap {
     pub(super) fn resolve_name_in_extern_prelude(&self, name: &Name) -> PerNs {
         self.extern_prelude
             .get(name)
-            .map_or(PerNs::none(), |&it| PerNs::types(it, ResolvedVisibility::Public))
+            .map_or(PerNs::none(), |&it| PerNs::types(it, Visibility::Public))
     }
 
     pub(crate) fn resolve_visibility(
@@ -72,7 +72,7 @@ impl CrateDefMap {
         db: &impl DefDatabase,
         original_module: LocalModuleId,
         visibility: &RawVisibility,
-    ) -> Option<ResolvedVisibility> {
+    ) -> Option<Visibility> {
         match visibility {
             RawVisibility::Module(path) => {
                 let (result, remaining) =
@@ -82,14 +82,14 @@ impl CrateDefMap {
                 }
                 let types = result.take_types()?;
                 match types {
-                    ModuleDefId::ModuleId(m) => Some(ResolvedVisibility::Module(m)),
+                    ModuleDefId::ModuleId(m) => Some(Visibility::Module(m)),
                     _ => {
                         // error: visibility needs to refer to module
                         None
                     }
                 }
             }
-            RawVisibility::Public => Some(ResolvedVisibility::Public),
+            RawVisibility::Public => Some(Visibility::Public),
         }
     }
 
@@ -119,18 +119,18 @@ impl CrateDefMap {
                     tested_by!(macro_dollar_crate_self);
                     PerNs::types(
                         ModuleId { krate: self.krate, local_id: self.root }.into(),
-                        ResolvedVisibility::Public,
+                        Visibility::Public,
                     )
                 } else {
                     let def_map = db.crate_def_map(krate);
                     let module = ModuleId { krate, local_id: def_map.root };
                     tested_by!(macro_dollar_crate_other);
-                    PerNs::types(module.into(), ResolvedVisibility::Public)
+                    PerNs::types(module.into(), Visibility::Public)
                 }
             }
             PathKind::Crate => PerNs::types(
                 ModuleId { krate: self.krate, local_id: self.root }.into(),
-                ResolvedVisibility::Public,
+                Visibility::Public,
             ),
             // plain import or absolute path in 2015: crate-relative with
             // fallback to extern prelude (with the simplification in
@@ -161,7 +161,7 @@ impl CrateDefMap {
                 if let Some(local_id) = m {
                     PerNs::types(
                         ModuleId { krate: self.krate, local_id }.into(),
-                        ResolvedVisibility::Public,
+                        Visibility::Public,
                     )
                 } else {
                     log::debug!("super path in root module");
@@ -176,7 +176,7 @@ impl CrateDefMap {
                 };
                 if let Some(def) = self.extern_prelude.get(&segment) {
                     log::debug!("absolute path {:?} resolved to crate {:?}", path, def);
-                    PerNs::types(*def, ResolvedVisibility::Public)
+                    PerNs::types(*def, Visibility::Public)
                 } else {
                     return ResolvePathResult::empty(ReachedFixedPoint::No); // extern crate declarations can add to the extern prelude
                 }
@@ -225,7 +225,7 @@ impl CrateDefMap {
                     match enum_data.variant(&segment) {
                         Some(local_id) => {
                             let variant = EnumVariantId { parent: e, local_id };
-                            PerNs::both(variant.into(), variant.into(), ResolvedVisibility::Public)
+                            PerNs::both(variant.into(), variant.into(), Visibility::Public)
                         }
                         None => {
                             return ResolvePathResult::with(
@@ -274,12 +274,12 @@ impl CrateDefMap {
         let from_legacy_macro = self[module]
             .scope
             .get_legacy_macro(name)
-            .map_or_else(PerNs::none, |m| PerNs::macros(m, ResolvedVisibility::Public));
+            .map_or_else(PerNs::none, |m| PerNs::macros(m, Visibility::Public));
         let from_scope = self[module].scope.get(name, shadow);
         let from_extern_prelude = self
             .extern_prelude
             .get(name)
-            .map_or(PerNs::none(), |&it| PerNs::types(it, ResolvedVisibility::Public));
+            .map_or(PerNs::none(), |&it| PerNs::types(it, Visibility::Public));
         let from_prelude = self.resolve_in_prelude(db, name, shadow);
 
         from_legacy_macro.or(from_scope).or(from_extern_prelude).or(from_prelude)