Rename PUO::Insert to InsertQuiet

This doesn't produce a halo, so it's Quiet.  We're going to want one
which *does* produce a halo, so we must make room.

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

diff --git a/apitest/atmain.rs b/apitest/atmain.rs
index 9ce5b36eceabb3fd2ea76638fd9a9928c5149e98..a6cbc9fa66e4695a8289f004847389ca72cafc20 100644 (file)
--- a/apitest/atmain.rs
+++ b/apitest/atmain.rs
@@ -574,7 +574,7 @@ impl UsualCtx {
         if k == "Piece";
         let piece = v["piece"].as_str().unwrap().to_string();
         let op = v["op"].as_object().unwrap();
-        if let Some(_) = op.get("Insert");
+        if let Some(_) = op.get("InsertQuiet");
         then { added.push(piece); }
       }
     )?;

diff --git a/daemon/cmdlistener.rs b/daemon/cmdlistener.rs
index 6c1a93aa13a1eb96925239911dac9731da3d97a6..2410e1d24bfb50229ea498f4ca1f6c31c339494e 100644 (file)
--- a/daemon/cmdlistener.rs
+++ b/daemon/cmdlistener.rs
@@ -1136,7 +1136,7 @@ fn execute_game_insn<'cs, 'igr, 'ig: 'igr>(
           ig.ipieces.as_mut(modperm).insert(piece, IPiece {
             p, occilk, special,
           });
-          updates.push((piece, PieceUpdateOp::Insert(())));
+          updates.push((piece, PieceUpdateOp::InsertQuiet(())));
         })(); // <- no ?, infallible (to avoid leaking ilk)
         pos = (pos + posd)?;
       }
@@ -1408,11 +1408,11 @@ impl UpdateHandler {
           use PieceUpdateOp::*;
           let oe = bulk.pieces.get(&upiece);
           let ne = match (oe, uuop) {
-            ( None               , e        ) => Some( e          ),
-            ( Some( Insert(()) ) , Delete() ) => None,
-            ( Some( Insert(()) ) , _        ) => Some( Insert(()) ),
-            ( Some( Delete(  ) ) , _        ) => Some( Modify(()) ),
-            ( _                  , _        ) => Some( Modify(()) ),
+            ( None                    , e        ) => Some( e               ),
+            ( Some( InsertQuiet(()) ) , Delete() ) => None,
+            ( Some( InsertQuiet(()) ) , _        ) => Some( InsertQuiet(()) ),
+            ( Some( Delete     (  ) ) , _        ) => Some( Modify     (()) ),
+            ( _                       , _        ) => Some( Modify     (()) ),
           };
           trace_dbg!("accumulate", upiece, oe, uuop, ne);
           match ne {

diff --git a/src/hidden.rs b/src/hidden.rs
index 1dcdf43cbd177752b6d8e77319d14708fb64dcf8..5f5232018668c4129f73739f232baeeb9e6d8e0b 100644 (file)
--- a/src/hidden.rs
+++ b/src/hidden.rs
@@ -646,7 +646,7 @@ fn recalculate_occultation_general<
         kinds.new().at_all_visible(),
       ) {
         (false, false) => None,
-        (false, true ) => Some(PUO::Insert(())),
+        (false, true ) => Some(PUO::InsertQuiet(())),
         (true,  false) => Some(PUO::Delete()),
         (true,  true ) => Some(PUO::Modify(())),
       };

diff --git a/src/pcrender.rs b/src/pcrender.rs
index 807e131e8c5279bd9d8a3c463fb1ed29f20a4805..38350e80dbcfcbdf9571d350a1741f3537399ff9 100644 (file)
--- a/src/pcrender.rs
+++ b/src/pcrender.rs
@@ -99,7 +99,7 @@ impl PieceRenderInstructions {
     if matches_doesnot!(
       op,
       = Move(_) | MoveQuiet(_) | SetZLevel(_) | SetZLevelQuiet(_),
-      ! Delete() | Insert(_) | Modify(_) | ModifyQuiet(_),
+      ! Delete() | InsertQuiet(_) | Modify(_) | ModifyQuiet(_),
     ) {
       match self.occulted {
         Visible(_) | Occulted => (),

diff --git a/src/updates.rs b/src/updates.rs
index 52cf49440928d9112588426544879bd59c827ec3..e9967c0721061c74c9a084422d0a3573c5e44220 100644 (file)
--- a/src/updates.rs
+++ b/src/updates.rs
@@ -155,7 +155,7 @@ pub struct DataLoadPlayer {
 // Quiet means not to produce the yellow halo (see `movements` in script.ts)
 pub enum PieceUpdateOp<NS,ZL> {
   Delete(),
-  Insert(NS),
+  InsertQuiet(NS),
   Modify(NS),
   ModifyQuiet(NS),
   Move(Pos),
@@ -428,7 +428,7 @@ impl<NS,ZC> PieceUpdateOp<NS,ZC> {
     use PieceUpdateOp::*;
     match self {
       Delete() => None,
-      Insert(ns) => Some(ns),
+      InsertQuiet(ns) => Some(ns),
       Modify(ns) => Some(ns),
       ModifyQuiet(ns) => Some(ns),
       Move(_) => None,
@@ -445,7 +445,7 @@ impl<NS,ZC> PieceUpdateOp<NS,ZC> {
     use PieceUpdateOp::*;
     Ok(match self {
       Delete() => Delete(),
-      Insert(ns) => Insert(f(ns)?),
+      InsertQuiet(ns) => InsertQuiet(f(ns)?),
       Modify(ns) => Modify(f(ns)?),
       ModifyQuiet(ns) => ModifyQuiet(f(ns)?),
       Move(pos) => Move(pos),
@@ -458,7 +458,7 @@ impl<NS,ZC> PieceUpdateOp<NS,ZC> {
     use PieceUpdateOp::*;
     match self {
       Delete() => Delete(),
-      Insert(ns) => Insert(ns),
+      InsertQuiet(ns) => InsertQuiet(ns),
       Modify(ns) => Modify(ns),
       ModifyQuiet(ns) => ModifyQuiet(ns),
       Move(pos) => Move(*pos),
@@ -483,7 +483,7 @@ impl<NS,ZC> PieceUpdateOp<NS,ZC> {
     use PieceUpdateOp::*;
     match self {
       Delete() => None,
-      Insert(_) => None,
+      InsertQuiet(_) => None,
       Modify(_) => None,
       ModifyQuiet(_) => None,
       Move(_) => None,

diff --git a/templates/script.ts b/templates/script.ts
index ee920fcd2855f3f319ae850c62719d2d1fff61ca..d78da55b1ad526c81b3a676aceb275c6669ff7db 100644 (file)
--- a/templates/script.ts
+++ b/templates/script.ts
@@ -1789,7 +1789,7 @@ pieceops.Modify = <PieceHandler>function
   piece_modify(piece, p, info);
 }
 
-pieceops.Insert = <PieceHandler>function
+pieceops.InsertQuiet = <PieceHandler>function
 (piece: PieceId, xp: any, info: PreparedPieceState) {
   console.log('PIECE UPDATE INSERT ',piece,info)
   let delem = document.createElementNS(svg_ns,'defs');