Merge #2222

2222: Remove owner from Body r=matklad a=matklad

cc @flodiebold 

I do this so that it's easier to move lowering code to another crate (owner is the only thing that tethers Body to the rest of the code), but it's interesting that this is a net reduction of lines. I think this might be considered an evidence that it's a good idea to not add "parent pointers" / parent ids to data structures, and instead add them to `ctx` objects which are used when building data structures

bors r+

Co-authored-by: Aleksey Kladov <[email protected]>

1 files changed, 2 insertions, 4 deletions

diff --git a/crates/ra_hir/src/expr/lower.rs b/crates/ra_hir/src/expr/lower.rs
index 7b0bfd919..adc68b23c 100644
--- a/crates/ra_hir/src/expr/lower.rs
+++ b/crates/ra_hir/src/expr/lower.rs
@@ -20,8 +20,8 @@ use ra_syntax::{
 use test_utils::tested_by;
 
 use crate::{
-    db::HirDatabase, AstId, DefWithBody, Either, HirFileId, MacroCallLoc, MacroFileKind,
-    Mutability, Path, Resolver, Source,
+    db::HirDatabase, AstId, Either, HirFileId, MacroCallLoc, MacroFileKind, Mutability, Path,
+    Resolver, Source,
 };
 
 use super::{
@@ -33,7 +33,6 @@ pub(super) fn lower(
     db: &impl HirDatabase,
     resolver: Resolver,
     file_id: HirFileId,
-    owner: DefWithBody,
     params: Option<ast::ParamList>,
     body: Option<ast::Expr>,
 ) -> (Body, BodySourceMap) {
@@ -44,7 +43,6 @@ pub(super) fn lower(
         current_file_id: file_id,
         source_map: BodySourceMap::default(),
         body: Body {
-            owner,
             exprs: Arena::default(),
             pats: Arena::default(),
             params: Vec::new(),