[Mono-bugs] [Bug 75166][Wis] Changed - break statement not getting proper parent.

[bugzilla-daemon at 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 cesar at ciencias.unam.mx.

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

--- shadow/75166	2005-06-06 19:18:38.000000000 -0400
+++ shadow/75166.tmp.6817	2005-06-06 19:32:12.000000000 -0400
@@ -1,17 +1,17 @@
 Bug#: 75166
 Product: Mono: Compilers
 Version: unspecified
-OS: 
+OS: unknown
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Wishlist
 Component: JScript
-AssignedTo: mono-bugs at ximian.com                            
+AssignedTo: cesar at ciencias.unam.mx                            
 ReportedBy: cesar at ciencias.unam.mx               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
 URL: 
 Cc: 
 Summary: break statement  not getting proper parent.
@@ -53,6 +53,26 @@
 svn version fromMon Jun  6 18:10:38 CDT 2005
 
 ------- Additional Comments From cesar at ciencias.unam.mx  2005-06-06 19:18 -------
 Created an attachment (id=15276)
 break stm test case.
 
+
+------- Additional Comments From cesar at ciencias.unam.mx  2005-06-06 19:32 -------
+The problem is that in the following lines pn when gets passed as
+'parent' parameter to Statement is null, and altought at the end we
+set the parent of body to pn (which at that time is a ForIn or For
+object), the objects contained in body do not get the reference
+properly set.
+
+From Parser.cs, lines: 497-to-503:
+****
+body = Statement (pn);
+
+if (incr == null) // cond could be null if 'in obj' got eaten by the\
+init node.
+    pn = new ForIn (parent, line_number, init, cond, body);
+else
+    pn = new For (parent, line_number, init, cond, incr, body);
+body.parent = pn;
+****
+