From 2991ff0dc2cbf762cb3e48027fa3baed5f1aebc3 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Sat, 30 Oct 2010 18:18:48 +0100 Subject: [PATCH] Move moveCount update near the SpNode case No functional change. Signed-off-by: Marco Costalba --- src/search.cpp | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/src/search.cpp b/src/search.cpp index dcfedcdb..f3b092d2 100644 --- a/src/search.cpp +++ b/src/search.cpp @@ -1194,16 +1194,17 @@ split_point_start: // At split points actual search starts from here && (move = mp.get_next_move()) != MOVE_NONE && !ThreadsMgr.thread_should_stop(threadID)) { + assert(move_is_ok(move)); + if (SpNode) { moveCount = ++sp->moveCount; lock_release(&(sp->lock)); } - - assert(move_is_ok(move)); - - if (move == excludedMove) + else if (move == excludedMove) continue; + else + movesSearched[moveCount++] = move; moveIsCheck = pos.move_is_check(move, ci); captureOrPromotion = pos.move_is_capture_or_promotion(move); @@ -1235,13 +1236,9 @@ split_point_start: // At split points actual search starts from here } } - newDepth = depth - ONE_PLY + ext; - // Update current move (this must be done after singular extension search) - movesSearched[moveCount] = ss->currentMove = move; - - if (!SpNode) - moveCount++; + ss->currentMove = move; + newDepth = depth - ONE_PLY + ext; // Step 12. Futility pruning (is omitted in PV nodes) if ( !PvNode @@ -1399,7 +1396,7 @@ split_point_start: // At split points actual search starts from here if (SpNode) { - /* Here we have the lock still grabbed */ + // Here we have the lock still grabbed sp->slaves[threadID] = 0; lock_release(&(sp->lock)); return bestValue; -- 2.39.2