hidden: recalculate: prep, move return construction

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/src/hidden.rs b/src/hidden.rs
index 209b5be2acbb22c737db1f66d05448226003c045..4c78c59a00b38e7aa6da4a4cad8c010e083c437d 100644 (file)
--- a/src/hidden.rs
+++ b/src/hidden.rs
@@ -223,7 +223,7 @@ fn recalculate_occultation_general<
   -> PieceUpdate
 {
   // fallible part
-  let (update, occids): (_, OldNew<Option<OccId>>) = {
+  let (puos, log, occids): (_, _, OldNew<Option<OccId>>) = {
     let nopiece = || internal_logic_error("piece vanished");
     let ipc = ipieces.get(piece).ok_or_else(nopiece)?;
     let gpc = gs.pieces.get(piece).ok_or_else(nopiece)?;
@@ -341,15 +341,9 @@ fn recalculate_occultation_general<
       },
     };
 
-    let update = PieceUpdate {
-      wrc: WRC::Unpredictable,
-      ops: PieceUpdateOps::PerPlayer(puos),
-      log,
-    };
-
     let occids = occulteds.map(|h| h.as_ref().map(|h| h.occid));
     
-    (update, occids)
+    (puos, log, occids)
   };
   
   // point of no return
@@ -368,7 +362,11 @@ fn recalculate_occultation_general<
     gs.pieces.byid_mut(piece).unwrap().occult.passive = *occids.new();
   })(); // <- no ?, infallible commitment
 
-  update
+  PieceUpdate {
+    wrc: WRC::Unpredictable,
+    ops: PieceUpdateOps::PerPlayer(puos),
+    log,
+  }
 }
 
 #[throws(InternalError)]