Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Palit <[email protected]>

Author: Sh4d0v

Intersect.Client.Core/Interface/Game/Admin/BanMuteBox.cs

@@ -24,7 +24,7 @@ public partial class BanMuteBox : WindowControl
         (Strings.BanMute.TwoMonths, 60),
         (Strings.BanMute.SixMonths, 180),
         (Strings.BanMute.OneYear, 365),
-        (Strings.BanMute.Forever, int.MaxValue),
+        (Strings.BanMute.Forever, 999999),
     };
 
     public BanMuteBox(string title, string prompt, EventHandler okayHandler) : base(

Intersect.Client.Framework/Gwen/Control/Base.cs

@@ -1301,12 +1301,12 @@ public void Dispose()
             return;
         }
 
-        // I think its only used for diagnosis, so Im adding the DEBUG condition. I hope so
+        // This stack trace is only used for diagnostics, so it is included only in DEBUG builds.
 #if DEBUG
         _disposeStack = new StackTrace(fNeedFileInfo: true);
 #endif
 
-        // We mark that Dispose() has been called to prevents Dispose from being executed again
+        // We mark that Dispose() has been called to prevent Dispose from being executed again
         _disposed = true;
 
         // Remove all pending tasks from the thread queue of this control
@@ -1315,7 +1315,7 @@ public void Dispose()
         // Virtual Dispose() so that inherited classes can release their resources
         Dispose(disposing: true);
 
-        // Reserve an assistant variable for the texture caching mechanism
+        // Reserve a temporary variable for the texture caching mechanism
         ICacheToTexture? cache = default;
 
 #pragma warning disable CA1031 // Do not catch general exception types