Simplify time management

noProblemFound condition is never true.
This was verified by running 800 games 1+0 match in 1 CPU computer.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>

diff --git a/src/search.cpp b/src/search.cpp
index ff791b8ea903e85649c99ee651661f26e31f38f0..6c28b52f3fad315bce9d866a926da6d5396d46f7 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -2771,15 +2771,8 @@ namespace {
                            && !FailLow
                            &&  t > MaxSearchTime + ExtraSearchTime;
 
                            && !FailLow
                            &&  t > MaxSearchTime + ExtraSearchTime;
 

-    bool noProblemFound =   !FailHigh
-                         && !FailLow
-                         && !fail_high_ply_1()
-                         && !Problem
-                         &&  t > 6 * (MaxSearchTime + ExtraSearchTime);
-

     bool noMoreTime =   t > AbsoluteMaxSearchTime
     bool noMoreTime =   t > AbsoluteMaxSearchTime

-                     || stillAtFirstMove //FIXME: We are not checking any problem flags, BUG?
-                     || noProblemFound;
+                     || stillAtFirstMove;

 
     if (   (Iteration >= 3 && UseTimeManagement && noMoreTime)
         || (ExactMaxTime && t >= ExactMaxTime)
 
     if (   (Iteration >= 3 && UseTimeManagement && noMoreTime)
         || (ExactMaxTime && t >= ExactMaxTime)


@@ -2801,15 +2794,8 @@ namespace {
                            && !FailLow
                            &&  t > MaxSearchTime + ExtraSearchTime;
 
                            && !FailLow
                            &&  t > MaxSearchTime + ExtraSearchTime;
 

-    bool noProblemFound =   !FailHigh
-                         && !FailLow
-                         && !fail_high_ply_1()
-                         && !Problem
-                         &&  t > 6 * (MaxSearchTime + ExtraSearchTime);
-

     bool noMoreTime =   t > AbsoluteMaxSearchTime
     bool noMoreTime =   t > AbsoluteMaxSearchTime

-                     || stillAtFirstMove
-                     || noProblemFound;
+                     || stillAtFirstMove;

 
     if (Iteration >= 3 && UseTimeManagement && (noMoreTime || StopOnPonderhit))
         AbortSearch = true;
 
     if (Iteration >= 3 && UseTimeManagement && (noMoreTime || StopOnPonderhit))
         AbortSearch = true;