Removed performance TODOs about string -> bytes conversion.

BattletechModders · Feb 8, 2025 · da7c6b8 · da7c6b8
1 parent e7b20f0
commit da7c6b8
Showing 1 changed file with 0 additions and 4 deletions.
diff --git a/ModTek/Features/Logging/Formatter.cs b/ModTek/Features/Logging/Formatter.cs
@@ -46,8 +46,6 @@ internal void SerializeMessage(ref MTLoggerMessageDto messageDto, FastBuffer buf
             buffer.Append(s_threadIdSuffix);
         }
 
-        // TODO create injector and add a nameAsBytes field that should be passed along instead of string
-        //  should improve performance by 20% since string/char[] -> byte[] is slow
         buffer.Append(messageDto.LoggerName);
 
         buffer.Append(LogLevelExtension.GetFormattedBytes(messageDto.LogLevel));
@@ -63,8 +61,6 @@ internal void SerializeMessage(ref MTLoggerMessageDto messageDto, FastBuffer buf
         if (messageDto.Exception != null)
         {
             buffer.Append(prefix);
-            // TODO find a faster (pinvoke?) method for conversion
-            //  this takes like 60-80% of the formatting time
             buffer.Append(messageDto.Exception.ToString());
             prefix = s_environmentNewline;
         }