2002-09-08 Miguel de Icaza <miguel@ximian.com>

	* statement.cs (For.Emit): for infinite loops (test == null)
	return whether there is a break inside, not always "true".

svn path=/trunk/mcs/; revision=7315

diff --git a/mcs/mcs/ChangeLog b/mcs/mcs/ChangeLog
index a28554971351296e166230a374ff41f1a744d5f2..dc6f148b34d10f873f10186abbe2f81b5148c1c3 100755 (executable)
--- a/mcs/mcs/ChangeLog
+++ b/mcs/mcs/ChangeLog
@@ -1,5 +1,8 @@
 2002-09-08  Miguel de Icaza  <miguel@ximian.com>
 
+       * statement.cs (For.Emit): for infinite loops (test == null)
+       return whether there is a break inside, not always "true".
+
        * namespace.cs (UsingEntry): New struct to hold the name of the
        using definition, the location where it is defined, and whether it
        has been used in a successful type lookup.

diff --git a/mcs/mcs/statement.cs b/mcs/mcs/statement.cs
index de4a71c4681fb65e2e9ffb1ac4b56067be4aa18e..a821cb9436772824027ef2e9e590bef2fc080895 100755 (executable)
--- a/mcs/mcs/statement.cs
+++ b/mcs/mcs/statement.cs
@@ -500,7 +500,7 @@ namespace Mono.CSharp {
                        ec.LoopBeginTryCatchLevel = old_loop_begin_try_catch_level;
                        
                        //
-                       // Inform whether we are infinite or not
+                       // Inform whether we are infinite or not
                        //
                        if (Test != null){
                                if (Test is BoolConstant){
@@ -511,7 +511,7 @@ namespace Mono.CSharp {
                                }
                                return false;
                        } else
-                               return true;
+                               return breaks == false;
                }
        }