better names

diff --git a/crates/ra_hir/src/module_tree.rs b/crates/ra_hir/src/module_tree.rs
index 47c14af35b076441fe4797aef9173e261d91bffb..c9a15967c90455aadf5bba74ba195d876dc9b002 100644 (file)
--- a/crates/ra_hir/src/module_tree.rs
+++ b/crates/ra_hir/src/module_tree.rs
@@ -131,6 +131,17 @@ pub(crate) fn find_module_by_source(&self, source: SourceItemId) -> Option<Modul
         let (res, _) = self.mods.iter().find(|(_, m)| m.source == source)?;
         Some(res)
     }
+
+    fn alloc_mod(&mut self, data: ModuleData) -> ModuleId {
+        self.mods.alloc(data)
+    }
+
+    fn alloc_link(&mut self, data: LinkData) -> LinkId {
+        let owner = data.owner;
+        let id = self.links.alloc(data);
+        self.mods[owner].children.push(id);
+        id
+    }
 }
 
 impl ModuleId {
@@ -198,18 +209,6 @@ pub(crate) fn source(self, tree: &ModuleTree, db: &impl HirDatabase) -> TreeArc<
     }
 }
 
-impl ModuleTree {
-    fn push_mod(&mut self, data: ModuleData) -> ModuleId {
-        self.mods.alloc(data)
-    }
-    fn push_link(&mut self, data: LinkData) -> LinkId {
-        let owner = data.owner;
-        let id = self.links.alloc(data);
-        self.mods[owner].children.push(id);
-        id
-    }
-}
-
 fn modules(root: &impl ast::ModuleItemOwner) -> impl Iterator<Item = (Name, &ast::Module)> {
     root.items()
         .filter_map(|item| match item.kind() {
@@ -266,13 +265,13 @@ fn build_subtree(
     source: SourceItemId,
 ) -> ModuleId {
     visited.insert(source);
-    let id = tree.push_mod(ModuleData {
+    let id = tree.alloc_mod(ModuleData {
         source,
         parent,
         children: Vec::new(),
     });
     for sub in db.submodules(source).iter() {
-        let link = tree.push_link(LinkData {
+        let link = tree.alloc_link(LinkData {
             source: sub.source,
             name: sub.name.clone(),
             owner: id,