cmdlistener: Break out pieceid_lookup

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

diff --git a/daemon/cmdlistener.rs b/daemon/cmdlistener.rs
index a80795ac804807a15dbdbfc8fbdd5acfcebc8df4..f56dbacf8baabc8fb48e078567fdb63395b36400 100644 (file)
--- a/daemon/cmdlistener.rs
+++ b/daemon/cmdlistener.rs
@@ -307,6 +307,23 @@ fn execute_game_insn<'cs, 'igr, 'ig: 'igr>(
     (U{ pcs: vec![], log: vec![], raw: None }, resp, None, ig)
   }
 
+  #[throws(MgmtError)]
+  fn pieceid_lookup<'igr, 'ig: 'igr, 'cs, 
+                    F: FnOnce(&PerPlayerIdMap) -> MGR>
+    (
+      cs: &'cs CommandStream,
+      ig: &'igr mut Unauthorised<InstanceGuard<'ig>, InstanceName>,
+      player: PlayerId,
+      f: F,
+    ) -> ExecuteGameInsnResults<'igr, 'ig>
+  {
+    let superuser = cs.superuser.ok_or(ME::SuperuserAuthorisationRequired)?;
+    let ig = ig.by_mut(superuser.into());
+    let gpl = ig.gs.players.byid(player)?;
+    let resp = f(&gpl.idmap);
+    no_updates(ig, resp)
+  }
+
   #[throws(MgmtError)]
   fn update_links<'igr, 'ig: 'igr, 'cs,
                F: FnOnce(&mut Arc<LinksTable>) -> Result<Html,ME>>
@@ -430,18 +447,16 @@ fn execute_game_insn<'cs, 'igr, 'ig: 'igr>(
     },
 
     MGI::PieceIdLookupFwd { player, piece } => {
-      let superuser = cs.superuser.ok_or(ME::SuperuserAuthorisationRequired)?;
-      let ig = ig.by_mut(superuser.into());
-      let gpl = ig.gs.players.byid(player)?;
-      let vpid = gpl.idmap.fwd(piece);
-      no_updates(ig, MGR::VisiblePieceId(vpid))
+      pieceid_lookup(
+        cs, ig, player,
+        |ppidm| MGR::VisiblePieceId(ppidm.fwd(piece))
+      )?
     },
     MGI::PieceIdLookupRev { player, vpid } => {
-      let superuser = cs.superuser.ok_or(ME::SuperuserAuthorisationRequired)?;
-      let ig = ig.by_mut(superuser.into());
-      let gpl = ig.gs.players.byid(player)?;
-      let piece = gpl.idmap.rev(vpid);
-      no_updates(ig, MGR::InternalPieceId(piece))
+      pieceid_lookup(
+        cs, ig, player,
+        |ppidm| MGR::InternalPieceId(ppidm.rev(vpid))
+      )?
     },
 
     MGI::ListPieces => readonly(cs,ag,ig, &[TP::ViewNotSecret], |ig|{