From 0a7c1a9629e3960fad3fd3d0f7fcd1236fbaa4e1 Mon Sep 17 00:00:00 2001
From: Ian Jackson <ijackson@chiark.greenend.org.uk>
Date: Sun, 23 Aug 2020 11:43:02 +0100
Subject: [PATCH] done

---
 src/bin/server.rs | 4 +---
 src/global.rs     | 2 --
 src/pieces.rs     | 1 +
 3 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/bin/server.rs b/src/bin/server.rs
index 0ff6afcf..1e23ef66 100644
--- a/src/bin/server.rs
+++ b/src/bin/server.rs
@@ -1,7 +1,5 @@
 
-// xxx need button(s) to kill old clients
-// and/or cause players to release grabs
-// ? need expiry of old clients?  limit of client count?
+// xxx need button(s) to cause players to release grabs ?
 
 #![feature(proc_macro_hygiene, decl_macro)]
 
diff --git a/src/global.rs b/src/global.rs
index 7212cb8c..9603fc1b 100644
--- a/src/global.rs
+++ b/src/global.rs
@@ -811,8 +811,6 @@ pub fn record_token<Id : AccessId> (
 
 // ---------- client expiiry ----------
 
-// xxx is lastseen updated by sse and by api calls ?
-
 pub fn client_expire_old_clients() {
   fn lock_even_poisoned(gref: &InstanceRef) -> MutexGuard<InstanceContainer> {
     match gref.0.lock() {
diff --git a/src/pieces.rs b/src/pieces.rs
index 45a34eb9..454c96b9 100644
--- a/src/pieces.rs
+++ b/src/pieces.rs
@@ -5,6 +5,7 @@ use crate::imports::*;
 type ColourMap = IndexVec<FaceId,Colour>;
 
 #[derive(Debug,Serialize,Deserialize)]
+// todo: this serialisation is rather large
 struct SimpleShape {
   desc : String,
   path : String,
-- 
2.30.2