perf: Added option to disable verbose logs

MidLevel · Sep 3, 2019 · 61a7b44 · 61a7b44
1 parent 4603b15
commit 61a7b44
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/Ruffles/Configuration/SocketConfig.cs b/Ruffles/Configuration/SocketConfig.cs
@@ -6,7 +6,13 @@
 namespace Ruffles.Configuration
 {
     public class SocketConfig
-    {
+    {
+        // Random
+        /// <summary>
+        /// Whether or not to use verbose info logging. This can cause tons of garbage allocation.
+        /// </summary>
+        public bool UseVerboseInfoLogging = false;
+
         // Connection
         /// <summary>
         /// The IPv4 address the socket will listen on.

diff --git a/Ruffles/Core/RuffleSocket.cs b/Ruffles/Core/RuffleSocket.cs
@@ -665,8 +665,11 @@ internal void HandlePacket(ArraySegment<byte> payload, EndPoint endpoint)
 
             // Unpack header, dont cast to MessageType enum for safety
             HeaderPacker.Unpack(payload.Array[payload.Offset], out byte messageType, out bool fragmented);
-
-            Logging.Info("Unpacked packet. [MessageType=" + (MessageType)messageType + "] [Fragmented=" + fragmented + "]");
+
+            if (config.UseVerboseInfoLogging)
+            {
+                Logging.Info("Unpacked packet. [MessageType=" + (MessageType)messageType + "] [Fragmented=" + fragmented + "]");
+            }
 
             switch (messageType)
             {