[Mono-bugs] [Bug 30020][Maj] Changed - mcs doesn't set correctly the flags on (non-virtual) insterface method implementations

[bugzilla-daemon@rocky.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 lupus@ximian.com.

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

--- shadow/30020	Sun Sep  8 01:41:32 2002
+++ shadow/30020.tmp.20425	Wed Sep 11 09:14:14 2002
@@ -1,23 +1,23 @@
 Bug#: 30020
-Product: Mono/Class Libraries
+Product: Mono/MCS
 Version: unspecified
 OS: other
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Major
-Component: CORLIB
+Component: Misc
 AssignedTo: lupus@ximian.com                            
 ReportedBy: miguel@ximian.com               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
-Summary: Assembly.GetTypes crashes mono.
+Summary: mcs doesn't set correctly the flags on (non-virtual) insterface method implementations
 OtherBugsDependingOnThis: 29479
 
 The following program crashes Mono:
 
 using System;
 using System.Reflection;
@@ -45,6 +45,26 @@
 METHOD IsContextOK(Context,IConstructionCallMessage)
 
 ** ERROR **: file class.c: line 666 (mono_class_setup_vtable): should not
 be reached
 aborting...
 Aborted
+
+------- Additional Comments From lupus@ximian.com  2002-09-11 09:14 -------
+This is a bug in mcs: it doesn't set the needed flags when an
+interface method is not virtual. This seems to happen only for attributes.
+Compile the following sample with mcs on either platform:
+
+using System;
+
+interface I {
+        void stuff ();
+}
+class T : Attribute, I {
+        public void stuff (){}
+
+        static void Main () {}
+}
+
+mcs sets only the public flag, while csc will set also:
+final  virtual  hidebysig  newslot.
+PEVerify will complain, too, if run on the resulting executable.