Use IReadOnlyList.

Moved methods to MainForm.

Author: KN4CK3R

ReClass.NET/CodeGenerator/CppCodeGenerator.cs

@@ -108,9 +108,9 @@ public CppCodeGenerator(CppTypeMapping typeMapping)
 			};
 		}
 
-		public string GenerateCode(IEnumerable<ClassNode> classes, ILogger logger)
+		public string GenerateCode(IReadOnlyList<ClassNode> classes, ILogger logger)
 		{
-			var classNodes = classes as IList<ClassNode> ?? classes.ToList();
+			var classNodes = classes;
 
 			var sb = new StringBuilder();
 			sb.AppendLine($"// Created with {Constants.ApplicationName} {Constants.ApplicationVersion} by {Constants.Author}");

ReClass.NET/CodeGenerator/ICodeGenerator.cs

@@ -16,15 +16,15 @@ public interface ICodeGenerator
 		/// <param name="classes">The classes to generate code from.</param>
 		/// <param name="logger">The logger used to output messages.</param>
 		/// <returns>The code for the classes.</returns>
-		string GenerateCode(IEnumerable<ClassNode> classes, ILogger logger);
+		string GenerateCode(IReadOnlyList<ClassNode> classes, ILogger logger);
 	}
 
 	[ContractClassFor(typeof(ICodeGenerator))]
 	internal abstract class CodeGeneratorContract : ICodeGenerator
 	{
 		public Language Language => throw new NotImplementedException();
 
-		public string GenerateCode(IEnumerable<ClassNode> classes, ILogger logger)
+		public string GenerateCode(IReadOnlyList<ClassNode> classes, ILogger logger)
 		{
 			Contract.Requires(classes != null);
 			Contract.Requires(Contract.ForAll(classes, c => c != null));

ReClass.NET/Forms/CodeForm.cs

@@ -19,7 +19,7 @@ namespace ReClassNET.Forms
 {
 	public partial class CodeForm : IconForm
 	{
-		public CodeForm(ICodeGenerator generator, IEnumerable<ClassNode> classes, ILogger logger)
+		public CodeForm(ICodeGenerator generator, IReadOnlyList<ClassNode> classes, ILogger logger)
 		{
 			Contract.Requires(generator != null);
 			Contract.Requires(classes != null);

ReClass.NET/Forms/MainForm.Functions.cs

@@ -20,8 +20,30 @@
 
 namespace ReClassNET.Forms
 {
-	public partial class MainForm : IconForm
+	public partial class MainForm
 	{
+		public void ShowDefaultCodeGeneratorForm(IReadOnlyList<ClassNode> classes)
+		{
+			Contract.Requires(classes != null);
+
+			ShowCodeGeneratorForm(classes, new CppCodeGenerator(currentProject.TypeMapping));
+		}
+
+		public void ShowCodeGeneratorForm(ICodeGenerator generator)
+		{
+			Contract.Requires(generator != null);
+
+			ShowCodeGeneratorForm(currentProject.Classes, generator);
+		}
+
+		public void ShowCodeGeneratorForm(IReadOnlyList<ClassNode> classes, ICodeGenerator generator)
+		{
+			Contract.Requires(classes != null);
+			Contract.Requires(generator != null);
+
+			new CodeForm(generator, classes, Program.Logger).Show();
+		}
+
 		public void AttachToProcess(string processName)
 		{
 			var info = Program.CoreFunctions.EnumerateProcesses().FirstOrDefault(p => string.Equals(p.Name, processName, StringComparison.OrdinalIgnoreCase));
@@ -93,15 +115,6 @@ void UpdateClassNodes(BaseNode node)
 			memoryViewControl.ClassNode = currentProject.Classes.FirstOrDefault();
 		}
 
-		/// <summary>Shows the code form with the given <paramref name="generator"/>.</summary>
-		/// <param name="generator">The generator.</param>
-		private void ShowCodeForm(ICodeGenerator generator)
-		{
-			Contract.Requires(generator != null);
-
-			LinkedWindowFeatures.ShowCodeGeneratorForm(currentProject.Classes, generator);
-		}
-
 		/// <summary>Opens the <see cref="InputBytesForm"/> and calls <paramref name="callback"/> with the result.</summary>
 		/// <param name="title">The title of the input form.</param>
 		/// <param name="callback">The function to call afterwards.</param>

ReClass.NET/Forms/MainForm.cs

@@ -383,12 +383,12 @@ private void cleanUnusedClassesToolStripMenuItem_Click(object sender, EventArgs
 
 		private void generateCppCodeToolStripMenuItem_Click(object sender, EventArgs e)
 		{
-			ShowCodeForm(new CppCodeGenerator(CurrentProject.TypeMapping));
+			ShowCodeGeneratorForm(new CppCodeGenerator(currentProject.TypeMapping));
 		}
 
 		private void generateCSharpCodeToolStripMenuItem_Click(object sender, EventArgs e)
 		{
-			ShowCodeForm(new CSharpCodeGenerator());
+			ShowCodeGeneratorForm(new CSharpCodeGenerator());
 		}
 
 		private void aboutToolStripMenuItem_Click(object sender, EventArgs e)
@@ -679,7 +679,7 @@ private void showCodeOfClassToolStripMenuItem_Click(object sender, EventArgs e)
 		{
 			if (memoryViewControl.GetSelectedNodes().FirstOrDefault()?.Node is ClassNode node)
 			{
-				LinkedWindowFeatures.ShowCodeGeneratorForm(node.Yield(), CurrentProject.TypeMapping);
+				ShowDefaultCodeGeneratorForm(new[] { node });
 			}
 		}
 
@@ -895,7 +895,7 @@ private void showCodeOfClassToolStripMenuItem2_Click(object sender, EventArgs e)
 				return;
 			}
 
-			LinkedWindowFeatures.ShowCodeGeneratorForm(classNode.Yield(), CurrentProject.TypeMapping);
+			ShowDefaultCodeGeneratorForm(new[] { classNode });
 		}
 
 		private void enableHierarchyViewToolStripMenuItem_Click(object sender, EventArgs e)

ReClass.NET/Project/ReClassNetProject.cs

@@ -20,9 +20,9 @@ public class ReClassNetProject : IDisposable
 		private readonly List<EnumDescription> enums = new List<EnumDescription>();
 		private readonly List<ClassNode> classes = new List<ClassNode>();
 
-		public IEnumerable<EnumDescription> Enums => enums;
+		public IReadOnlyList<EnumDescription> Enums => enums;
 
-		public IEnumerable<ClassNode> Classes => classes;
+		public IReadOnlyList<ClassNode> Classes => classes;
 
 		public string Path { get; set; }
 

ReClass.NET/UI/LinkedWindowFeatures.cs

@@ -132,21 +132,5 @@ public static void StartMemoryScan(IScanComparer comparer)
 
 			sf.ExcuteScan(settings, comparer);
 		}
-
-		public static void ShowCodeGeneratorForm(IEnumerable<ClassNode> classes, CppTypeMapping typeMapping)
-		{
-			Contract.Requires(classes != null);
-			Contract.Requires(typeMapping != null);
-
-			ShowCodeGeneratorForm(classes, new CppCodeGenerator(typeMapping));
-		}
-
-		public static void ShowCodeGeneratorForm(IEnumerable<ClassNode> classes, ICodeGenerator generator)
-		{
-			Contract.Requires(classes != null);
-			Contract.Requires(generator != null);
-
-			new CodeForm(generator, classes, Program.Logger).Show();
-		}
 	}
 }