Updated comments

This commit is contained in:
MrGadget1024 2021-09-06 13:15:36 -04:00
parent 50ee8737a5
commit e565fce30b

View File

@ -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
{