[Mono-bugs] [Bug 21598] Changed - Optional Attributes for the Compile Unit

[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 miguel@ximian.com.

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

--- shadow/21598	Fri Mar  8 16:02:29 2002
+++ shadow/21598.tmp.26497	Tue Mar 12 01:36:02 2002
@@ -1,14 +1,14 @@
 Bug#: 21598
 Product: Mono/MCS
 Version: unspecified
-OS: 
+OS: unknown
 OS Details: 
-Status: NEW   
+Status: ASSIGNED   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: Misc
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: f_ai@hotmail.com               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
@@ -25,6 +25,21 @@
 "
 
 Then clause for 'compilation_unit' should be more like:
 compilation_unit
 	: opt_using_directives opt_attributes 
 opt_namespace_member_declarations opt_EOF
+
+------- Additional Comments From miguel@ximian.com  2002-03-12 01:36 -------
+I am aware of this problem (this is what is causing the various
+problems with the assembly attributes).  The problem is that changing
+the grammar will lead to incorrectly applied attributes currently, or
+will lead to many reduce/reduce errors.
+
+The right fix is to rewrite chunks of the attribute-application code,
+to actually allow for the code to know where to apply the bits.
+
+You can try the various effects by changing the rules as you described
+and re-running the couple of tests we have for attributes in the test
+suite, that will shed some light into the problem.
+
+Miguel.