hidden: Do not permute pieces for players with Visible view

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

diff --git a/src/hidden.rs b/src/hidden.rs
index 1d2a90c98e591005d4363ee43747704556ba62b8..03b4ff8fe835bf957b63d85f456e8dd82b9eb6e8 100644 (file)
--- a/src/hidden.rs
+++ b/src/hidden.rs
@@ -441,8 +441,13 @@ mod vpid {
       new_notches
     };
 
-    for gpl in gplayers.values_mut() {
-      // xxx don't do this for Visible, check occk
+    for (player, gpl) in gplayers.iter_mut() {
+      match occ.views.get_kind(player) {
+        OccK::Visible => continue,
+        OccK::Scrambled |
+        OccK::Displaced{..} |
+        OccK::Invisible => (),
+      };
 
       let mut fwd_updates = vec![];
       for (old, new) in izip!(&occ.notches.table, &new_notches) {