Move collections in ResolveConflictsProcessor to be closer to where they are used

Author: Exanite

sources/SilkTouch/SilkTouch/Mods/PrettifyNames.cs

@@ -1132,17 +1132,6 @@ public void ProcessNames(NameProcessorContext context)
                 }
             }
 
-            // These collections are used later.
-            // These keep track of method discriminators to determine whether we have incompatible overloads.
-            // We keep track of the first original name so that we can add it to conflictingOriginalNames when we
-            // do discover a conflict (along with the original name of the actual conflict).
-            var methodDiscriminators =
-                new Dictionary<
-                    string,
-                    (string? FirstOriginalName, List<MethodDeclarationSyntax> Methods)
-                >();
-            var conflictingOriginalNames = new HashSet<string>();
-
             // This loop cannot be part of the loop below because it modifies the primaries
             foreach (var (scope, members) in context.Names)
             {
@@ -1208,6 +1197,17 @@ d is not MethodDeclarationSyntax
                 }
             }
 
+            // These collections are used later.
+            // These keep track of method discriminators to determine whether we have incompatible overloads.
+            // We keep track of the first original name so that we can add it to conflictingOriginalNames when we
+            // do discover a conflict (along with the original name of the actual conflict).
+            var methodDiscriminators =
+                new Dictionary<
+                    string,
+                    (string? FirstOriginalName, List<MethodDeclarationSyntax> Methods)
+                >();
+            var conflictingOriginalNames = new HashSet<string>();
+
             foreach (var (scope, members) in context.Names)
             {
                 nameData.Names.TryGetValue(scope, out var scopeData);