Merge #4071 Omit duplicate inflation warnings in queue

KSP-CKAN · Mar 31, 2024 · adbe192 · adbe192
2 parents 76318a5 + a1ced32
commit adbe192
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -45,6 +45,7 @@ All notable changes to this project will be documented in this file.
 - [Core] Get KSP2 version from game assembly (#4034 by: HebaruSan)
 - [Multiple] Build nuget package, support netstandard2.0 build (#4039 by: HebaruSan)
 - [Core] Use fully sanitized archive.org bucket names (#4043 by: HebaruSan)
+- [Netkan] Omit duplicate inflation warnings in queue (#4071 by: HebaruSan)
 
 ## v1.34.4 (Niven)
 

diff --git a/Netkan/QueueAppender.cs b/Netkan/QueueAppender.cs
@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+
 using log4net.Core;
 using log4net.Appender;
 
@@ -10,9 +11,13 @@ public class QueueAppender : AppenderSkeleton
 
         protected override void Append(LoggingEvent evt)
         {
-            Warnings.Add(evt.RenderedMessage);
+            // Skip duplicate messages for better multi-kref handling
+            if (!Warnings.Contains(evt.RenderedMessage))
+            {
+                Warnings.Add(evt.RenderedMessage);
+            }
         }
 
-        public List<string> Warnings = new List<string>();
+        public readonly List<string> Warnings = new List<string>();
     }
 }