[Mono-bugs] [Bug 52589][Wis] Changed - We allow break, continue, goto to exit a finally block

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

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

--- shadow/52589	2004-01-07 08:48:58.000000000 -0500
+++ shadow/52589.tmp.24977	2004-01-20 16:07:35.000000000 -0500
@@ -1,13 +1,13 @@
 Bug#: 52589
 Product: Mono/Compilers
 Version: unspecified
 OS: unknown
 OS Details: 
-Status: NEW   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: Unknown
 Priority: Wishlist
 Component: C#
 AssignedTo: martin@ximian.com                            
 ReportedBy: bmaurer@users.sf.net               
 QAContact: mono-bugs@ximian.com
@@ -76,6 +76,14 @@
 and increment/deincrement it. Each loop and label would store the 
 finally level that it was at durring its decl. A break/continue/goto 
 statement would then check if its FinallyLevel was different than the 
 FinallyLevel of its enclosing loop. If they are different, than the 
 statement would cause the exiting of the FinallyLoop, and cause an 
 error. 
+
+------- Additional Comments From martin@ximian.com  2004-01-20 16:07 -------
+I don't like your implementation idea at all.  There's no need to add 
+yet another field to EmitContext for things the flow code already 
+knows about. 
+ 
+It's now FIXED in CVS (except the goto case, but that's another 
+story).