[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdegames/kmines/solver
From:       Nicolas Hadacek <hadacek () kde ! org>
Date:       2004-02-26 4:59:39
Message-ID: 20040226045939.3097A9962 () office ! kde ! org
[Download RAW message or body]

CVS commit by hadacek: 

fix longstanding bug #55977 - your favorite Heisenbug ... no 
wonder it was only present in production code :(

the lesson for today is do not put non-const code in an assert !!!


  M +5 -2      advFastRules.cpp   1.6


--- kdegames/kmines/solver/advFastRules.cpp  #1.5:1.6
@@ -71,6 +71,9 @@ void AdviseFast::RuleSet::solve()
 {
         Coord p;
-        while(getSurePoint(&p))
-                assert(reveal(p));
+        while(getSurePoint(&p)) {
+          bool res = reveal(p);
+          assert(res);
+          Q_UNUSED(res);
+        }
 }
 


[prev in list] [next in list] [prev in thread] [next in thread]