First pass at a branch with reduced memory usage; Keeps an open handle on the file, dispose your modules

1 files changed, 18 insertions, 15 deletions

diff --git a/Test/Mono.Cecil.Tests/Addin.cs b/Test/Mono.Cecil.Tests/Addin.cs
index 38b02ce..469d158 100644
--- a/Test/Mono.Cecil.Tests/Addin.cs
+++ b/Test/Mono.Cecil.Tests/Addin.cs
@@ -153,19 +153,20 @@ namespace Mono.Cecil.Tests {
 
 		ModuleDefinition RoundTrip (string location, ReaderParameters reader_parameters, string folder)
 		{
-			var module = ModuleDefinition.ReadModule (location, reader_parameters);
-			var rt_folder = Path.Combine (Path.GetTempPath (), folder);
-			if (!Directory.Exists (rt_folder))
-				Directory.CreateDirectory (rt_folder);
-			var rt_module = Path.Combine (rt_folder, Path.GetFileName (location));
-
-			var writer_parameters = new WriterParameters {
-				SymbolWriterProvider = GetSymbolWriterProvider (attribute),
-			};
+			var rt_folder = Path.Combine(Path.GetTempPath(), folder);
+			if (!Directory.Exists(rt_folder))
+				Directory.CreateDirectory(rt_folder);
+			var rt_module = Path.Combine(rt_folder, Path.GetFileName(location));
+
+			using (var module = ModuleDefinition.ReadModule (location, reader_parameters)) {
+				var writer_parameters = new WriterParameters {
+					SymbolWriterProvider = GetSymbolWriterProvider (attribute),
+				};
 
-			Reflect.InvokeMethod (Method, Fixture, new object [] { module });
+				Reflect.InvokeMethod (Method, Fixture, new object [] { module });
 
-			module.Write (rt_module, writer_parameters);
+				module.Write (rt_module, writer_parameters);
+			}
 
 			if (attribute.Verify)
 				CompilationService.Verify (rt_module);
@@ -176,11 +177,13 @@ namespace Mono.Cecil.Tests {
 		public override TestResult RunTest ()
 		{
 			var result = new TestResult (TestName);
-			var module = GetModule ();
-			if (module == null)
-				return result;
 
-			Reflect.InvokeMethod (Method, Fixture, new object [] { module });
+			using (var module = GetModule ()) {
+				if (module == null)
+					return result;
+
+				Reflect.InvokeMethod (Method, Fixture, new object [] { module });
+			}
 
 			result.Success ();
 			return result;