Removed unnecessary finishing of base TypeWrapper and interfaces from CompiledTypeWrapper and DotNetTypeWrapper.

2 files changed, 0 insertions, 31 deletions

diff --git a/runtime/DotNetTypeWrapper.cs b/runtime/DotNetTypeWrapper.cs
index efcce048..94034749 100644
--- a/runtime/DotNetTypeWrapper.cs
+++ b/runtime/DotNetTypeWrapper.cs
@@ -57,7 +57,6 @@ namespace IKVM.Internal
 		private volatile TypeWrapper[] innerClasses;
 		private TypeWrapper outerClass;
 		private volatile TypeWrapper[] interfaces;
-		private volatile bool finished;
 
 		private static Modifiers GetModifiers(Type type)
 		{
@@ -2784,28 +2783,6 @@ namespace IKVM.Internal
 
 		internal override void Finish()
 		{
-			// we don't need locking, because Finish and Link are idempotent
-			if (finished)
-			{
-				return;
-			}
-			if (BaseTypeWrapper != null)
-			{
-				BaseTypeWrapper.Finish();
-			}
-			foreach (TypeWrapper tw in this.Interfaces)
-			{
-				tw.Finish();
-			}
-			foreach (MethodWrapper mw in GetMethods())
-			{
-				mw.Link();
-			}
-			foreach (FieldWrapper fw in GetFields())
-			{
-				fw.Link();
-			}
-			finished = true;
 		}
 
 		internal override MethodParametersEntry[] GetMethodParameters(MethodWrapper mw)
diff --git a/runtime/TypeWrapper.cs b/runtime/TypeWrapper.cs
index 038ea0e5..5d4e0be6 100644
--- a/runtime/TypeWrapper.cs
+++ b/runtime/TypeWrapper.cs
@@ -5110,14 +5110,6 @@ namespace IKVM.Internal
 
 		internal override void Finish()
 		{
-			if(BaseTypeWrapper != null)
-			{
-				BaseTypeWrapper.Finish();
-			}
-			foreach(TypeWrapper tw in this.Interfaces)
-			{
-				tw.Finish();
-			}
 		}
 
 #if EMITTERS