[Mono-bugs] [Bug 58572][Nor] Changed - ilasm delegate output

[bugzilla-daemon@bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by ck@carlo-kok.com.

http://bugzilla.ximian.com/show_bug.cgi?id=58572

--- shadow/58572	2004-05-16 08:54:56.000000000 -0400
+++ shadow/58572.tmp.29899	2004-05-16 08:56:01.000000000 -0400
@@ -2,13 +2,13 @@
 Product: Mono: Compilers
 Version: unspecified
 OS: All
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: IL assembler
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: ck@carlo-kok.com               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
@@ -91,6 +91,16 @@
     .method public virtual  hidebysig 
            instance default void 'Invoke' (int32 'i')  runtime managed 
     {
     }
   } 
 }
+
+------- Additional Comments From ck@carlo-kok.com  2004-05-16 08:56 -------
+I also noticed the RVA isn't right, RVA for "runtime" methods should
+be 0 according to peverify and is not 0. 
+I think this is related to the ZeroRva() function in MethodDef not
+checking for the Runtime flag, adding
+((ImplFlags & (ushort)(ImplAttr.Runtime)) != 0) ||
+
+to the internal bool ZeroRva()    method in the MethodDef class in
+PEAPI.cs makes sure the RVA is 0.