Code cleanup

src/Impostor.Api/Events/Announcements/IAnnouncementRequestEvent.cs

@@ -3,40 +3,40 @@ using Impostor.Api.Innersloth;
 namespace Impostor.Api.Events.Announcements
 {
     /// <summary>
-    /// Event fired after client requests a announcement.
+    ///     Event fired after client requests a announcement.
     /// </summary>
     public interface IAnnouncementRequestEvent : IEvent
     {
         public interface IResponse
         {
             /// <summary>
-            /// Gets or sets FreeWeekendState, currently unused by the client.
+            ///     Gets or sets FreeWeekendState, currently unused by the client.
             /// </summary>
             public FreeWeekendState FreeWeekendState { get; set; }
 
             /// <summary>
-            /// Gets or sets a value indicating whether announcement should be loaded from client's cache, can save some bytes.
+            ///     Gets or sets a value indicating whether announcement should be loaded from client's cache, can save some bytes.
             /// </summary>
             public bool UseCached { get; set; }
 
             /// <summary>
-            /// Gets or sets announcement, should be null when <see cref="UseCached"/> is set to true.
+            ///     Gets or sets announcement, should be null when <see cref="UseCached" /> is set to true.
             /// </summary>
             public Announcement? Announcement { get; set; }
         }
 
         /// <summary>
-        /// Gets client's last announcement id.
+        ///     Gets client's last announcement id.
         /// </summary>
         public int Id { get; }
 
         /// <summary>
-        /// Gets client's language.
+        ///     Gets client's language.
         /// </summary>
         public Language Language { get; }
 
         /// <summary>
-        /// Gets or sets plugin made response.
+        ///     Gets or sets plugin made response.
         /// </summary>
         public IResponse Response { get; set; }
     }

src/Impostor.Api/Events/Attributes/EventListenerAttribute.cs

@@ -27,8 +27,8 @@ namespace Impostor.Api.Events
         public Type? Event { get; set; }
 
         /// <summary>
-        ///     If set to true, the listener will be called regardless of the <see cref="IEventCancelable.IsCancelled"/>.
+        ///     If set to true, the listener will be called regardless of the <see cref="IEventCancelable.IsCancelled" />.
         /// </summary>
         public bool IgnoreCancelled { get; set; }
     }
-}
+}

src/Impostor.Api/Events/Game/IGameCreatedEvent.cs

@@ -3,7 +3,7 @@
 namespace Impostor.Api.Events
 {
     /// <summary>
-    ///     Called whenever a new <see cref="IGame"/> is created.
+    ///     Called whenever a new <see cref="IGame" /> is created.
     /// </summary>
     public interface IGameCreatedEvent : IGameEvent
     {

src/Impostor.Api/Events/Game/IGameDestroyedEvent.cs

@@ -3,7 +3,7 @@
 namespace Impostor.Api.Events
 {
     /// <summary>
-    ///     Called whenever a new <see cref="IGame"/> is destroyed.
+    ///     Called whenever a new <see cref="IGame" /> is destroyed.
     /// </summary>
     public interface IGameDestroyedEvent : IGameEvent
     {

src/Impostor.Api/Events/Game/IGameEvent.cs

@@ -5,7 +5,7 @@ namespace Impostor.Api.Events
     public interface IGameEvent : IEvent
     {
         /// <summary>
-        ///     Gets the <see cref="IGame"/> this event belongs to.
+        ///     Gets the <see cref="IGame" /> this event belongs to.
         /// </summary>
         IGame Game { get; }
     }

src/Impostor.Api/Events/Game/IGameStartingEvent.cs

@@ -3,7 +3,7 @@
     /// <summary>
     ///     Called when the game is going to start.
     ///     When this is called, not all players are initialized properly yet.
-    ///     If you want to get correct player states, use <see cref="IGameStartedEvent"/>.
+    ///     If you want to get correct player states, use <see cref="IGameStartedEvent" />.
     /// </summary>
     public interface IGameStartingEvent : IGameEvent
     {

src/Impostor.Api/Events/Game/Player/IPlayerCompletedTaskEvent.cs

@@ -1,5 +1,4 @@
-using Impostor.Api.Innersloth;
-using Impostor.Api.Net.Inner.Objects;
+using Impostor.Api.Net.Inner.Objects;
 
 namespace Impostor.Api.Events.Player
 {

src/Impostor.Api/Events/Game/Player/IPlayerEvent.cs

@@ -6,14 +6,14 @@ namespace Impostor.Api.Events.Player
     public interface IPlayerEvent : IGameEvent
     {
         /// <summary>
-        ///     Gets the <see cref="IClientPlayer"/> that triggered this <see cref="IPlayerEvent"/>.
+        ///     Gets the <see cref="IClientPlayer" /> that triggered this <see cref="IPlayerEvent" />.
         /// </summary>
         IClientPlayer ClientPlayer { get; }
 
         /// <summary>
-        ///     Gets the networked <see cref="IInnerPlayerControl"/> that triggered this <see cref="IPlayerEvent"/>.
-        ///     This <see cref="IInnerPlayerControl"/> belongs to the <see cref="IClientPlayer"/>.
+        ///     Gets the networked <see cref="IInnerPlayerControl" /> that triggered this <see cref="IPlayerEvent" />.
+        ///     This <see cref="IInnerPlayerControl" /> belongs to the <see cref="IClientPlayer" />.
         /// </summary>
         IInnerPlayerControl PlayerControl { get; }
     }
-}
+}

src/Impostor.Api/Events/IEvent.cs

@@ -3,4 +3,4 @@
     public interface IEvent
     {
     }
-}
+}

src/Impostor.Api/Events/IEventCancelable.cs

@@ -7,4 +7,4 @@
         /// </summary>
         bool IsCancelled { get; set; }
     }
-}
+}

src/Impostor.Api/Events/IEventListener.cs

@@ -3,4 +3,4 @@
     public interface IEventListener
     {
     }
-}
+}

src/Impostor.Api/Events/Managers/IEventManager.cs

@@ -16,20 +16,20 @@ namespace Impostor.Api.Events.Managers
             where TListener : IEventListener;
 
         /// <summary>
-        ///     Returns true if an event with the type <see cref="TEvent"/> is registered.
+        ///     Returns true if an event with the type <see cref="TEvent" /> is registered.
         /// </summary>
-        /// <returns>True if the <see cref="TEvent"/> is registered.</returns>
+        /// <returns>True if the <see cref="TEvent" /> is registered.</returns>
         /// <typeparam name="TEvent">Type of the event.</typeparam>
         bool IsRegistered<TEvent>()
             where TEvent : IEvent;
 
         /// <summary>
-        ///     Call all the event listeners for the type <see cref="TEvent"/>.
+        ///     Call all the event listeners for the type <see cref="TEvent" />.
         /// </summary>
         /// <param name="event">The event argument.</param>
         /// <typeparam name="TEvent">Type of the event.</typeparam>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask CallAsync<TEvent>(TEvent @event)
             where TEvent : IEvent;
     }
-}
+}

src/Impostor.Api/Extensions/SpanReaderExtensions.cs

@@ -5,7 +5,7 @@ using System.Runtime.CompilerServices;
 namespace Impostor.Api
 {
     /// <summary>
-    /// Priovides a StreamReader-like api throught extensions
+    ///     Priovides a StreamReader-like api throught extensions
     /// </summary>
     public static class SpanReaderExtensions
     {
@@ -53,7 +53,7 @@ namespace Impostor.Api
         }
 
         /// <summary>
-        /// Advances the position of <see cref="input"/> by the size of <see cref="T"/>.
+        ///     Advances the position of <see cref="input" /> by the size of <see cref="T" />.
         /// </summary>
         /// <typeparam name="T">Type that will be read.</typeparam>
         /// <param name="input">input "stream"/span.</param>
@@ -67,4 +67,4 @@ namespace Impostor.Api
             return original;
         }
     }
-}
+}

src/Impostor.Api/Extensions/SystemTypesExtensions.cs

@@ -9,4 +9,4 @@ namespace Impostor.Api
             return SystemTypeHelpers.Names[(int)type];
         }
     }
-}
+}

src/Impostor.Api/Games/Extensions/GameExtensions.cs

@@ -39,4 +39,4 @@ namespace Impostor.Api.Games
             return game.SendToAsync(writer, player.Client);
         }
     }
-}
+}

src/Impostor.Api/Games/Extensions/GameManagerExtensions.cs

@@ -11,4 +11,4 @@ namespace Impostor.Api.Games
             return manager.Games.Count(game => map.HasFlag((MapFlags)(1 << (byte)game.Options.Map)));
         }
     }
-}
+}

src/Impostor.Api/Games/GameCode.cs

@@ -48,19 +48,19 @@ namespace Impostor.Api.Games
 
         public static GameCode From(string value) => new GameCode(value);
 
-        /// <inheritdoc/>
+        /// <inheritdoc />
         public bool Equals(GameCode other)
         {
             return Code == other.Code && Value == other.Value;
         }
 
-        /// <inheritdoc/>
+        /// <inheritdoc />
         public override bool Equals(object? obj)
         {
             return obj is GameCode other && Equals(other);
         }
 
-        /// <inheritdoc/>
+        /// <inheritdoc />
         public override int GetHashCode()
         {
             return HashCode.Combine(Code, Value);
@@ -71,4 +71,4 @@ namespace Impostor.Api.Games
             return Code;
         }
     }
-}
+}

src/Impostor.Api/Games/GameJoinError.cs

@@ -41,8 +41,8 @@
         ///     Custom error by a plugin.
         /// </summary>
         /// <remarks>
-        ///     A custom message can be set in <see cref="GameJoinResult.Message"/>.
+        ///     A custom message can be set in <see cref="GameJoinResult.Message" />.
         /// </remarks>
         Custom,
     }
-}
+}

src/Impostor.Api/Games/GameJoinResult.cs

@@ -45,4 +45,4 @@ namespace Impostor.Api.Games
             return new GameJoinResult(error);
         }
     }
-}
+}

src/Impostor.Api/Games/IGame.cs

@@ -29,7 +29,7 @@ namespace Impostor.Api.Games
         bool IsPublic { get; }
 
         /// <summary>
-        /// Gets or sets display name on game list.
+        ///     Gets or sets display name on game list.
         /// </summary>
         string? DisplayName { get; set; }
 
@@ -43,28 +43,27 @@ namespace Impostor.Api.Games
             where T : IInnerNetObject;
 
         /// <summary>
-        ///     Adds an <see cref="IPAddress"/> to the ban list of this game.
-        ///     Prevents all future joins from this <see cref="IPAddress"/>.
-        ///
-        ///     This does not kick the player with that <see cref="IPAddress"/> from the lobby.
+        ///     Adds an <see cref="IPAddress" /> to the ban list of this game.
+        ///     Prevents all future joins from this <see cref="IPAddress" />.
+        ///     This does not kick the player with that <see cref="IPAddress" /> from the lobby.
         /// </summary>
         /// <param name="ipAddress">
-        ///     The <see cref="IPAddress"/> to ban.
+        ///     The <see cref="IPAddress" /> to ban.
         /// </param>
         void BanIp(IPAddress ipAddress);
 
         /// <summary>
-        ///     Syncs the internal <see cref="GameOptionsData"/> to all players.
+        ///     Syncs the internal <see cref="GameOptionsData" /> to all players.
         ///     Necessary to do if you modified it, otherwise it won't be used.
         /// </summary>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask SyncSettingsAsync();
 
         /// <summary>
         ///     Sets game's privacy.
         /// </summary>
         /// <param name="isPublic">Privacy to set.</param>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask SetPrivacyAsync(bool isPublic);
 
         /// <summary>
@@ -72,7 +71,7 @@ namespace Impostor.Api.Games
         /// </summary>
         /// <param name="writer">Message to send.</param>
         /// <param name="states">Required limbo state of the player.</param>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask SendToAllAsync(IMessageWriter writer, LimboStates states = LimboStates.NotLimbo);
 
         /// <summary>
@@ -81,7 +80,7 @@ namespace Impostor.Api.Games
         /// <param name="writer">Message to send.</param>
         /// <param name="senderId">The player to exclude from sending the message.</param>
         /// <param name="states">Required limbo state of the player.</param>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask SendToAllExceptAsync(IMessageWriter writer, int senderId, LimboStates states = LimboStates.NotLimbo);
 
         /// <summary>
@@ -89,7 +88,7 @@ namespace Impostor.Api.Games
         /// </summary>
         /// <param name="writer">Message to send.</param>
         /// <param name="id">ID of the client.</param>
-        /// <returns>A <see cref="ValueTask"/> representing the asynchronous operation.</returns>
+        /// <returns>A <see cref="ValueTask" /> representing the asynchronous operation.</returns>
         ValueTask SendToAsync(IMessageWriter writer, int id);
     }
 }

src/Impostor.Api/Games/IGameCodeFactory.cs

@@ -4,4 +4,4 @@
     {
         GameCode Create();
     }
-}
+}

src/Impostor.Api/Games/Managers/IGameManager.cs

@@ -12,4 +12,4 @@ namespace Impostor.Api.Games.Managers
 
         ValueTask<IGame> CreateAsync(GameOptionsData options);
     }
-}
+}

src/Impostor.Api/Innersloth/AlterGameTags.cs

@@ -4,4 +4,4 @@
     {
         ChangePrivacy = 1,
     }
-}
+}

src/Impostor.Api/Innersloth/ChatNoteType.cs

@@ -4,4 +4,4 @@
     {
         DidVote = 0,
     }
-}
+}

src/Impostor.Api/Innersloth/DeathReason.cs

@@ -6,4 +6,4 @@
         Kill = 1,
         Disconnect = 2,
     }
-}
+}

src/Impostor.Api/Innersloth/GameKeywords.cs

@@ -16,4 +16,4 @@ namespace Impostor.Api.Innersloth
         Polish = 128,
         English = 256,
     }
-}
+}

src/Impostor.Api/Innersloth/GameOptionsData.cs

@@ -7,130 +7,130 @@ namespace Impostor.Api.Innersloth
     public class GameOptionsData
     {
         /// <summary>
-        /// The latest major version of the game client.
+        ///     The latest major version of the game client.
         /// </summary>
         public const int LatestVersion = 4;
 
         /// <summary>
-        /// Gets or sets host's version of the game.
+        ///     Gets or sets host's version of the game.
         /// </summary>
         public byte Version { get; set; } = LatestVersion;
 
         /// <summary>
-        /// Gets or sets the maximum amount of players for this lobby.
+        ///     Gets or sets the maximum amount of players for this lobby.
         /// </summary>
         public byte MaxPlayers { get; set; } = 10;
 
         /// <summary>
-        /// Gets or sets the language of the lobby as per <see cref="GameKeywords"/> enum.
+        ///     Gets or sets the language of the lobby as per <see cref="GameKeywords" /> enum.
         /// </summary>
         public GameKeywords Keywords { get; set; } = GameKeywords.English;
 
         /// <summary>
-        /// Gets or sets the Map selected for this lobby.
+        ///     Gets or sets the Map selected for this lobby.
         /// </summary>
         public MapTypes Map { get; set; } = MapTypes.Skeld;
 
         /// <summary>
-        /// Gets or sets the Player speed modifier.
+        ///     Gets or sets the Player speed modifier.
         /// </summary>
         public float PlayerSpeedMod { get; set; } = 1f;
 
         /// <summary>
-        /// Gets or sets the Light modifier for the players that are members of the crew as a multiplier value.
+        ///     Gets or sets the Light modifier for the players that are members of the crew as a multiplier value.
         /// </summary>
         public float CrewLightMod { get; set; } = 1f;
 
         /// <summary>
-        /// Gets or sets the Light modifier for the players that are Impostors as a multiplier value.
+        ///     Gets or sets the Light modifier for the players that are Impostors as a multiplier value.
         /// </summary>
         public float ImpostorLightMod { get; set; } = 1f;
 
         /// <summary>
-        /// Gets or sets the Impostor cooldown to kill in seconds.
+        ///     Gets or sets the Impostor cooldown to kill in seconds.
         /// </summary>
         public float KillCooldown { get; set; } = 15f;
 
         /// <summary>
-        /// Gets or sets the number of common tasks.
+        ///     Gets or sets the number of common tasks.
         /// </summary>
         public int NumCommonTasks { get; set; } = 1;
 
         /// <summary>
-        /// Gets or sets the number of long tasks.
+        ///     Gets or sets the number of long tasks.
         /// </summary>
         public int NumLongTasks { get; set; } = 1;
 
         /// <summary>
-        /// Gets or sets the number of short tasks.
+        ///     Gets or sets the number of short tasks.
         /// </summary>
         public int NumShortTasks { get; set; } = 2;
 
         /// <summary>
-        /// Gets or sets the maximum amount of emergency meetings each player can call during the game in seconds.
+        ///     Gets or sets the maximum amount of emergency meetings each player can call during the game in seconds.
         /// </summary>
         public int NumEmergencyMeetings { get; set; } = 1;
 
         /// <summary>
-        /// Gets or sets the cooldown between each time any player can call an emergency meeting in seconds.
+        ///     Gets or sets the cooldown between each time any player can call an emergency meeting in seconds.
         /// </summary>
         public int EmergencyCooldown { get; set; } = 15;
 
         /// <summary>
-        /// Gets or sets the number of impostors for this lobby.
+        ///     Gets or sets the number of impostors for this lobby.
         /// </summary>
         public int NumImpostors { get; set; } = 1;
 
         /// <summary>
-        /// Gets or sets a value indicating whether ghosts (dead crew members) can do tasks.
+        ///     Gets or sets a value indicating whether ghosts (dead crew members) can do tasks.
         /// </summary>
         public bool GhostsDoTasks { get; set; } = true;
 
         /// <summary>
-        /// Gets or sets the Kill as per values in <see cref="KillDistances"/>.
+        ///     Gets or sets the Kill as per values in <see cref="KillDistances" />.
         /// </summary>
         public KillDistances KillDistance { get; set; } = KillDistances.Normal;
 
         /// <summary>
-        /// Gets or sets the time for discussion before voting time in seconds.
+        ///     Gets or sets the time for discussion before voting time in seconds.
         /// </summary>
         public int DiscussionTime { get; set; } = 15;
 
         /// <summary>
-        /// Gets or sets the time for voting in seconds.
+        ///     Gets or sets the time for voting in seconds.
         /// </summary>
         public int VotingTime { get; set; } = 120;
 
         /// <summary>
-        /// Gets or sets a value indicating whether an ejected player is an impostor or not.
+        ///     Gets or sets a value indicating whether an ejected player is an impostor or not.
         /// </summary>
         public bool ConfirmImpostor { get; set; } = true;
 
         /// <summary>
-        /// Gets or sets a value indicating whether players are able to see tasks being performed by other players.
+        ///     Gets or sets a value indicating whether players are able to see tasks being performed by other players.
         /// </summary>
         /// <remarks>
-        /// By being set to true, tasks such as Empty Garbage, Submit Scan, Clear asteroids, Prime shields execution will be visible to other players.
+        ///     By being set to true, tasks such as Empty Garbage, Submit Scan, Clear asteroids, Prime shields execution will be visible to other players.
         /// </remarks>
         public bool VisualTasks { get; set; } = true;
 
         /// <summary>
-        /// Gets or sets a value indicating whether the vote is anonymous.
+        ///     Gets or sets a value indicating whether the vote is anonymous.
         /// </summary>
         public bool AnonymousVotes { get; set; }
 
         /// <summary>
-        /// Gets or sets the task bar update mode as per values in <see cref="Innersloth.TaskBarUpdate"/>.
+        ///     Gets or sets the task bar update mode as per values in <see cref="Innersloth.TaskBarUpdate" />.
         /// </summary>
         public TaskBarUpdate TaskBarUpdate { get; set; } = TaskBarUpdate.Always;
 
         /// <summary>
-        /// Gets or sets a value indicating whether the GameOptions are the default ones.
+        ///     Gets or sets a value indicating whether the GameOptions are the default ones.
         /// </summary>
         public bool IsDefaults { get; set; } = true;
 
         /// <summary>
-        /// Deserialize a packet/message to a new GameOptionsData object.
+        ///     Deserialize a packet/message to a new GameOptionsData object.
         /// </summary>
         /// <param name="reader">Message reader object containing the raw message.</param>
         /// <returns>GameOptionsData object.</returns>
@@ -142,7 +142,7 @@ namespace Impostor.Api.Innersloth
         }
 
         /// <summary>
-        /// Serializes this instance of GameOptionsData object to a specified BinaryWriter.
+        ///     Serializes this instance of GameOptionsData object to a specified BinaryWriter.
         /// </summary>
         /// <param name="writer">The stream to write the message to.</param>
         /// <param name="version">The version of the game.</param>
@@ -198,7 +198,7 @@ namespace Impostor.Api.Innersloth
         }
 
         /// <summary>
-        /// Deserialize a ReadOnlyMemory object to this instance of the GameOptionsData object.
+        ///     Deserialize a ReadOnlyMemory object to this instance of the GameOptionsData object.
         /// </summary>
         /// <param name="memory">Memory containing the message/packet.</param>
         public void Deserialize(ReadOnlyMemory<byte> memory)

src/Impostor.Api/Innersloth/GameStates.cs

@@ -8,4 +8,4 @@
         Ended = 3,
         Destroyed = 4,
     }
-}
+}

src/Impostor.Api/Innersloth/GameVersion.cs

@@ -7,4 +7,4 @@ namespace Impostor.Api.Innersloth
             return (year * 25000) + (month * 1800) + (day * 50) + rev;
         }
     }
-}
+}

src/Impostor.Api/Innersloth/MapFlags.cs

@@ -9,4 +9,4 @@ namespace Impostor.Api.Innersloth
         MiraHQ = 2,
         Polus = 4,
     }
-}
+}

src/Impostor.Api/Innersloth/MapTypes.cs

@@ -6,4 +6,4 @@
         MiraHQ = 1,
         Polus = 2,
     }
-}
+}

src/Impostor.Api/Innersloth/SystemTypeHelpers.cs

@@ -20,9 +20,9 @@ namespace Impostor.Api.Innersloth
                     SystemTypes.LifeSupp => "O2",
                     SystemTypes.LowerEngine => "Lower Engine",
                     SystemTypes.LockerRoom => "Locker Room",
-                    _ => x.ToString()
+                    _ => x.ToString(),
                 };
             }).ToArray();
         }
     }
-}
+}

src/Impostor.Api/Innersloth/SystemTypes.cs

@@ -39,7 +39,7 @@
         Sabotage = 17,
 
         /// <summary>
-        /// Decontam on Mira and bottom decontam on Polus
+        ///     Decontam on Mira and bottom decontam on Polus
         /// </summary>
         Decontamination = 18,
 
@@ -58,7 +58,7 @@
         Dropship = 25,
 
         /// <summary>
-        /// Top decontam on Polus
+        ///     Top decontam on Polus
         /// </summary>
         Decontamination2 = 26,
 

src/Impostor.Api/Innersloth/TaskTypes.cs

@@ -46,4 +46,4 @@
         RecordTemperature = 41,
         RebootWifi = 42,
     }
-}
+}

src/Impostor.Api/Innersloth/TextBox.cs

@@ -7,4 +7,4 @@
             return i == ' ' || (i >= 'A' && i <= 'Z') || (i >= 'a' && i <= 'z') || (i >= '0' && i <= '9') || (i >= 'À' && i <= 'ÿ') || (i >= 'Ѐ' && i <= 'џ') || (i >= 'ㄱ' && i <= 'ㆎ') || (i >= '가' && i <= '힣');
         }
     }
-}
+}

src/Impostor.Api/Net/IClient.cs

@@ -29,7 +29,7 @@ namespace Impostor.Api.Net
         string Name { get; }
 
         /// <summary>
-        /// Gets mods sent by client in modded handshake.
+        ///     Gets mods sent by client in modded handshake.
         /// </summary>
         ISet<Mod> Mods { get; }
 
@@ -57,7 +57,7 @@ namespace Impostor.Api.Net
         IDictionary<object, object> Items { get; }
 
         /// <summary>
-        ///     Gets or sets the current game data of the <see cref="IClient"/>.
+        ///     Gets or sets the current game data of the <see cref="IClient" />.
         /// </summary>
         IClientPlayer? Player { get; }
 
@@ -68,16 +68,16 @@ namespace Impostor.Api.Net
         ValueTask HandleDisconnectAsync(string reason);
 
         /// <summary>
-        ///     Disconnect the client with a <see cref="DisconnectReason"/>.
+        ///     Disconnect the client with a <see cref="DisconnectReason" />.
         /// </summary>
         /// <param name="reason">
         ///     The message to show to the player.
         /// </param>
         /// <param name="message">
-        ///     Only used when <see cref="reason"/> is set to <see cref="DisconnectReason.Custom"/>.
+        ///     Only used when <see cref="reason" /> is set to <see cref="DisconnectReason.Custom" />.
         /// </param>
         /// <returns>
-        ///     A <see cref="ValueTask"/> representing the asynchronous operation.
+        ///     A <see cref="ValueTask" /> representing the asynchronous operation.
         /// </returns>
         ValueTask DisconnectAsync(DisconnectReason reason, string? message = null);
     }

src/Impostor.Api/Net/IClientPlayer.cs

@@ -6,7 +6,7 @@ using Impostor.Api.Net.Inner.Objects;
 namespace Impostor.Api.Net
 {
     /// <summary>
-    ///     Represents a player in <see cref="IGame"/>.
+    ///     Represents a player in <see cref="IGame" />.
     /// </summary>
     public interface IClientPlayer
     {
@@ -16,7 +16,7 @@ namespace Impostor.Api.Net
         IClient Client { get; }
 
         /// <summary>
-        ///     Gets the game where the <see cref="IClientPlayer"/> belongs to.
+        ///     Gets the game where the <see cref="IClientPlayer" /> belongs to.
         /// </summary>
         IGame Game { get; }
 
@@ -30,10 +30,10 @@ namespace Impostor.Api.Net
         public bool IsHost { get; }
 
         /// <summary>
-        ///     Checks if the specified <see cref="IInnerNetObject"/> is owned by <see cref="IClientPlayer"/>.
+        ///     Checks if the specified <see cref="IInnerNetObject" /> is owned by <see cref="IClientPlayer" />.
         /// </summary>
-        /// <param name="netObject">The <see cref="IInnerNetObject"/>.</param>
-        /// <returns>Returns true if owned by <see cref="IClientPlayer"/>.</returns>
+        /// <param name="netObject">The <see cref="IInnerNetObject" />.</param>
+        /// <returns>Returns true if owned by <see cref="IClientPlayer" />.</returns>
         bool IsOwner(IInnerNetObject netObject);
 
         ValueTask KickAsync();

src/Impostor.Api/Net/IHazelConnection.cs

@@ -38,4 +38,4 @@ namespace Impostor.Api.Net
         /// <returns></returns>
         ValueTask DisconnectAsync(string? reason);
     }
-}
+}

src/Impostor.Api/Net/Inner/IInnerNetObject.cs

@@ -6,4 +6,4 @@
 
         public int OwnerId { get; }
     }
-}
+}

src/Impostor.Api/Net/Inner/Objects/Components/IInnerCustomNetworkTransform.cs

@@ -16,7 +16,7 @@ namespace Impostor.Api.Net.Inner.Objects.Components
         Vector2 Velocity { get; }
 
         /// <summary>
-        ///     Snaps the current to the given position <see cref="IInnerPlayerControl"/>.
+        ///     Snaps the current to the given position <see cref="IInnerPlayerControl" />.
         /// </summary>
         /// <param name="position">The target position.</param>
         /// <returns>Task that must be awaited.</returns>

src/Impostor.Api/Net/Inner/Objects/IInnerPlayerControl.cs

@@ -7,30 +7,30 @@ namespace Impostor.Api.Net.Inner.Objects
     public interface IInnerPlayerControl : IInnerNetObject
     {
         /// <summary>
-        ///     Gets the <see cref="PlayerId"/> assigned by the client of the host of the game.
+        ///     Gets the <see cref="PlayerId" /> assigned by the client of the host of the game.
         /// </summary>
         byte PlayerId { get; }
 
         /// <summary>
-        ///     Gets the <see cref="IInnerPlayerPhysics"/> of the <see cref="IInnerPlayerControl"/>.
+        ///     Gets the <see cref="IInnerPlayerPhysics" /> of the <see cref="IInnerPlayerControl" />.
         ///     Contains vent logic.
         /// </summary>
         IInnerPlayerPhysics Physics { get; }
 
         /// <summary>
-        ///     Gets the <see cref="IInnerCustomNetworkTransform"/> of the <see cref="IInnerPlayerControl"/>.
+        ///     Gets the <see cref="IInnerCustomNetworkTransform" /> of the <see cref="IInnerPlayerControl" />.
         ///     Contains position data about the player.
         /// </summary>
         IInnerCustomNetworkTransform NetworkTransform { get; }
 
         /// <summary>
-        ///     Gets the <see cref="IInnerPlayerInfo"/> of the <see cref="IInnerPlayerControl"/>.
+        ///     Gets the <see cref="IInnerPlayerInfo" /> of the <see cref="IInnerPlayerControl" />.
         ///     Contains metadata about the player.
         /// </summary>
         IInnerPlayerInfo PlayerInfo { get; }
 
         /// <summary>
-        ///     Sets the name of the current <see cref="IInnerPlayerControl"/>.
+        ///     Sets the name of the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="name">A name for the player.</param>
@@ -38,7 +38,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SetNameAsync(string name);
 
         /// <summary>
-        ///     Sets the color of the current <see cref="IInnerPlayerControl"/>.
+        ///     Sets the color of the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="colorType">A color for the player.</param>
@@ -46,7 +46,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SetColorAsync(ColorType colorType);
 
         /// <summary>
-        ///     Sets the hat of the current <see cref="IInnerPlayerControl"/>.
+        ///     Sets the hat of the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="hatType">An hat for the player.</param>
@@ -54,7 +54,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SetHatAsync(HatType hatType);
 
         /// <summary>
-        ///     Sets the pet of the current <see cref="IInnerPlayerControl"/>.
+        ///     Sets the pet of the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="petType">A pet for the player.</param>
@@ -62,7 +62,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SetPetAsync(PetType petType);
 
         /// <summary>
-        ///     Sets the skin of the current <see cref="IInnerPlayerControl"/>.
+        ///     Sets the skin of the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="skinType">A skin for the player.</param>
@@ -70,7 +70,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SetSkinAsync(SkinType skinType);
 
         /// <summary>
-        ///     Send a chat message as the current <see cref="IInnerPlayerControl"/>.
+        ///     Send a chat message as the current <see cref="IInnerPlayerControl" />.
         ///     Visible to all players.
         /// </summary>
         /// <param name="text">The message to send.</param>
@@ -78,7 +78,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SendChatAsync(string text);
 
         /// <summary>
-        ///     Send a chat message as the current <see cref="IInnerPlayerControl"/>.
+        ///     Send a chat message as the current <see cref="IInnerPlayerControl" />.
         ///     Visible to only the current.
         /// </summary>
         /// <param name="text">The message to send.</param>
@@ -90,7 +90,7 @@ namespace Impostor.Api.Net.Inner.Objects
         ValueTask SendChatToPlayerAsync(string text, IInnerPlayerControl? player = null);
 
         /// <summary>
-        ///     Murder <paramref name="target"/> player.
+        ///     Murder <paramref name="target" /> player.
         /// </summary>
         /// <param name="target">Target player to murder.</param>
         /// <exception cref="ImpostorProtocolException">Thrown when player is not the impostor.</exception>

src/Impostor.Api/Net/Inner/Objects/ITaskInfo.cs

@@ -1,5 +1,4 @@
 using Impostor.Api.Innersloth;
-using Impostor.Api.Net.Messages;
 
 namespace Impostor.Api.Net.Inner.Objects
 {

src/Impostor.Api/Net/LimboStates.cs

@@ -10,4 +10,4 @@ namespace Impostor.Api.Net
         WaitingForHost = 4,
         All = PreSpawn | NotLimbo | WaitingForHost,
     }
-}
+}

src/Impostor.Api/Net/Manager/IClientManager.cs

@@ -6,4 +6,4 @@ namespace Impostor.Api.Net.Manager
     {
         IEnumerable<IClient> Clients { get; }
     }
-}
+}

src/Impostor.Api/Net/Messages/C2S/Message00HostGameC2S.cs

@@ -14,9 +14,9 @@ namespace Impostor.Api.Net.Messages.C2S
         /// <summary>
         ///     Deserialize a packet.
         /// </summary>
-        /// <param name="reader"><see cref="IMessageReader"/> with <see cref="IMessageReader.Tag"/> 0.</param>
+        /// <param name="reader"><see cref="IMessageReader" /> with <see cref="IMessageReader.Tag" /> 0.</param>
         /// <param name="chatType">The chat type selected in the client of the player.</param>
-        /// <returns>Deserialized <see cref="GameOptionsData"/>.</returns>
+        /// <returns>Deserialized <see cref="GameOptionsData" />.</returns>
         public static GameOptionsData Deserialize(IMessageReader reader, out ChatType chatType)
         {
             var gameOptionsData = GameOptionsData.DeserializeCreate(reader);

src/Impostor.Api/Net/Messages/C2S/Message01JoinGameC2S.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Games;
+using System;
+using Impostor.Api.Games;
 
 namespace Impostor.Api.Net.Messages.C2S
 {
@@ -6,7 +7,7 @@ namespace Impostor.Api.Net.Messages.C2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out GameCode gameCode)

src/Impostor.Api/Net/Messages/C2S/Message04RemovePlayerC2S.cs

@@ -1,10 +1,12 @@
-namespace Impostor.Api.Net.Messages.C2S
+using System;
+
+namespace Impostor.Api.Net.Messages.C2S
 {
     public class Message04RemovePlayerC2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out int playerId, out byte reason)
@@ -13,4 +15,4 @@
             reason = reader.ReadByte();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/C2S/Message08EndGameC2S.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Innersloth;
+using System;
+using Impostor.Api.Innersloth;
 
 namespace Impostor.Api.Net.Messages.C2S
 {
@@ -6,7 +7,7 @@ namespace Impostor.Api.Net.Messages.C2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out GameOverReason gameOverReason)
@@ -15,4 +16,4 @@ namespace Impostor.Api.Net.Messages.C2S
             reader.ReadBoolean(); // showAd
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/C2S/Message10AlterGameC2S.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Innersloth;
+using System;
+using Impostor.Api.Innersloth;
 
 namespace Impostor.Api.Net.Messages.C2S
 {
@@ -6,7 +7,7 @@ namespace Impostor.Api.Net.Messages.C2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out AlterGameTags gameTag, out bool isPublic)
@@ -17,4 +18,4 @@ namespace Impostor.Api.Net.Messages.C2S
             isPublic = slice.ReadBoolean();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/C2S/Message11KickPlayerC2S.cs

@@ -1,10 +1,12 @@
-namespace Impostor.Api.Net.Messages.C2S
+using System;
+
+namespace Impostor.Api.Net.Messages.C2S
 {
     public class Message11KickPlayerC2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out int playerId, out bool isBan)
@@ -13,4 +15,4 @@
             isBan = reader.ReadBoolean();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/C2S/Message16GetGameListC2S.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Innersloth;
+using System;
+using Impostor.Api.Innersloth;
 
 namespace Impostor.Api.Net.Messages.C2S
 {
@@ -6,7 +7,7 @@ namespace Impostor.Api.Net.Messages.C2S
     {
         public static void Serialize(IMessageWriter writer)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
 
         public static void Deserialize(IMessageReader reader, out GameOptionsData options, out ChatType chatType)

src/Impostor.Api/Net/Messages/IMessageReader.cs

@@ -18,7 +18,7 @@ namespace Impostor.Api.Net.Messages
         byte[] Buffer { get; }
 
         /// <summary>
-        ///     Gets the offset of our current <see cref="IMessageReader"/> in the entire <see cref="Buffer"/>.
+        ///     Gets the offset of our current <see cref="IMessageReader" /> in the entire <see cref="Buffer" />.
         /// </summary>
         int Offset { get; }
 

src/Impostor.Api/Net/Messages/IMessageWriter.cs

@@ -74,7 +74,7 @@ namespace Impostor.Api.Net.Messages
         void Write(string value);
 
         /// <summary>
-        ///     Writes a <see cref="IPAddress"/> to the message.
+        ///     Writes a <see cref="IPAddress" /> to the message.
         /// </summary>
         /// <param name="value">Value to write.</param>
         void Write(IPAddress value);

src/Impostor.Api/Net/Messages/IMessageWriterProvider.cs

@@ -3,14 +3,14 @@
     public interface IMessageWriterProvider
     {
         /// <summary>
-        ///     Retrieves a <see cref="IMessageWriter"/> from the internal pool.
-        ///     Make sure to call <see cref="IMessageWriter.Dispose"/> when you are done!
+        ///     Retrieves a <see cref="IMessageWriter" /> from the internal pool.
+        ///     Make sure to call <see cref="IMessageWriter.Dispose" /> when you are done!
         /// </summary>
         /// <param name="sendOption">
-        ///     Whether to send the message as <see cref="MessageType.Reliable"/> or <see cref="MessageType.Unreliable"/>.
+        ///     Whether to send the message as <see cref="MessageType.Reliable" /> or <see cref="MessageType.Unreliable" />.
         ///     Reliable packets will ensure delivery while unreliable packets may be lost.
         /// </param>
-        /// <returns>A <see cref="IMessageWriter"/> from the pool.</returns>
+        /// <returns>A <see cref="IMessageWriter" /> from the pool.</returns>
         IMessageWriter Get(MessageType sendOption = MessageType.Unreliable);
     }
 }

src/Impostor.Api/Net/Messages/MessageFlags.cs

@@ -19,4 +19,4 @@
         public const byte GetGameList = 9;
         public const byte GetGameListV2 = 16;
     }
-}
+}

src/Impostor.Api/Net/Messages/MessageType.cs

@@ -29,4 +29,4 @@ namespace Impostor.Api.Net.Messages
         /// </remarks>
         Reliable,
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message00HostGameS2C.cs

@@ -17,4 +17,4 @@ namespace Impostor.Api.Net.Messages.S2C
             throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message01JoinGameS2C.cs

@@ -44,7 +44,7 @@ namespace Impostor.Api.Net.Messages.S2C
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message04RemovePlayerS2C.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Innersloth;
+using System;
+using Impostor.Api.Innersloth;
 
 namespace Impostor.Api.Net.Messages.S2C
 {
@@ -23,7 +24,7 @@ namespace Impostor.Api.Net.Messages.S2C
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message07JoinedGameS2C.cs

@@ -1,4 +1,6 @@
-namespace Impostor.Api.Net.Messages.S2C
+using System;
+
+namespace Impostor.Api.Net.Messages.S2C
 {
     public static class Message07JoinedGameS2C
     {
@@ -25,7 +27,7 @@
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message10AlterGameS2C.cs

@@ -1,4 +1,5 @@
-using Impostor.Api.Innersloth;
+using System;
+using Impostor.Api.Innersloth;
 
 namespace Impostor.Api.Net.Messages.S2C
 {
@@ -20,7 +21,7 @@ namespace Impostor.Api.Net.Messages.S2C
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message11KickPlayerS2C.cs

@@ -1,4 +1,6 @@
-namespace Impostor.Api.Net.Messages.S2C
+using System;
+
+namespace Impostor.Api.Net.Messages.S2C
 {
     public class Message11KickPlayerS2C
     {
@@ -18,7 +20,7 @@
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message12WaitForHostS2C.cs

@@ -1,4 +1,6 @@
-namespace Impostor.Api.Net.Messages.S2C
+using System;
+
+namespace Impostor.Api.Net.Messages.S2C
 {
     public class Message12WaitForHostS2C
     {
@@ -17,7 +19,7 @@
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message13RedirectS2C.cs

@@ -1,4 +1,5 @@
-using System.Net;
+using System;
+using System.Net;
 
 namespace Impostor.Api.Net.Messages.S2C
 {
@@ -19,7 +20,7 @@ namespace Impostor.Api.Net.Messages.S2C
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
-}
+}

src/Impostor.Api/Net/Messages/S2C/Message16GetGameListS2C.cs

@@ -1,4 +1,5 @@
-using System.Collections.Generic;
+using System;
+using System.Collections.Generic;
 using Impostor.Api.Games;
 
 namespace Impostor.Api.Net.Messages.S2C
@@ -33,7 +34,7 @@ namespace Impostor.Api.Net.Messages.S2C
 
         public static void Deserialize(IMessageReader reader)
         {
-            throw new System.NotImplementedException();
+            throw new NotImplementedException();
         }
     }
 }

src/Impostor.Api/Plugins/IPlugin.cs

@@ -11,4 +11,4 @@ namespace Impostor.Api.Plugins
 
         ValueTask ReloadAsync();
     }
-}
+}

src/Impostor.Api/Plugins/IPluginStartup.cs

@@ -9,4 +9,4 @@ namespace Impostor.Api.Plugins
 
         void ConfigureServices(IServiceCollection services);
     }
-}
+}

src/Impostor.Api/Plugins/ImpostorPluginAttribute.cs

@@ -21,4 +21,4 @@ namespace Impostor.Api.Plugins
 
         public string Version { get; }
     }
-}
+}

src/Impostor.Api/Plugins/PluginBase.cs

@@ -19,4 +19,4 @@ namespace Impostor.Api.Plugins
             return default;
         }
     }
-}
+}

src/Impostor.Api/Properties/AssemblyInfo.cs

@@ -1,3 +1,3 @@
 using System.Runtime.CompilerServices;
 
-[assembly:InternalsVisibleTo("Impostor.Server")]
+[assembly: InternalsVisibleTo("Impostor.Server")]

src/Impostor.Api/Reactor/PluginSide.cs

@@ -1,22 +1,22 @@
 namespace Impostor.Api.Reactor
 {
     /// <summary>
-    /// Plugin side used in modded handshake
+    ///     Plugin side used in modded handshake
     /// </summary>
     public enum PluginSide : byte
     {
         /// <summary>
-        /// Required by both sides, reject connection if missing on the other side
+        ///     Required by both sides, reject connection if missing on the other side
         /// </summary>
         Both,
 
         /// <summary>
-        /// Required only by client
+        ///     Required only by client
         /// </summary>
         ClientOnly,
 
         /// <summary>
-        /// Required only by server
+        ///     Required only by server
         /// </summary>
         ServerOnly,
     }

src/Impostor.Api/Unity/Mathf.cs

@@ -3,13 +3,13 @@
     public static class Mathf
     {
         /// <summary>
-        ///   <para>Clamps the given value between the given minimum float and maximum float values.  Returns the given value if it is within the min and max range.</para>
+        ///     <para>Clamps the given value between the given minimum float and maximum float values.  Returns the given value if it is within the min and max range.</para>
         /// </summary>
         /// <param name="value">The floating point value to restrict inside the range defined by the min and max values.</param>
         /// <param name="min">The minimum floating point value to compare against.</param>
         /// <param name="max">The maximum floating point value to compare against.</param>
         /// <returns>
-        ///   <para>The float result between the min and max values.</para>
+        ///     <para>The float result between the min and max values.</para>
         /// </returns>
         public static float Clamp(float value, float min, float max)
         {
@@ -26,7 +26,7 @@
         }
 
         /// <summary>
-        ///   <para>Clamps value between 0 and 1 and returns value.</para>
+        ///     <para>Clamps value between 0 and 1 and returns value.</para>
         /// </summary>
         /// <param name="value">Value.</param>
         /// <returns>Clamped value.</returns>
@@ -41,13 +41,13 @@
         }
 
         /// <summary>
-        ///   <para>Linearly interpolates between a and b by t.</para>
+        ///     <para>Linearly interpolates between a and b by t.</para>
         /// </summary>
         /// <param name="a">The start value.</param>
         /// <param name="b">The end value.</param>
         /// <param name="t">The interpolation value between the two floats.</param>
         /// <returns>
-        ///   <para>The interpolated float result between the two float values.</para>
+        ///     <para>The interpolated float result between the two float values.</para>
         /// </returns>
         public static float Lerp(float a, float b, float t) => a + ((b - a) * Clamp01(t));
 

src/Impostor.Benchmarks/Data/MessageReader_Bytes.cs

@@ -41,7 +41,7 @@ namespace Impostor.Benchmarks.Data
 
         public bool ReadBoolean()
         {
-            byte val = FastByte();
+            var val = FastByte();
             return val != 0;
         }
 
@@ -88,7 +88,7 @@ namespace Impostor.Benchmarks.Data
             float output = 0;
             fixed (byte* bufPtr = &this.Buffer[Position])
             {
-                byte* outPtr = (byte*)&output;
+                var outPtr = (byte*)&output;
 
                 *outPtr = *bufPtr;
                 *(outPtr + 1) = *(bufPtr + 1);
@@ -134,13 +134,13 @@ namespace Impostor.Benchmarks.Data
 
         public uint ReadPackedUInt32()
         {
-            bool readMore = true;
-            int shift = 0;
+            var readMore = true;
+            var shift = 0;
             uint output = 0;
 
             while (readMore)
             {
-                byte b = FastByte();
+                var b = FastByte();
                 if (b >= 0x80)
                 {
                     readMore = true;

src/Impostor.Benchmarks/Data/MessageReader_Bytes_Pooled.cs

@@ -65,7 +65,7 @@ namespace Impostor.Benchmarks.Data
 
         public bool ReadBoolean()
         {
-            byte val = FastByte();
+            var val = FastByte();
             return val != 0;
         }
 
@@ -112,7 +112,7 @@ namespace Impostor.Benchmarks.Data
             float output = 0;
             fixed (byte* bufPtr = &this.Buffer[Position])
             {
-                byte* outPtr = (byte*)&output;
+                var outPtr = (byte*)&output;
 
                 *outPtr = *bufPtr;
                 *(outPtr + 1) = *(bufPtr + 1);
@@ -158,13 +158,13 @@ namespace Impostor.Benchmarks.Data
 
         public uint ReadPackedUInt32()
         {
-            bool readMore = true;
-            int shift = 0;
+            var readMore = true;
+            var shift = 0;
             uint output = 0;
 
             while (readMore)
             {
-                byte b = FastByte();
+                var b = FastByte();
                 if (b >= 0x80)
                 {
                     readMore = true;

src/Impostor.Benchmarks/Data/MessageReader_Bytes_Pooled_Improved.cs

@@ -1,10 +1,6 @@
 using System;
 using System.Buffers.Binary;
-using System.Collections.Concurrent;
-using System.Collections.Generic;
-using System.IO;
 using System.Runtime.CompilerServices;
-using System.Text;
 using Microsoft.Extensions.ObjectPool;
 
 namespace Impostor.Benchmarks.Data

src/Impostor.Benchmarks/Data/MessageWriter.cs

@@ -32,7 +32,7 @@ namespace Impostor.Benchmarks.Data
         {
             if (includeHeader)
             {
-                byte[] output = new byte[this.Length];
+                var output = new byte[this.Length];
                 System.Buffer.BlockCopy(this.Buffer, 0, output, 0, this.Length);
                 return output;
             }
@@ -41,17 +41,17 @@ namespace Impostor.Benchmarks.Data
                 switch (this.SendOption)
                 {
                     case MessageType.Reliable:
-                        {
-                            byte[] output = new byte[this.Length - 3];
-                            System.Buffer.BlockCopy(this.Buffer, 3, output, 0, this.Length - 3);
-                            return output;
-                        }
+                    {
+                        var output = new byte[this.Length - 3];
+                        System.Buffer.BlockCopy(this.Buffer, 3, output, 0, this.Length - 3);
+                        return output;
+                    }
                     case MessageType.Unreliable:
-                        {
-                            byte[] output = new byte[this.Length - 1];
-                            System.Buffer.BlockCopy(this.Buffer, 1, output, 0, this.Length - 1);
-                            return output;
-                        }
+                    {
+                        var output = new byte[this.Length - 1];
+                        System.Buffer.BlockCopy(this.Buffer, 1, output, 0, this.Length - 1);
+                        return output;
+                    }
                     default:
                         throw new ArgumentOutOfRangeException();
                 }
@@ -87,7 +87,7 @@ namespace Impostor.Benchmarks.Data
         public void EndMessage()
         {
             var lastMessageStart = messageStarts.Pop();
-            ushort length = (ushort)(this.Position - lastMessageStart - 3); // Minus length and type byte
+            var length = (ushort)(this.Position - lastMessageStart - 3); // Minus length and type byte
             this.Buffer[lastMessageStart] = (byte)length;
             this.Buffer[lastMessageStart + 1] = (byte)(length >> 8);
         }
@@ -173,7 +173,7 @@ namespace Impostor.Benchmarks.Data
         {
             fixed (byte* ptr = &this.Buffer[this.Position])
             {
-                byte* valuePtr = (byte*)&value;
+                var valuePtr = (byte*)&value;
 
                 *ptr = *valuePtr;
                 *(ptr + 1) = *(valuePtr + 1);
@@ -260,7 +260,7 @@ namespace Impostor.Benchmarks.Data
         {
             do
             {
-                byte b = (byte)(value & 0xFF);
+                var b = (byte)(value & 0xFF);
                 if (value >= 0x80)
                 {
                     b |= 0x80;
@@ -275,7 +275,7 @@ namespace Impostor.Benchmarks.Data
 
         public void Write(MessageWriter msg, bool includeHeader)
         {
-            int offset = 0;
+            var offset = 0;
             if (!includeHeader)
             {
                 switch (msg.SendOption)
@@ -298,8 +298,8 @@ namespace Impostor.Benchmarks.Data
             byte b;
             unsafe
             {
-                int i = 1;
-                byte* bp = (byte*)&i;
+                var i = 1;
+                var bp = (byte*)&i;
                 b = *bp;
             }
 

src/Impostor.Benchmarks/Data/Span/MessageReader_Span.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Buffers.Binary;
-using Impostor.Hazel;
 
 namespace Impostor.Benchmarks.Data.Span
 {

src/Impostor.Benchmarks/Tests/MessageReaderBenchmark.cs

@@ -4,10 +4,8 @@ using BenchmarkDotNet.Attributes;
 using Impostor.Benchmarks.Data;
 using Impostor.Benchmarks.Data.Pool;
 using Impostor.Benchmarks.Extensions;
-using Impostor.Hazel;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.ObjectPool;
-using MessageWriter = Impostor.Benchmarks.Data.MessageWriter;
 
 namespace Impostor.Benchmarks.Tests
 {
@@ -24,7 +22,7 @@ namespace Impostor.Benchmarks.Tests
             message.StartMessage(1);
             message.Write((ushort)3100);
             message.Write((byte)100);
-            message.Write((int) int.MaxValue);
+            message.Write((int)int.MaxValue);
             message.WritePacked(int.MaxValue);
             message.EndMessage();
 

src/Impostor.Client.App/Program.cs

@@ -1,5 +1,4 @@
-using System;
-using System.Net;
+using System.Net;
 using System.Threading;
 using System.Threading.Tasks;
 using Impostor.Api.Innersloth;
@@ -31,7 +30,7 @@ namespace Impostor.Client.App
             Message00HostGameC2S.Serialize(writeGameCreate, new GameOptionsData
             {
                 MaxPlayers = 4,
-                NumImpostors = 2
+                NumImpostors = 2,
             });
 
             // TODO: ObjectPool for MessageReaders

src/Impostor.Hazel/Connection.cs

@@ -11,7 +11,7 @@ namespace Impostor.Hazel
     /// </summary>
     /// <remarks>
     ///     <para>
-    ///         Connection is the base class for all connections that Hazel can make. It provides common functionality and a 
+    ///         Connection is the base class for all connections that Hazel can make. It provides common functionality and a
     ///         standard interface to allow connections to be swapped easily.
     ///     </para>
     ///     <para>
@@ -29,7 +29,7 @@ namespace Impostor.Hazel
     ///         </list>
     ///     </para>
     /// </remarks>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public abstract class Connection : IDisposable
     {
         private static readonly ILogger Logger = Log.ForContext<Connection>();
@@ -39,34 +39,34 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         DataReceived is invoked everytime a message is received from the end point of this connection, the message 
-        ///         that was received can be found in the <see cref="DataReceivedEventArgs"/> alongside other information from the 
+        ///         DataReceived is invoked everytime a message is received from the end point of this connection, the message
+        ///         that was received can be found in the <see cref="DataReceivedEventArgs" /> alongside other information from the
         ///         event.
         ///     </para>
         ///     <include file="DocInclude/common.xml" path="docs/item[@name='Event_Thread_Safety_Warning']/*" />
         /// </remarks>
         /// <example>
-        ///     <code language="C#" source="DocInclude/TcpClientExample.cs"/>
+        ///     <code language="C#" source="DocInclude/TcpClientExample.cs" />
         /// </example>
         public Func<DataReceivedEventArgs, ValueTask> DataReceived;
 
         public int TestLagMs = -1;
         public int TestDropRate = 0;
         protected int testDropCount = 0;
-        
+
         /// <summary>
         ///     Called when the end point disconnects or an error occurs.
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         Disconnected is invoked when the connection is closed due to an exception occuring or because the remote 
-        ///         end point disconnected. If it was invoked due to an exception occuring then the exception is available 
-        ///         in the <see cref="DisconnectedEventArgs"/> passed with the event.
+        ///         Disconnected is invoked when the connection is closed due to an exception occuring or because the remote
+        ///         end point disconnected. If it was invoked due to an exception occuring then the exception is available
+        ///         in the <see cref="DisconnectedEventArgs" /> passed with the event.
         ///     </para>
         ///     <include file="DocInclude/common.xml" path="docs/item[@name='Event_Thread_Safety_Warning']/*" />
         /// </remarks>
         /// <example>
-        ///     <code language="C#" source="DocInclude/TcpClientExample.cs"/>
+        ///     <code language="C#" source="DocInclude/TcpClientExample.cs" />
         /// </example>
         public Func<DisconnectedEventArgs, ValueTask> Disconnected;
 
@@ -74,8 +74,8 @@ namespace Impostor.Hazel
         ///     The remote end point of this Connection.
         /// </summary>
         /// <remarks>
-        ///     This is the end point that this connection is connected to (i.e. the other device). This returns an abstract 
-        ///     <see cref="ConnectionEndPoint"/> which can then be cast to an appropriate end point depending on the 
+        ///     This is the end point that this connection is connected to (i.e. the other device). This returns an abstract
+        ///     <see cref="ConnectionEndPoint" /> which can then be cast to an appropriate end point depending on the
         ///     connection type.
         /// </remarks>
         public IPEndPoint EndPoint { get; protected set; }
@@ -95,7 +95,7 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     All implementers should be aware that when this is set to ConnectionState.Connected it will
-        ///     release all threads that are blocked on <see cref="WaitOnConnect"/>.
+        ///     release all threads that are blocked on <see cref="WaitOnConnect" />.
         /// </remarks>
         public ConnectionState State
         {
@@ -103,7 +103,7 @@ namespace Impostor.Hazel
             {
                 return this._state;
             }
-            
+
             protected set
             {
                 this._state = value;
@@ -113,13 +113,13 @@ namespace Impostor.Hazel
 
         protected ConnectionState _state;
         protected virtual void SetState(ConnectionState state) { }
-        
+
         /// <summary>
         ///     Constructor that initializes the ConnecitonStatistics object.
         /// </summary>
         /// <remarks>
-        ///     This constructor initialises <see cref="Statistics"/> with empty statistics and sets <see cref="State"/> to 
-        ///     <see cref="ConnectionState.NotConnected"/>.
+        ///     This constructor initialises <see cref="Statistics" /> with empty statistics and sets <see cref="State" /> to
+        ///     <see cref="ConnectionState.NotConnected" />.
         /// </remarks>
         protected Connection()
         {
@@ -128,21 +128,21 @@ namespace Impostor.Hazel
         }
 
         /// <summary>
-        ///     Sends a number of bytes to the end point of the connection using the specified <see cref="MessageType"/>.
+        ///     Sends a number of bytes to the end point of the connection using the specified <see cref="MessageType" />.
         /// </summary>
         /// <param name="msg">The message to send.</param>
         /// <remarks>
         ///     <include file="DocInclude/common.xml" path="docs/item[@name='Connection_SendBytes_General']/*" />
         ///     <para>
         ///         The messageType parameter is only a request to use those options and the actual method used to send the
-        ///         data is up to the implementation. There are circumstances where this parameter may be ignored but in 
+        ///         data is up to the implementation. There are circumstances where this parameter may be ignored but in
         ///         general any implementer should aim to always follow the user's request.
         ///     </para>
         /// </remarks>
         public abstract ValueTask SendAsync(IMessageWriter msg);
 
         /// <summary>
-        ///     Sends a number of bytes to the end point of the connection using the specified <see cref="MessageType"/>.
+        ///     Sends a number of bytes to the end point of the connection using the specified <see cref="MessageType" />.
         /// </summary>
         /// <param name="bytes">The bytes of the message to send.</param>
         /// <param name="messageType">The option specifying how the message should be sent.</param>
@@ -150,7 +150,7 @@ namespace Impostor.Hazel
         ///     <include file="DocInclude/common.xml" path="docs/item[@name='Connection_SendBytes_General']/*" />
         ///     <para>
         ///         The messageType parameter is only a request to use those options and the actual method used to send the
-        ///         data is up to the implementation. There are circumstances where this parameter may be ignored but in 
+        ///         data is up to the implementation. There are circumstances where this parameter may be ignored but in
         ///         general any implementer should aim to always follow the user's request.
         ///     </para>
         /// </remarks>
@@ -167,9 +167,9 @@ namespace Impostor.Hazel
         ///     Invokes the DataReceived event.
         /// </summary>
         /// <param name="msg">The bytes received.</param>
-        /// <param name="messageType">The <see cref="MessageType"/> the message was received with.</param>
+        /// <param name="messageType">The <see cref="MessageType" /> the message was received with.</param>
         /// <remarks>
-        ///     Invokes the <see cref="DataReceived"/> event on this connection to alert subscribers a new message has been
+        ///     Invokes the <see cref="DataReceived" /> event on this connection to alert subscribers a new message has been
         ///     received. The bytes and the send option that the message was sent with should be passed in to give to the
         ///     subscribers.
         /// </remarks>
@@ -197,8 +197,8 @@ namespace Impostor.Hazel
         /// <param name="e">The exception, if any, that occurred to cause this.</param>
         /// <param name="reader">Extra disconnect data</param>
         /// <remarks>
-        ///     Invokes the <see cref="Disconnected"/> event to alert subscribres this connection has been disconnected either 
-        ///     by the end point or because an error occurred. If an error occurred the error should be passed in in order to 
+        ///     Invokes the <see cref="Disconnected" /> event to alert subscribres this connection has been disconnected either
+        ///     by the end point or because an error occurred. If an error occurred the error should be passed in in order to
         ///     pass to the subscribers, otherwise null can be passed in.
         /// </remarks>
         protected async ValueTask InvokeDisconnected(string e, IMessageReader reader)
@@ -219,11 +219,11 @@ namespace Impostor.Hazel
         }
 
         /// <summary>
-        /// For times when you want to force the disconnect handler to fire as well as close it.
-        /// If you only want to close it, just use Dispose.
+        ///     For times when you want to force the disconnect handler to fire as well as close it.
+        ///     If you only want to close it, just use Dispose.
         /// </summary>
         public abstract ValueTask Disconnect(string reason, MessageWriter writer = null);
-        
+
         /// <summary>
         ///     Disposes of this NetworkConnection.
         /// </summary>

src/Impostor.Hazel/ConnectionListener.cs

@@ -10,16 +10,16 @@ namespace Impostor.Hazel
     /// </summary>
     /// <remarks>
     ///     <para>
-    ///         ConnectionListeners are server side objects that listen for clients and create matching server side connections 
+    ///         ConnectionListeners are server side objects that listen for clients and create matching server side connections
     ///         for each client in a similar way to TCP does. These connections should be ready for communication immediately.
     ///     </para>
     ///     <para>
-    ///         Each time a client connects the <see cref="NewConnection"/> event will be invoked to alert all subscribers to
-    ///         the new connection. A disconnected event is then present on the <see cref="Connection"/> that is passed to the
+    ///         Each time a client connects the <see cref="NewConnection" /> event will be invoked to alert all subscribers to
+    ///         the new connection. A disconnected event is then present on the <see cref="Connection" /> that is passed to the
     ///         subscribers.
     ///     </para>
     /// </remarks>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public abstract class ConnectionListener : IAsyncDisposable
     {
         private static readonly ILogger Logger = Log.ForContext<ConnectionListener>();
@@ -29,18 +29,18 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         NewConnection is invoked each time a client connects to the listener. The 
-        ///         <see cref="NewConnectionEventArgs"/> contains the new <see cref="Connection"/> for communication with this
+        ///         NewConnection is invoked each time a client connects to the listener. The
+        ///         <see cref="NewConnectionEventArgs" /> contains the new <see cref="Connection" /> for communication with this
         ///         client.
         ///     </para>
         ///     <para>
-        ///         Hazel may or may not store connections so it is your responsibility to keep track and properly Dispose of 
-        ///         connections to your server. 
+        ///         Hazel may or may not store connections so it is your responsibility to keep track and properly Dispose of
+        ///         connections to your server.
         ///     </para>
         ///     <include file="DocInclude/common.xml" path="docs/item[@name='Event_Thread_Safety_Warning']/*" />
         /// </remarks>
         /// <example>
-        ///     <code language="C#" source="DocInclude/TcpListenerExample.cs"/>
+        ///     <code language="C#" source="DocInclude/TcpListenerExample.cs" />
         /// </example>
         public Func<NewConnectionEventArgs, ValueTask> NewConnection;
 
@@ -49,15 +49,15 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         This instructs the listener to begin listening for new clients connecting to the server. When a new client 
-        ///         connects the <see cref="NewConnection"/> event will be invoked containing the connection to the new client.
+        ///         This instructs the listener to begin listening for new clients connecting to the server. When a new client
+        ///         connects the <see cref="NewConnection" /> event will be invoked containing the connection to the new client.
         ///     </para>
         ///     <para>
-        ///         To stop listening you should call <see cref="DisposeAsync()"/>.
+        ///         To stop listening you should call <see cref="DisposeAsync()" />.
         ///     </para>
         /// </remarks>
         /// <example>
-        ///     <code language="C#" source="DocInclude/TcpListenerExample.cs"/>
+        ///     <code language="C#" source="DocInclude/TcpListenerExample.cs" />
         /// </example>
         public abstract Task StartAsync();
 
@@ -67,7 +67,7 @@ namespace Impostor.Hazel
         /// <param name="msg">The user sent bytes that were received as part of the handshake.</param>
         /// <param name="connection">The connection to pass in the arguments.</param>
         /// <remarks>
-        ///     Implementers should call this to invoke the <see cref="NewConnection"/> event before data is received so that
+        ///     Implementers should call this to invoke the <see cref="NewConnection" /> event before data is received so that
         ///     subscribers do not miss any data that may have been sent immediately after connecting.
         /// </remarks>
         internal async Task InvokeNewConnection(IMessageReader msg, Connection connection)

src/Impostor.Hazel/ConnectionState.cs

@@ -1,7 +1,7 @@
 namespace Impostor.Hazel
 {
     /// <summary>
-    ///     Represents the state a <see cref="Connection"/> is currently in.
+    ///     Represents the state a <see cref="Connection" /> is currently in.
     /// </summary>
     public enum ConnectionState
     {
@@ -9,7 +9,7 @@
         ///     The Connection has either not been established yet or has been disconnected.
         /// </summary>
         NotConnected,
-        
+
         /// <summary>
         ///     The Connection is currently connecting to an endpoint.
         /// </summary>

src/Impostor.Hazel/ConnectionStatistics.cs

@@ -2,12 +2,13 @@
 using System.Threading;
 
 [assembly: InternalsVisibleTo("Hazel.Tests")]
+
 namespace Impostor.Hazel
 {
     /// <summary>
-    ///     Holds statistics about the traffic through a <see cref="Connection"/>.
+    ///     Holds statistics about the traffic through a <see cref="Connection" />.
     /// </summary>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public class ConnectionStatistics
     {
         private const int ExpectedMTU = 1200;
@@ -27,8 +28,8 @@ namespace Impostor.Hazel
         ///     The number of messages sent larger than 576 bytes. This is smaller than most default MTUs.
         /// </summary>
         /// <remarks>
-        ///     This is the number of unreliable messages that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogUnreliableSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of unreliable messages that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogUnreliableSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int FragmentableMessagesSent
@@ -48,8 +49,8 @@ namespace Impostor.Hazel
         ///     The number of unreliable messages sent.
         /// </summary>
         /// <remarks>
-        ///     This is the number of unreliable messages that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogUnreliableSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of unreliable messages that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogUnreliableSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int UnreliableMessagesSent
@@ -69,8 +70,8 @@ namespace Impostor.Hazel
         ///     The number of reliable messages sent.
         /// </summary>
         /// <remarks>
-        ///     This is the number of reliable messages that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogReliableSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of reliable messages that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogReliableSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int ReliableMessagesSent
@@ -90,8 +91,8 @@ namespace Impostor.Hazel
         ///     The number of fragmented messages sent.
         /// </summary>
         /// <remarks>
-        ///     This is the number of fragmented messages that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogFragmentedSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of fragmented messages that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogFragmentedSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int FragmentedMessagesSent
@@ -111,8 +112,8 @@ namespace Impostor.Hazel
         ///     The number of acknowledgement messages sent.
         /// </summary>
         /// <remarks>
-        ///     This is the number of acknowledgements that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogAcknowledgementSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of acknowledgements that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogAcknowledgementSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int AcknowledgementMessagesSent
@@ -132,8 +133,8 @@ namespace Impostor.Hazel
         ///     The number of hello messages sent.
         /// </summary>
         /// <remarks>
-        ///     This is the number of hello messages that were sent from the <see cref="Connection"/>, incremented 
-        ///     each time that LogHelloSend is called by the Connection. Messages that caused an error are not 
+        ///     This is the number of hello messages that were sent from the <see cref="Connection" />, incremented
+        ///     each time that LogHelloSend is called by the Connection. Messages that caused an error are not
         ///     counted and messages are only counted once all other operations in the send are complete.
         /// </remarks>
         public int HelloMessagesSent
@@ -154,12 +155,12 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         This is the number of bytes of data (i.e. user bytes) that were sent from the <see cref="Connection"/>, 
-        ///         accumulated each time that LogSend is called by the Connection. Messages that caused an error are not 
+        ///         This is the number of bytes of data (i.e. user bytes) that were sent from the <see cref="Connection" />,
+        ///         accumulated each time that LogSend is called by the Connection. Messages that caused an error are not
         ///         counted and messages are only counted once all other operations in the send are complete.
         ///     </para>
         ///     <para>
-        ///         For the number of bytes including protocol bytes see <see cref="TotalBytesSent"/>.
+        ///         For the number of bytes including protocol bytes see <see cref="TotalBytesSent" />.
         ///     </para>
         /// </remarks>
         public long DataBytesSent
@@ -180,13 +181,13 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         This is the total number of bytes (the data bytes plus protocol bytes) that were sent from the 
-        ///         <see cref="Connection"/>, accumulated each time that LogSend is called by the Connection. Messages that 
-        ///         caused an error are not counted and messages are only counted once all other operations in the send are 
+        ///         This is the total number of bytes (the data bytes plus protocol bytes) that were sent from the
+        ///         <see cref="Connection" />, accumulated each time that LogSend is called by the Connection. Messages that
+        ///         caused an error are not counted and messages are only counted once all other operations in the send are
         ///         complete.
         ///     </para>
         ///     <para>
-        ///         For the number of data bytes excluding protocol bytes see <see cref="DataBytesSent"/>.
+        ///         For the number of data bytes excluding protocol bytes see <see cref="DataBytesSent" />.
         ///     </para>
         /// </remarks>
         public long TotalBytesSent
@@ -212,12 +213,12 @@ namespace Impostor.Hazel
                 return UnreliableMessagesReceived + ReliableMessagesReceived + FragmentedMessagesReceived + AcknowledgementMessagesReceived + helloMessagesReceived;
             }
         }
-        
+
         /// <summary>
         ///     The number of unreliable messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of unreliable messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of unreliable messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogUnreliableReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int UnreliableMessagesReceived
@@ -237,7 +238,7 @@ namespace Impostor.Hazel
         ///     The number of reliable messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of reliable messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of reliable messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogReliableReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int ReliableMessagesReceived
@@ -257,7 +258,7 @@ namespace Impostor.Hazel
         ///     The number of fragmented messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of fragmented messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of fragmented messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogFragmentedReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int FragmentedMessagesReceived
@@ -277,7 +278,7 @@ namespace Impostor.Hazel
         ///     The number of acknowledgement messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of acknowledgement messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of acknowledgement messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogAcknowledgemntReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int AcknowledgementMessagesReceived
@@ -297,7 +298,7 @@ namespace Impostor.Hazel
         ///     The number of ping messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of hello messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of hello messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogHelloReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int PingMessagesReceived
@@ -317,7 +318,7 @@ namespace Impostor.Hazel
         ///     The number of hello messages received.
         /// </summary>
         /// <remarks>
-        ///     This is the number of hello messages that were received by the <see cref="Connection"/>, incremented
+        ///     This is the number of hello messages that were received by the <see cref="Connection" />, incremented
         ///     each time that LogHelloReceive is called by the Connection. Messages are counted before the receive event is invoked.
         /// </remarks>
         public int HelloMessagesReceived
@@ -338,12 +339,12 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         This is the number of bytes of data (i.e. user bytes) that were received by the <see cref="Connection"/>, 
+        ///         This is the number of bytes of data (i.e. user bytes) that were received by the <see cref="Connection" />,
         ///         accumulated each time that LogReceive is called by the Connection. Messages are counted before the receive
         ///         event is invoked.
         ///     </para>
         ///     <para>
-        ///         For the number of bytes including protocol bytes see <see cref="TotalBytesReceived"/>.
+        ///         For the number of bytes including protocol bytes see <see cref="TotalBytesReceived" />.
         ///     </para>
         /// </remarks>
         public long DataBytesReceived
@@ -364,12 +365,12 @@ namespace Impostor.Hazel
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         This is the total number of bytes (the data bytes plus protocol bytes) that were received by the 
-        ///         <see cref="Connection"/>, accumulated each time that LogReceive is called by the Connection. Messages are 
+        ///         This is the total number of bytes (the data bytes plus protocol bytes) that were received by the
+        ///         <see cref="Connection" />, accumulated each time that LogReceive is called by the Connection. Messages are
         ///         counted before the receive event is invoked.
         ///     </para>
         ///     <para>
-        ///         For the number of data bytes excluding protocol bytes see <see cref="DataBytesReceived"/>.
+        ///         For the number of data bytes excluding protocol bytes see <see cref="DataBytesReceived" />.
         ///     </para>
         /// </remarks>
         public long TotalBytesReceived

src/Impostor.Hazel/DataReceivedEventArgs.cs

@@ -12,10 +12,10 @@ namespace Impostor.Hazel
         public readonly IMessageReader Message;
 
         /// <summary>
-        ///     The <see cref="Type"/> the data was sent with.
+        ///     The <see cref="Type" /> the data was sent with.
         /// </summary>
         public readonly MessageType Type;
-        
+
         public DataReceivedEventArgs(Connection sender, IMessageReader msg, MessageType type)
         {
             this.Sender = sender;

src/Impostor.Hazel/DisconnectedEventArgs.cs

@@ -6,13 +6,13 @@ namespace Impostor.Hazel
     public class DisconnectedEventArgs : EventArgs
     {
         /// <summary>
-        /// Optional disconnect reason. May be null.
+        ///     Optional disconnect reason. May be null.
         /// </summary>
         public readonly string Reason;
 
         /// <summary>
-        /// Optional data sent with a disconnect message. May be null. 
-        /// You must not recycle this. If you need the message outside of a callback, you should copy it.
+        ///     Optional data sent with a disconnect message. May be null.
+        ///     You must not recycle this. If you need the message outside of a callback, you should copy it.
         /// </summary>
         public readonly IMessageReader Message;
 

src/Impostor.Hazel/HazelException.cs

@@ -8,14 +8,12 @@ namespace Impostor.Hazel
     [Serializable]
     public class HazelException : Exception
     {
-        internal HazelException(string msg) : base (msg)
+        internal HazelException(string msg) : base(msg)
         {
-
         }
 
-        internal HazelException(string msg, Exception e) : base (msg, e)
+        internal HazelException(string msg, Exception e) : base(msg, e)
         {
-
         }
     }
 }

src/Impostor.Hazel/IPMode.cs

@@ -4,8 +4,8 @@
     ///     Represents the IP version that a connection or listener will use.
     /// </summary>
     /// <remarks>
-    ///     If you wand a client to connect or be able to connect using IPv6 then you should use <see cref="IPv4AndIPv6"/>, 
-    ///     this sets the underlying sockets to use IPv6 but still allow IPv4 sockets to connect for backwards compatability 
+    ///     If you wand a client to connect or be able to connect using IPv6 then you should use <see cref="IPv4AndIPv6" />,
+    ///     this sets the underlying sockets to use IPv6 but still allow IPv4 sockets to connect for backwards compatability
     ///     and hence it is the default IPMode in most cases.
     /// </remarks>
     public enum IPMode
@@ -16,9 +16,9 @@
         IPv4,
 
         /// <summary>
-        ///     Instruction to use IPv6 only, IPv4 connections will not be able to connect. IPv4 addresses can be connected 
+        ///     Instruction to use IPv6 only, IPv4 connections will not be able to connect. IPv4 addresses can be connected
         ///     by converting to IPv6 addresses.
         /// </summary>
-        IPv6
+        IPv6,
     }
 }

src/Impostor.Hazel/IRecyclable.cs

@@ -3,7 +3,7 @@
     /// <summary>
     ///     Interface for all items that can be returned to an object pool.
     /// </summary>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public interface IRecyclable
     {
         /// <summary>

src/Impostor.Hazel/MessageReader.cs

@@ -78,7 +78,7 @@ namespace Impostor.Hazel
 
         public bool ReadBoolean()
         {
-            byte val = FastByte();
+            var val = FastByte();
             return val != 0;
         }
 
@@ -159,13 +159,13 @@ namespace Impostor.Hazel
 
         public uint ReadPackedUInt32()
         {
-            bool readMore = true;
-            int shift = 0;
+            var readMore = true;
+            var shift = 0;
             uint output = 0;
 
             while (readMore)
             {
-                byte b = FastByte();
+                var b = FastByte();
                 if (b >= 0x80)
                 {
                     readMore = true;
@@ -185,8 +185,8 @@ namespace Impostor.Hazel
 
         public void CopyTo(IMessageWriter writer)
         {
-            writer.Write((ushort) Length);
-            writer.Write((byte) Tag);
+            writer.Write((ushort)Length);
+            writer.Write((byte)Tag);
             writer.Write(Buffer.AsMemory(Offset, Length));
         }
 
@@ -213,7 +213,7 @@ namespace Impostor.Hazel
 
             System.Buffer.BlockCopy(Buffer, offsetEnd, Buffer, offsetStart, lengthToCopy);
 
-            ((MessageReader) message).Parent.AdjustLength(message.Offset, message.Length + 3);
+            ((MessageReader)message).Parent.AdjustLength(message.Offset, message.Length + 3);
         }
 
         private void AdjustLength(int offset, int amount)
@@ -255,9 +255,9 @@ namespace Impostor.Hazel
         public Vector2 ReadVector2()
         {
             const float range = 50f;
-            
-            var x = ReadUInt16() / (float) ushort.MaxValue;
-            var y = ReadUInt16() / (float) ushort.MaxValue;
+
+            var x = ReadUInt16() / (float)ushort.MaxValue;
+            var y = ReadUInt16() / (float)ushort.MaxValue;
 
             return new Vector2(Mathf.Lerp(-range, range, x), Mathf.Lerp(-range, range, y));
         }

src/Impostor.Hazel/MessageWriter.cs

@@ -1,12 +1,11 @@
-using Impostor.Api.Games;
-using Impostor.Api.Net.Messages;
-
-using System;
+using System;
 using System.Collections.Generic;
 using System.Net;
 using System.Numerics;
 using System.Text;
+using Impostor.Api.Games;
 using Impostor.Api.Net.Inner;
+using Impostor.Api.Net.Messages;
 using Impostor.Api.Unity;
 
 namespace Impostor.Hazel
@@ -39,7 +38,7 @@ namespace Impostor.Hazel
         {
             if (includeHeader)
             {
-                byte[] output = new byte[this.Length];
+                var output = new byte[this.Length];
                 System.Buffer.BlockCopy(this.Buffer, 0, output, 0, this.Length);
                 return output;
             }
@@ -48,17 +47,17 @@ namespace Impostor.Hazel
                 switch (this.SendOption)
                 {
                     case MessageType.Reliable:
-                        {
-                            byte[] output = new byte[this.Length - 3];
-                            System.Buffer.BlockCopy(this.Buffer, 3, output, 0, this.Length - 3);
-                            return output;
-                        }
+                    {
+                        var output = new byte[this.Length - 3];
+                        System.Buffer.BlockCopy(this.Buffer, 3, output, 0, this.Length - 3);
+                        return output;
+                    }
                     case MessageType.Unreliable:
-                        {
-                            byte[] output = new byte[this.Length - 1];
-                            System.Buffer.BlockCopy(this.Buffer, 1, output, 0, this.Length - 1);
-                            return output;
-                        }
+                    {
+                        var output = new byte[this.Length - 1];
+                        System.Buffer.BlockCopy(this.Buffer, 1, output, 0, this.Length - 1);
+                        return output;
+                    }
                     default:
                         throw new ArgumentOutOfRangeException();
                 }
@@ -67,7 +66,6 @@ namespace Impostor.Hazel
             throw new NotImplementedException();
         }
 
-        ///
         /// <param name="sendOption">The option specifying how the message should be sent.</param>
         public static MessageWriter Get(MessageType sendOption = MessageType.Unreliable)
         {
@@ -106,8 +104,8 @@ namespace Impostor.Hazel
 
         public void Write(Vector2 vector)
         {
-            Write((ushort)(Mathf.ReverseLerp(vector.X) * (double) ushort.MaxValue));
-            Write((ushort)(Mathf.ReverseLerp(vector.Y) * (double) ushort.MaxValue));
+            Write((ushort)(Mathf.ReverseLerp(vector.X) * (double)ushort.MaxValue));
+            Write((ushort)(Mathf.ReverseLerp(vector.Y) * (double)ushort.MaxValue));
         }
 
         ///
@@ -122,7 +120,7 @@ namespace Impostor.Hazel
         public void EndMessage()
         {
             var lastMessageStart = messageStarts.Pop();
-            ushort length = (ushort)(this.Position - lastMessageStart - 3); // Minus length and type byte
+            var length = (ushort)(this.Position - lastMessageStart - 3); // Minus length and type byte
             this.Buffer[lastMessageStart] = (byte)length;
             this.Buffer[lastMessageStart + 1] = (byte)(length >> 8);
         }
@@ -215,7 +213,7 @@ namespace Impostor.Hazel
         {
             fixed (byte* ptr = &this.Buffer[this.Position])
             {
-                byte* valuePtr = (byte*)&value;
+                var valuePtr = (byte*)&value;
 
                 *ptr = *valuePtr;
                 *(ptr + 1) = *(valuePtr + 1);
@@ -302,7 +300,7 @@ namespace Impostor.Hazel
         {
             do
             {
-                byte b = (byte)(value & 0xFF);
+                var b = (byte)(value & 0xFF);
                 if (value >= 0x80)
                 {
                     b |= 0x80;
@@ -317,7 +315,7 @@ namespace Impostor.Hazel
 
         public void Write(MessageWriter msg, bool includeHeader)
         {
-            int offset = 0;
+            var offset = 0;
             if (!includeHeader)
             {
                 switch (msg.SendOption)
@@ -340,8 +338,8 @@ namespace Impostor.Hazel
             byte b;
             unsafe
             {
-                int i = 1;
-                byte* bp = (byte*)&i;
+                var i = 1;
+                var bp = (byte*)&i;
                 b = *bp;
             }
 

src/Impostor.Hazel/NetworkConnection.cs

@@ -12,17 +12,17 @@ namespace Impostor.Hazel
         ReceivedZeroBytes,
         PingsWithoutResponse,
         ReliablePacketWithoutResponse,
-        ConnectionDisconnected
+        ConnectionDisconnected,
     }
 
     /// <summary>
-    ///     Abstract base class for a <see cref="Connection"/> to a remote end point via a network protocol like TCP or UDP.
+    ///     Abstract base class for a <see cref="Connection" /> to a remote end point via a network protocol like TCP or UDP.
     /// </summary>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public abstract class NetworkConnection : Connection
     {
         /// <summary>
-        /// An event that gives us a chance to send well-formed disconnect messages to clients when an internal disconnect happens.
+        ///     An event that gives us a chance to send well-formed disconnect messages to clients when an internal disconnect happens.
         /// </summary>
         public Func<HazelInternalErrors, MessageWriter> OnInternalDisconnect;
 
@@ -30,8 +30,8 @@ namespace Impostor.Hazel
         ///     The remote end point of this connection.
         /// </summary>
         /// <remarks>
-        ///     This is the end point of the other device given as an <see cref="System.Net.EndPoint"/> rather than a generic
-        ///     <see cref="ConnectionEndPoint"/> as the base <see cref="Connection"/> does.
+        ///     This is the end point of the other device given as an <see cref="System.Net.EndPoint" /> rather than a generic
+        ///     <see cref="ConnectionEndPoint" /> as the base <see cref="Connection" /> does.
         /// </remarks>
         public IPEndPoint RemoteEndPoint { get; protected set; }
 
@@ -67,14 +67,14 @@ namespace Impostor.Hazel
         }
 
         /// <summary>
-        /// Called when socket is disconnected internally
+        ///     Called when socket is disconnected internally
         /// </summary>
         internal async ValueTask DisconnectInternal(HazelInternalErrors error, string reason)
         {
             var handler = this.OnInternalDisconnect;
             if (handler != null)
             {
-                MessageWriter messageToRemote = handler(error);
+                var messageToRemote = handler(error);
                 if (messageToRemote != null)
                 {
                     try

src/Impostor.Hazel/NetworkConnectionListener.cs

@@ -3,9 +3,9 @@
 namespace Impostor.Hazel
 {
     /// <summary>
-    ///     Abstract base class for a <see cref="ConnectionListener"/> for network based connections.
+    ///     Abstract base class for a <see cref="ConnectionListener" /> for network based connections.
     /// </summary>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public abstract class NetworkConnectionListener : ConnectionListener
     {
         /// <summary>

src/Impostor.Hazel/NewConnectionEventArgs.cs

@@ -5,13 +5,13 @@ namespace Impostor.Hazel
     public struct NewConnectionEventArgs
     {
         /// <summary>
-        /// The data received from the client in the handshake.
-        /// This data is yours. Remember to recycle it.
+        ///     The data received from the client in the handshake.
+        ///     This data is yours. Remember to recycle it.
         /// </summary>
         public readonly IMessageReader HandshakeData;
 
         /// <summary>
-        /// The <see cref="Connection"/> to the new client.
+        ///     The <see cref="Connection" /> to the new client.
         /// </summary>
         public readonly Connection Connection;
 

src/Impostor.Hazel/ObjectPoolCustom.cs

@@ -8,7 +8,7 @@ namespace Impostor.Hazel
     ///     A fairly simple object pool for items that will be created a lot.
     /// </summary>
     /// <typeparam name="T">The type that is pooled.</typeparam>
-    /// <threadsafety static="true" instance="true"/>
+    /// <threadsafety static="true" instance="true" />
     public sealed class ObjectPoolCustom<T> where T : IRecyclable
     {
         private int numberCreated;
@@ -32,7 +32,7 @@ namespace Impostor.Hazel
         /// </summary>
         /// <returns></returns>
         private readonly Func<T> objectFactory;
-        
+
         /// <summary>
         ///     Internal constructor for our ObjectPool.
         /// </summary>
@@ -48,7 +48,7 @@ namespace Impostor.Hazel
         internal T GetObject()
         {
 #if HAZEL_BAG
-            if (!pool.TryTake(out T item))
+            if (!pool.TryTake(out var item))
             {
                 Interlocked.Increment(ref numberCreated);
                 item = objectFactory.Invoke();
@@ -85,7 +85,7 @@ namespace Impostor.Hazel
         /// <param name="item">The item to return.</param>
         internal void PutObject(T item)
         {
-            if (inuse.TryRemove(item, out bool b))
+            if (inuse.TryRemove(item, out var b))
             {
 #if HAZEL_BAG
                 pool.Add(item);

src/Impostor.Hazel/Udp/SendOptionInternal.cs

@@ -11,7 +11,7 @@
         Hello = 8,
 
         /// <summary>
-        /// A single byte of continued existence
+        ///     A single byte of continued existence
         /// </summary>
         Ping = 12,
 

src/Impostor.Hazel/Udp/UdpBroadcastListener.cs

@@ -89,21 +89,21 @@ namespace Impostor.Hazel.Udp
                 return;
             }
 
-            if (numBytes < 3 
+            if (numBytes < 3
                 || buffer[0] != 4 || buffer[1] != 2)
             {
                 this.StartListen();
                 return;
             }
 
-            IPEndPoint ipEnd = (IPEndPoint)endpt;
-            string data = UTF8Encoding.UTF8.GetString(buffer, 2, numBytes - 2);
-            int dataHash = data.GetHashCode();
+            var ipEnd = (IPEndPoint)endpt;
+            var data = UTF8Encoding.UTF8.GetString(buffer, 2, numBytes - 2);
+            var dataHash = data.GetHashCode();
 
             lock (packets)
             {
-                bool found = false;
-                for (int i = 0; i < this.packets.Count; ++i)
+                var found = false;
+                for (var i = 0; i < this.packets.Count; ++i)
                 {
                     var pkt = this.packets[i];
                     if (pkt == null || pkt.Data == null)
@@ -146,11 +146,17 @@ namespace Impostor.Hazel.Udp
         {
             if (this.socket != null)
             {
-                try { this.socket.Shutdown(SocketShutdown.Both); } catch { }
-                try { this.socket.Close(); } catch { }
-                try { this.socket.Dispose(); } catch { }
+                try { this.socket.Shutdown(SocketShutdown.Both); }
+                catch { }
+
+                try { this.socket.Close(); }
+                catch { }
+
+                try { this.socket.Dispose(); }
+                catch { }
+
                 this.socket = null;
             }
         }
     }
-}
+}

src/Impostor.Hazel/Udp/UdpBroadcaster.cs

@@ -26,7 +26,7 @@ namespace Impostor.Hazel.Udp
         ///
         public void SetData(string data)
         {
-            int len = UTF8Encoding.UTF8.GetByteCount(data);
+            var len = UTF8Encoding.UTF8.GetByteCount(data);
             this.data = new byte[len + 2];
             this.data[0] = 4;
             this.data[1] = 2;
@@ -69,11 +69,17 @@ namespace Impostor.Hazel.Udp
         {
             if (this.socket != null)
             {
-                try { this.socket.Shutdown(SocketShutdown.Both); } catch { }
-                try { this.socket.Close(); } catch { }
-                try { this.socket.Dispose(); } catch { }
+                try { this.socket.Shutdown(SocketShutdown.Both); }
+                catch { }
+
+                try { this.socket.Close(); }
+                catch { }
+
+                try { this.socket.Dispose(); }
+                catch { }
+
                 this.socket = null;
             }
         }
     }
-}
+}

src/Impostor.Hazel/Udp/UdpClientConnection.cs

@@ -1,9 +1,7 @@
 using System;
-using System.Buffers;
 using System.Net;
 using System.Net.Sockets;
 using System.Threading;
-using System.Threading.Channels;
 using System.Threading.Tasks;
 using Impostor.Api.Net.Messages;
 using Microsoft.Extensions.ObjectPool;
@@ -14,7 +12,7 @@ namespace Impostor.Hazel.Udp
     /// <summary>
     ///     Represents a client's connection to a server that uses the UDP protocol.
     /// </summary>
-    /// <inheritdoc/>
+    /// <inheritdoc />
     public sealed class UdpClientConnection : UdpConnection
     {
         private static readonly ILogger Logger = Log.ForContext<UdpClientConnection>();
@@ -31,7 +29,7 @@ namespace Impostor.Hazel.Udp
         /// <summary>
         ///     Creates a new UdpClientConnection.
         /// </summary>
-        /// <param name="remoteEndPoint">A <see cref="NetworkEndPoint"/> to connect to.</param>
+        /// <param name="remoteEndPoint">A <see cref="NetworkEndPoint" /> to connect to.</param>
         public UdpClientConnection(IPEndPoint remoteEndPoint, ObjectPool<MessageReader> readerPool, IPMode ipMode = IPMode.IPv4) : base(null, readerPool)
         {
             EndPoint = remoteEndPoint;
@@ -40,7 +38,7 @@ namespace Impostor.Hazel.Udp
 
             _socket = new UdpClient
             {
-                DontFragment = false
+                DontFragment = false,
             };
 
             _reliablePacketTimer = new Timer(ManageReliablePacketsInternal, null, 100, Timeout.Infinite);
@@ -213,8 +211,11 @@ namespace Impostor.Hazel.Udp
         {
             State = ConnectionState.NotConnected;
 
-            try { _socket.Close(); } catch { }
-            try { _socket.Dispose(); } catch { }
+            try { _socket.Close(); }
+            catch { }
+
+            try { _socket.Dispose(); }
+            catch { }
 
             _reliablePacketTimer.Dispose();
             _connectWaitLock.Dispose();

src/Impostor.Hazel/Udp/UdpConnection.KeepAlive.cs

@@ -8,7 +8,6 @@ namespace Impostor.Hazel.Udp
 {
     partial class UdpConnection
     {
-
         /// <summary>
         ///     Class to hold packet data
         /// </summary>
@@ -58,6 +57,7 @@ namespace Impostor.Hazel.Udp
                 ResetKeepAliveTimer();
             }
         }
+
         private int keepAliveInterval = 1500;
 
         public int MissingPingsUntilDisconnect { get; set; } = 6;
@@ -109,9 +109,9 @@ namespace Impostor.Hazel.Udp
         // pings should cause a disconnect.
         private async ValueTask SendPing()
         {
-            ushort id = (ushort)Interlocked.Increment(ref lastIDAllocated);
+            var id = (ushort)Interlocked.Increment(ref lastIDAllocated);
 
-            byte[] bytes = new byte[3];
+            var bytes = new byte[3];
             bytes[0] = (byte)UdpSendOption.Ping;
             bytes[1] = (byte)(id >> 8);
             bytes[2] = (byte)id;
@@ -164,4 +164,4 @@ namespace Impostor.Hazel.Udp
             }
         }
     }
-}
+}

src/Impostor.Hazel/Udp/UdpConnection.Reliable.cs

@@ -15,25 +15,25 @@ namespace Impostor.Hazel.Udp
         /// </summary>
         /// <remarks>
         ///     <para>
-        ///         For reliable delivery data is resent at specified intervals unless an acknowledgement is received from the 
+        ///         For reliable delivery data is resent at specified intervals unless an acknowledgement is received from the
         ///         receiving device. The ResendTimeout specifies the interval between the packets being resent, each time a packet
-        ///         is resent the interval is increased for that packet until the duration exceeds the <see cref="DisconnectTimeout"/> value.
+        ///         is resent the interval is increased for that packet until the duration exceeds the <see cref="DisconnectTimeout" /> value.
         ///     </para>
         ///     <para>
-        ///         Setting this to its default of 0 will mean the timeout is 2 times the value of the average ping, usually 
+        ///         Setting this to its default of 0 will mean the timeout is 2 times the value of the average ping, usually
         ///         resulting in a more dynamic resend that responds to endpoints on slower or faster connections.
         ///     </para>
         /// </remarks>
         public volatile int ResendTimeout = 0;
 
         /// <summary>
-        /// Max number of times to resend. 0 == no limit
+        ///     Max number of times to resend. 0 == no limit
         /// </summary>
         public volatile int ResendLimit = 0;
 
         /// <summary>
-        /// A compounding multiplier to back off resend timeout.
-        /// Applied to ping before first timeout when ResendTimeout == 0.
+        ///     A compounding multiplier to back off resend timeout.
+        ///     Applied to ping before first timeout when ResendTimeout == 0.
         /// </summary>
         public volatile float ResendPingMultiplier = 2;
 
@@ -48,7 +48,7 @@ namespace Impostor.Hazel.Udp
         internal ConcurrentDictionary<ushort, Packet> reliableDataPacketsSent = new ConcurrentDictionary<ushort, Packet>();
 
         /// <summary>
-        ///     Packet ids that have not been received, but are expected. 
+        ///     Packet ids that have not been received, but are expected.
         /// </summary>
         private HashSet<ushort> reliableDataPacketsMissing = new HashSet<ushort>();
 
@@ -63,7 +63,7 @@ namespace Impostor.Hazel.Udp
         ///     Returns the average ping to this endpoint.
         /// </summary>
         /// <remarks>
-        ///     This returns the average ping for a one-way trip as calculated from the reliable packets that have been sent 
+        ///     This returns the average ping for a one-way trip as calculated from the reliable packets that have been sent
         ///     and acknowledged by the endpoint.
         /// </remarks>
         public float AveragePingMs = 500;
@@ -72,7 +72,7 @@ namespace Impostor.Hazel.Udp
         ///     The maximum times a message should be resent before marking the endpoint as disconnected.
         /// </summary>
         /// <remarks>
-        ///     Reliable packets will be resent at an interval defined in <see cref="ResendTimeout"/> for the number of times
+        ///     Reliable packets will be resent at an interval defined in <see cref="ResendTimeout" /> for the number of times
         ///     specified here. Once a packet has been retransmitted this number of times and has not been acknowledged the
         ///     connection will be marked as disconnected and the <see cref="Connection.Disconnected">Disconnected</see> event
         ///     will be invoked.
@@ -136,10 +136,10 @@ namespace Impostor.Hazel.Udp
                 var connection = this.Connection;
                 if (!this.Acknowledged && connection != null)
                 {
-                    long lifetime = this.Stopwatch.ElapsedMilliseconds;
+                    var lifetime = this.Stopwatch.ElapsedMilliseconds;
                     if (lifetime >= connection.DisconnectTimeout)
                     {
-                        if (connection.reliableDataPacketsSent.TryRemove(this.Id, out Packet self))
+                        if (connection.reliableDataPacketsSent.TryRemove(this.Id, out var self))
                         {
                             await connection.DisconnectInternal(HazelInternalErrors.ReliablePacketWithoutResponse, $"Reliable packet {self.Id} (size={this.Length}) was not ack'd after {lifetime}ms ({self.Retransmissions} resends)");
 
@@ -155,7 +155,7 @@ namespace Impostor.Hazel.Udp
                         if (connection.ResendLimit != 0
                             && this.Retransmissions > connection.ResendLimit)
                         {
-                            if (connection.reliableDataPacketsSent.TryRemove(this.Id, out Packet self))
+                            if (connection.reliableDataPacketsSent.TryRemove(this.Id, out var self))
                             {
                                 await connection.DisconnectInternal(HazelInternalErrors.ReliablePacketWithoutResponse, $"Reliable packet {self.Id} (size={this.Length}) was not ack'd after {self.Retransmissions} resends ({lifetime}ms)");
 
@@ -196,12 +196,12 @@ namespace Impostor.Hazel.Udp
 
         internal async ValueTask<int> ManageReliablePackets()
         {
-            int output = 0;
+            var output = 0;
             if (this.reliableDataPacketsSent.Count > 0)
             {
                 foreach (var kvp in this.reliableDataPacketsSent)
                 {
-                    Packet pkt = kvp.Value;
+                    var pkt = kvp.Value;
 
                     try
                     {
@@ -222,12 +222,12 @@ namespace Impostor.Hazel.Udp
         /// <param name="ackCallback">The callback to make once the packet has been acknowledged.</param>
         protected void AttachReliableID(byte[] buffer, int offset, int sendLength, Action ackCallback = null)
         {
-            ushort id = (ushort)Interlocked.Increment(ref lastIDAllocated);
+            var id = (ushort)Interlocked.Increment(ref lastIDAllocated);
 
             buffer[offset] = (byte)(id >> 8);
             buffer[offset + 1] = (byte)id;
 
-            Packet packet = Packet.GetObject();
+            var packet = Packet.GetObject();
             packet.Set(
                 id,
                 this,
@@ -260,7 +260,7 @@ namespace Impostor.Hazel.Udp
             //Inform keepalive not to send for a while
             ResetKeepAliveTimer();
 
-            byte[] bytes = new byte[data.Length + 3];
+            var bytes = new byte[data.Length + 3];
 
             //Add message type
             bytes[0] = sendOption;
@@ -333,35 +333,35 @@ namespace Impostor.Hazel.Udp
              * 
              * So...
              */
-            
+
             var result = true;
 
             lock (reliableDataPacketsMissing)
             {
                 //Calculate overwritePointer
-                ushort overwritePointer = (ushort)(reliableReceiveLast - 32768);
+                var overwritePointer = (ushort)(reliableReceiveLast - 32768);
 
                 //Calculate if it is a new packet by examining if it is within the range
                 bool isNew;
                 if (overwritePointer < reliableReceiveLast)
-                    isNew = id > reliableReceiveLast || id <= overwritePointer;     //Figure (2)
+                    isNew = id > reliableReceiveLast || id <= overwritePointer; //Figure (2)
                 else
-                    isNew = id > reliableReceiveLast && id <= overwritePointer;     //Figure (3)
-                
+                    isNew = id > reliableReceiveLast && id <= overwritePointer; //Figure (3)
+
                 //If it's new or we've not received anything yet
                 if (isNew)
                 {
                     // Mark items between the most recent receive and the id received as missing
                     if (id > reliableReceiveLast)
                     {
-                        for (ushort i = (ushort)(reliableReceiveLast + 1); i < id; i++)
+                        for (var i = (ushort)(reliableReceiveLast + 1); i < id; i++)
                         {
                             reliableDataPacketsMissing.Add(i);
                         }
                     }
                     else
                     {
-                        int cnt = (ushort.MaxValue - reliableReceiveLast) + id;
+                        var cnt = (ushort.MaxValue - reliableReceiveLast) + id;
                         for (ushort i = 1; i < cnt; ++i)
                         {
                             reliableDataPacketsMissing.Add((ushort)(i + reliableReceiveLast));
@@ -371,7 +371,7 @@ namespace Impostor.Hazel.Udp
                     //Update the most recently received
                     reliableReceiveLast = id;
                 }
-                
+
                 //Else it could be a missing packet
                 else
                 {
@@ -382,7 +382,7 @@ namespace Impostor.Hazel.Udp
                     }
                 }
             }
-            
+
             //Send an acknowledgement
             await SendAck(id);
 
@@ -397,13 +397,13 @@ namespace Impostor.Hazel.Udp
         {
             this.pingsSinceAck = 0;
 
-            ushort id = (ushort)((bytes[1] << 8) + bytes[2]);
+            var id = (ushort)((bytes[1] << 8) + bytes[2]);
             AcknowledgeMessageId(id);
 
             if (bytes.Length == 4)
             {
-                byte recentPackets = bytes[3];
-                for (int i = 1; i <= 8; ++i)
+                var recentPackets = bytes[3];
+                for (var i = 1; i <= 8; ++i)
                 {
                     if ((recentPackets & 1) != 0)
                     {
@@ -420,7 +420,7 @@ namespace Impostor.Hazel.Udp
         private void AcknowledgeMessageId(ushort id)
         {
             // Dispose of timer and remove from dictionary
-            if (reliableDataPacketsSent.TryRemove(id, out Packet packet))
+            if (reliableDataPacketsSent.TryRemove(id, out var packet))
             {
                 float rt = packet.Stopwatch.ElapsedMilliseconds;
 
@@ -432,7 +432,7 @@ namespace Impostor.Hazel.Udp
                     this.AveragePingMs = Math.Max(50, this.AveragePingMs * .7f + rt * .3f);
                 }
             }
-            else if (this.activePingPackets.TryRemove(id, out PingPacket pingPkt))
+            else if (this.activePingPackets.TryRemove(id, out var pingPkt))
             {
                 float rt = pingPkt.Stopwatch.ElapsedMilliseconds;
 
@@ -455,7 +455,7 @@ namespace Impostor.Hazel.Udp
             byte recentPackets = 0;
             lock (this.reliableDataPacketsMissing)
             {
-                for (int i = 1; i <= 8; ++i)
+                for (var i = 1; i <= 8; ++i)
                 {
                     if (!this.reliableDataPacketsMissing.Contains((ushort)(id - i)))
                     {
@@ -464,12 +464,12 @@ namespace Impostor.Hazel.Udp
                 }
             }
 
-            byte[] bytes = new byte[]
+            var bytes = new byte[]
             {
                 (byte)UdpSendOption.Acknowledgement,
                 (byte)(id >> 8),
                 (byte)(id >> 0),
-                recentPackets
+                recentPackets,
             };
 
             try