From 5e8f01f6221227db70e090c291d23475c7a90a2b Mon Sep 17 00:00:00 2001
From: Simon Cropp <simon.cropp@gmail.com>
Date: Thu, 9 Apr 2026 15:17:59 +1000
Subject: [PATCH] use HashSet for usedSnippets  and usedIncludes

---
 .../Processing/IncludeProcessor.cs                 |  6 +++---
 .../Processing/MarkdownProcessor.cs                | 14 +++++++-------
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/MarkdownSnippets/Processing/IncludeProcessor.cs b/src/MarkdownSnippets/Processing/IncludeProcessor.cs
index 6b77ecf2..59a2bda0 100644
--- a/src/MarkdownSnippets/Processing/IncludeProcessor.cs
+++ b/src/MarkdownSnippets/Processing/IncludeProcessor.cs
@@ -21,7 +21,7 @@ public IncludeProcessor(
         this.allFiles = allFiles;
     }
 
-    public bool TryProcessInclude(List<Line> lines, Line line, List<Include> used, int index, List<MissingInclude> missing, string? relativePath)
+    public bool TryProcessInclude(List<Line> lines, Line line, ICollection<Include> used, int index, List<MissingInclude> missing, string? relativePath)
     {
         var current = line.Current;
 
@@ -79,7 +79,7 @@ static string GetIncludeKey(CharSpan substring)
         return false;
     }
 
-    void Inner(List<Line> lines, Line line, List<Include> used, int index, List<MissingInclude> missing, string includeKey, string? relativePath)
+    void Inner(List<Line> lines, Line line, ICollection<Include> used, int index, List<MissingInclude> missing, string includeKey, string? relativePath)
     {
         if (includesLookup.TryGetValue(includeKey, out var include))
         {
@@ -125,7 +125,7 @@ void Inner(List<Line> lines, Line line, List<Include> used, int index, List<Miss
         line.Current = $"** Could not find include '{includeKey}' ** <!-- singleLineInclude: {includeKey} -->";
     }
 
-    void AddInclude(List<Line> lines, Line line, List<Include> used, int index, Include include, bool writePath)
+    void AddInclude(List<Line> lines, Line line, ICollection<Include> used, int index, Include include, bool writePath)
     {
         used.Add(include);
         var linesToInject = BuildIncludes(line, include, writePath).ToList();
diff --git a/src/MarkdownSnippets/Processing/MarkdownProcessor.cs b/src/MarkdownSnippets/Processing/MarkdownProcessor.cs
index 939680de..3a4bc147 100644
--- a/src/MarkdownSnippets/Processing/MarkdownProcessor.cs
+++ b/src/MarkdownSnippets/Processing/MarkdownProcessor.cs
@@ -129,8 +129,8 @@ internal ProcessResult Apply(List<Line> lines, string newLine, string? relativeP
         var missingSnippets = new List<MissingSnippet>();
         var validationErrors = new List<ValidationError>();
         var missingIncludes = new List<MissingInclude>();
-        var usedSnippets = new List<Snippet>();
-        var usedIncludes = new List<Include>();
+        var usedSnippets = new HashSet<Snippet>();
+        var usedIncludes = new HashSet<Include>();
         var builder = new StringBuilder();
         Line? tocLine = null;
 
@@ -263,8 +263,8 @@ void AppendWebSnippet(string url, string snippetKey, string? viewUrl = null)
 
         return new(
             missingSnippets: missingSnippets,
-            usedSnippets: usedSnippets.Distinct().ToList(),
-            usedIncludes: usedIncludes.Distinct().ToList(),
+            usedSnippets: usedSnippets.ToList(),
+            usedIncludes: usedIncludes.ToList(),
             missingIncludes: missingIncludes,
             validationErrors: validationErrors);
     }
@@ -292,7 +292,7 @@ bool ValidateContent(string? relativePath, Line line, List<ValidationError> vali
         return found;
     }
 
-    void ProcessSnippetLine(Action<string> appendLine, List<MissingSnippet> missings, List<Snippet> used, string key, string? relativePath, Line line)
+    void ProcessSnippetLine(Action<string> appendLine, List<MissingSnippet> missings, HashSet<Snippet> used, string key, string? relativePath, Line line)
     {
         appendLine($"<!-- snippet: {key} -->");
 
@@ -300,7 +300,7 @@ void ProcessSnippetLine(Action<string> appendLine, List<MissingSnippet> missings
         {
             appendSnippets(key, snippetsForKey, appendLine);
             appendLine("<!-- endSnippet -->");
-            used.AddRange(snippetsForKey);
+            used.UnionWith(snippetsForKey);
             return;
         }
 
@@ -312,7 +312,7 @@ void ProcessSnippetLine(Action<string> appendLine, List<MissingSnippet> missings
         appendLine("<!-- endSnippet -->");
     }
 
-    void ProcessWebSnippetLine(Action<string> appendLine, List<MissingSnippet> missings, List<Snippet> used, string url, string snippetKey, string? viewUrl, Line line)
+    void ProcessWebSnippetLine(Action<string> appendLine, List<MissingSnippet> missings, HashSet<Snippet> used, string url, string snippetKey, string? viewUrl, Line line)
     {
         var commentText = viewUrl == null
             ? $"<!-- web-snippet: {url}#{snippetKey} -->"