From 7af1b40b4e560da33a35825d61eedf1775cc7110 Mon Sep 17 00:00:00 2001 From: Joona Kiiski Date: Wed, 15 Apr 2009 18:14:06 +0300 Subject: [PATCH] Restore calling insert_pv after search is aborted + small clean-up Restore old behaviour that after search is aborted we call insert_pv, before breaking out from the iterative deepening loop. Remove one useless FIXME and document other better. Signed-off-by: Marco Costalba --- src/search.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/search.cpp b/src/search.cpp index 8414e329..d769619d 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -712,13 +712,14 @@ namespace { // Search to the current depth Value value = root_search(p, ss, rml, alpha, beta); - if (AbortSearch) - break; // Value cannot be trusted. Break out immediately! // Write PV to transposition table, in case the relevant entries have // been overwritten during the search. TT.insert_pv(p, ss[0].pv); + if (AbortSearch) + break; // Value cannot be trusted. Break out immediately! + //Save info about search result Value speculatedValue; bool fHigh = false; @@ -857,7 +858,6 @@ namespace { Value root_search(Position &pos, SearchStack ss[], RootMoveList &rml, Value alpha, Value beta) { - //FIXME: Implement bestValue Value oldAlpha = alpha; Value value; Bitboard dcCandidates = pos.discovered_check_candidates(pos.side_to_move()); @@ -2507,7 +2507,7 @@ namespace { return; bool overTime = t > AbsoluteMaxSearchTime - || (RootMoveNumber == 1 && t > MaxSearchTime + ExtraSearchTime && !FailLow) //FIXME: BUG?? + || (RootMoveNumber == 1 && t > MaxSearchTime + ExtraSearchTime && !FailLow) //FIXME: We are not checking any problem flags, BUG? || ( !FailHigh && !FailLow && !fail_high_ply_1() && !Problem && t > 6*(MaxSearchTime + ExtraSearchTime)); -- 2.39.2