From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Sat, 30 Apr 2022 12:13:07 +0000 (+0100)
Subject: currency: Move some assignments about
X-Git-Tag: otter-1.1.0~403
X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=7861be1fe9ae0ece02c79c3105cf56d2c626d518;p=otter.git

currency: Move some assignments about

Tidying up.  No functional change.
(Was originally part of the next, bugfix, commit.)

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

diff --git a/src/currency.rs b/src/currency.rs
index 4c4704c7..b381cb28 100644
--- a/src/currency.rs
+++ b/src/currency.rs
@@ -139,11 +139,13 @@ impl PieceTrait for Banknote {
     let remaining = tgpc_value.qty.checked_sub(take)
       .ok_or(Ia::CurrencyShortfall)?;
 
-    tgpc.held = Some(player);
     ngpc.xdata_init(Value { qty: take })?;
-    tgpc.pinned = false;
+
+    tgpc.held = Some(player);
     ngpc.held = None;
 
+    tgpc.pinned = false;
+
     let logents = log_did_to_piece(
       ioccults, goccults, gpl, tgpc, tipc,
       &format!("took {}{}, leaving {}{}",