[Ikvm-commit] ikvm/ikvmc AotTypeWrapper.cs,1.45.2.2,1.45.2.3

[Jeroen F. <jfr...@us...>]

Update of /cvsroot/ikvm/ikvm/ikvmc
In directory sfp-cvsdas-1.v30.ch3.sourceforge.com:/tmp/cvs-serv18570/ikvmc

Modified Files:
      Tag: v0_42
	AotTypeWrapper.cs 
Log Message:
Back port of fix for #2946842.

Index: AotTypeWrapper.cs
===================================================================
RCS file: /cvsroot/ikvm/ikvm/ikvmc/AotTypeWrapper.cs,v
retrieving revision 1.45.2.2
retrieving revision 1.45.2.3
diff -C2 -d -r1.45.2.2 -r1.45.2.3
*** AotTypeWrapper.cs	12 Jan 2010 16:57:12 -0000	1.45.2.2
--- AotTypeWrapper.cs	19 Feb 2010 08:14:27 -0000	1.45.2.3
***************
*** 89,93 ****
  					AttributeHelper.HideFromJava(typeBuilder);
  					AttributeHelper.SetEditorBrowsableNever(typeBuilder);
! 					workaroundBaseClass = new WorkaroundBaseClass(typeBuilder, methods.ToArray());
  					List<MethodWrapper> constructors = new List<MethodWrapper>();
  					foreach (MethodWrapper mw in baseTypeWrapper.GetMethods())
--- 89,93 ----
  					AttributeHelper.HideFromJava(typeBuilder);
  					AttributeHelper.SetEditorBrowsableNever(typeBuilder);
! 					workaroundBaseClass = new WorkaroundBaseClass(this, typeBuilder, methods.ToArray());
  					List<MethodWrapper> constructors = new List<MethodWrapper>();
  					foreach (MethodWrapper mw in baseTypeWrapper.GetMethods())
***************
*** 98,102 ****
  						}
  					}
- 					Serialization.AddAutomagicSerializationToWorkaroundBaseClass(typeBuilder);
  					replacedMethods = constructors.ToArray();
  					return typeBuilder;
--- 98,101 ----
***************
*** 109,119 ****
  		{
  			base.Finish();
! 			lock (this)
  			{
! 				if (workaroundBaseClass != null)
! 				{
! 					workaroundBaseClass.Finish();
! 					workaroundBaseClass = null;
! 				}
  			}
  		}
--- 108,114 ----
  		{
  			base.Finish();
! 			if (workaroundBaseClass != null)
  			{
! 				workaroundBaseClass.Finish();
  			}
  		}
***************
*** 121,143 ****
  		private sealed class WorkaroundBaseClass
  		{
  			private readonly TypeBuilder typeBuilder;
  			private readonly MethodWrapper[] methods;
  
! 			internal WorkaroundBaseClass(TypeBuilder typeBuilder, MethodWrapper[] methods)
  			{
  				this.typeBuilder = typeBuilder;
  				this.methods = methods;
  			}
  
  			internal void Finish()
  			{
! 				foreach (MethodWrapper mw in methods)
  				{
! 					MethodBuilder mb = typeBuilder.DefineMethod(mw.Name, MethodAttributes.FamORAssem | MethodAttributes.Virtual | MethodAttributes.HideBySig | MethodAttributes.CheckAccessOnOverride, mw.ReturnTypeForDefineMethod, mw.GetParametersForDefineMethod());
! 					AttributeHelper.HideFromJava(mb);
! 					CodeEmitter ilgen = CodeEmitter.Create(mb);
! 					ilgen.EmitThrow("java.lang.AbstractMethodError");
  				}
- 				typeBuilder.CreateType();
  			}
  		}
--- 116,155 ----
  		private sealed class WorkaroundBaseClass
  		{
+ 			private readonly AotTypeWrapper wrapper;
  			private readonly TypeBuilder typeBuilder;
  			private readonly MethodWrapper[] methods;
+ 			private ConstructorInfo baseSerializationCtor;
+ 			private bool finished;
  
! 			internal WorkaroundBaseClass(AotTypeWrapper wrapper, TypeBuilder typeBuilder, MethodWrapper[] methods)
  			{
+ 				this.wrapper = wrapper;
  				this.typeBuilder = typeBuilder;
  				this.methods = methods;
  			}
  
+ 			internal ConstructorInfo GetSerializationConstructor()
+ 			{
+ 				if (baseSerializationCtor == null)
+ 				{
+ 					baseSerializationCtor = Serialization.AddAutomagicSerializationToWorkaroundBaseClass(typeBuilder, wrapper.BaseTypeWrapper.GetSerializationConstructor());
+ 				}
+ 				return baseSerializationCtor;
+ 			}
+ 
  			internal void Finish()
  			{
! 				if (!finished)
  				{
! 					finished = true;
! 					foreach (MethodWrapper mw in methods)
! 					{
! 						MethodBuilder mb = typeBuilder.DefineMethod(mw.Name, MethodAttributes.FamORAssem | MethodAttributes.Virtual | MethodAttributes.HideBySig | MethodAttributes.CheckAccessOnOverride, mw.ReturnTypeForDefineMethod, mw.GetParametersForDefineMethod());
! 						AttributeHelper.HideFromJava(mb);
! 						CodeEmitter ilgen = CodeEmitter.Create(mb);
! 						ilgen.EmitThrow("java.lang.AbstractMethodError");
! 					}
! 					typeBuilder.CreateType();
  				}
  			}
  		}
***************
*** 1215,1218 ****
--- 1227,1259 ----
  			get { return true; }
  		}
+ 
+ 		internal MethodWrapper ReplaceMethodWrapper(MethodWrapper mw)
+ 		{
+ 			if (replacedMethods != null)
+ 			{
+ 				foreach (MethodWrapper r in replacedMethods)
+ 				{
+ 					if (mw.DeclaringType == r.DeclaringType
+ 						&& mw.Name == r.Name
+ 						&& mw.Signature == r.Signature)
+ 					{
+ 						return r;
+ 					}
+ 				}
+ 			}
+ 			return mw;
+ 		}
+ 
+ 		internal override ConstructorInfo GetBaseSerializationConstructor()
+ 		{
+ 			if (workaroundBaseClass != null)
+ 			{
+ 				return workaroundBaseClass.GetSerializationConstructor();
+ 			}
+ 			else
+ 			{
+ 				return base.GetBaseSerializationConstructor();
+ 			}
+ 		}
  	}
  }