[jnimarshalmethod-gen] Update the originating assembly (#300)

Update the source assembly instead of creating a *new* assembly named
`<AssemblyBaseName>-new.dll`

I expected I would need to load the originating assembly in the
separate AppDomain, but instead it was enough to load it with Cecil
with reading parameters having `ReadWrite = true`.

Because the referenced assemblies might not be writable, we need to
keep loading them as read only, thus with default `ReadWrite = false`.

For that I added new method to our Cecil assembly resolver, to add
existing assembly definition to its cache. We first read the
originating assemblies as `ReadWrite` and add them to the resolver
cache.

Author: radekdoulik

Makefile

@@ -159,9 +159,7 @@ run-android: $(ATESTS)
 run-test-jnimarshal: bin/Test$(CONFIGURATION)/Java.Interop.Export-Tests.dll bin/Test$(CONFIGURATION)/$(JAVA_INTEROP_LIB)
 	MONO_TRACE_LISTENER=Console.Out \
 	$(RUNTIME) bin/$(CONFIGURATION)/jnimarshalmethod-gen.exe -v -L $(JI_MONO_LIB_PATH)mono/4.5 -L $(JI_MONO_LIB_PATH)mono/4.5/Facades "$<"
-	$(RM) "$<" "$(basename $<)-JniMarshalMethods.dll"
-	mv "$(basename $<)-new.dll" "$<"
-	mv "$(basename $<)-new.pdb" "$(basename $<).pdb"
+	$(RM) "$(basename $<)-JniMarshalMethods.dll"
 	$(call RUN_TEST,$<)
 
 

src/Java.Interop.Tools.Cecil/Java.Interop.Tools.Cecil/DirectoryAssemblyResolver.cs

@@ -110,6 +110,18 @@ public Dictionary<string, AssemblyDefinition> ToResolverCache ()
 			return new Dictionary<string, AssemblyDefinition>(cache);
 		}
 
+		public bool AddToCache (AssemblyDefinition assembly)
+		{
+			var name = Path.GetFileNameWithoutExtension (assembly.MainModule.FileName);
+
+			if (cache.ContainsKey (name))
+				return false;
+
+			cache [name] = assembly;
+
+			return true;
+		}
+
 		public virtual AssemblyDefinition Load (string fileName, bool forceLoad = false)
 		{
 			if (!File.Exists (fileName))

tools/jnimarshalmethod-gen/App.cs

@@ -62,14 +62,20 @@ public static int Main (string [] args)
 				return 0;
 			}
 
+			var readWriteParameters    = new ReaderParameters {
+				AssemblyResolver   = resolver,
+				ReadSymbols        = true,
+				ReadWrite          = true,
+			};
 			foreach (var assembly in assemblies) {
 				if (!File.Exists (assembly)) {
 					Error ($"Path '{assembly}' does not exist.");
 					return 1;
 				}
 
 				resolver.SearchDirectories.Add (Path.GetDirectoryName (assembly));
-				resolver.Load (assembly);
+				var ad = AssemblyDefinition.ReadAssembly (assembly, readWriteParameters);
+				resolver.AddToCache (ad);
 
 				try {
 					CreateMarshalMethodAssembly (assembly);

tools/jnimarshalmethod-gen/TypeMover.cs

@@ -46,11 +46,10 @@ public void Move ()
 				return;
 			}
 
-			var newName = $"{Path.Combine (Path.GetDirectoryName (Destination.MainModule.FileName), Path.GetFileNameWithoutExtension (Destination.MainModule.FileName))}-new{Path.GetExtension (Destination.MainModule.FileName)}";
-			Destination.Write (newName, new WriterParameters () { WriteSymbols = true });
+			Destination.Write (new WriterParameters () { WriteSymbols = true });
 
 			if (App.Verbose)
-				App.ColorWriteLine ($"Wrote {newName} assembly", ConsoleColor.Cyan);
+				App.ColorWriteLine ($"Wrote updated {Destination.MainModule.FileName} assembly", ConsoleColor.Cyan);
 		}
 
 		static readonly string nestedName = "__<$>_jni_marshal_methods";