From e565fce30bb54bea0d620d16b0d24d4d0a37f6c6 Mon Sep 17 00:00:00 2001
From: MrGadget1024 <github@clevertech.net>
Date: Mon, 6 Sep 2021 13:15:36 -0400
Subject: [PATCH] Updated comments

---
 Assets/Mirror/Runtime/NetworkClient.cs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Assets/Mirror/Runtime/NetworkClient.cs b/Assets/Mirror/Runtime/NetworkClient.cs
index 5ef9daabc..14bf395c0 100644
--- a/Assets/Mirror/Runtime/NetworkClient.cs
+++ b/Assets/Mirror/Runtime/NetworkClient.cs
@@ -451,7 +451,8 @@ public static void RegisterHandler<T>(Action<T> handler, bool requireAuthenticat
         }
 
         /// <summary>Replace a handler for a particular message type. Should require authentication by default.</summary>
-        // TODO does anyone even use that? consider removing
+        // RegisterHandler throws a warning (as it should) if a handler is assigned twice
+        // Use of ReplaceHandler makes it clear the user intended to replace the handler
         public static void ReplaceHandler<T>(Action<NetworkConnection, T> handler, bool requireAuthentication = true)
             where T : struct, NetworkMessage
         {
@@ -460,7 +461,8 @@ public static void ReplaceHandler<T>(Action<NetworkConnection, T> handler, bool
         }
 
         /// <summary>Replace a handler for a particular message type. Should require authentication by default.</summary>
-        // TODO does anyone even use that? consider removing
+        // RegisterHandler throws a warning (as it should) if a handler is assigned twice
+        // Use of ReplaceHandler makes it clear the user intended to replace the handler
         public static void ReplaceHandler<T>(Action<T> handler, bool requireAuthentication = true)
             where T : struct, NetworkMessage
         {