From 87815b4615cae5b6c1d93f84bc1046fee06630a4 Mon Sep 17 00:00:00 2001
From: Imbris <imbrisf@gmail.com>
Date: Sat, 20 Aug 2022 17:21:46 -0400
Subject: [PATCH] Use `send_fallible` in `notify_client` implementation to
 document that any errors are ignored (and use `if let` instead of `map`)

---
 server/src/client.rs | 1 +
 server/src/lib.rs    | 7 ++++---
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/server/src/client.rs b/server/src/client.rs
index 6db1dc479a..64921b1402 100644
--- a/server/src/client.rs
+++ b/server/src/client.rs
@@ -140,6 +140,7 @@ impl Client {
         }*/
     }
 
+    /// Like `send` but any errors are explicitly ignored.
     pub(crate) fn send_fallible<M: Into<ServerMsg>>(&self, msg: M) { let _ = self.send(msg); }
 
     pub(crate) fn send_prepared(&self, msg: &PreparedMsg) -> Result<(), StreamError> {
diff --git a/server/src/lib.rs b/server/src/lib.rs
index bcc50f13e0..ecfd922977 100644
--- a/server/src/lib.rs
+++ b/server/src/lib.rs
@@ -1172,11 +1172,12 @@ impl Server {
     where
         S: Into<ServerMsg>,
     {
-        self.state
+        if let Some(client) = self.state
             .ecs()
             .read_storage::<Client>()
-            .get(entity)
-            .map(|c| c.send(msg));
+            .get(entity) {
+                client.send_fallible(msg);
+        }
     }
 
     pub fn notify_players(&mut self, msg: ServerGeneral) { self.state.notify_players(msg); }