diff options
Diffstat (limited to 'src/search.c')
| -rw-r--r--[-rwxr-xr-x] | src/search.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/src/search.c b/src/search.c index fdcb955..fae2567 100755..100644 --- a/src/search.c +++ b/src/search.c @@ -356,7 +356,6 @@ Search(IN SEARCHER_THREAD_CONTEXT *ctx, iImprovement = (iEval - ctx->sPlyInfo[ctx->uPly - 2].iEval); } - GENERATE_NO_MOVES; if (!fSkipNull && !fThreat && @@ -486,21 +485,6 @@ Search(IN SEARCHER_THREAD_CONTEXT *ctx, ctx->sSearchFlags.fAvoidNullmove = TRUE; RescoreMovesViaSearch(ctx, uDepth, iAlpha, iBeta); ctx->sSearchFlags.fAvoidNullmove = FALSE; - // NOT always TRUE here -- pre-existing bug, found - // via debug_smoke_test.sh (a deeper/larger-than- - // usual sample finally hit the rare path). - // RescoreMovesViaSearch's own fail-high branch - // (searchsup.c) deliberately leaves this FALSE by - // design -- a fail-high only proves uBest is good - // enough, not honest eval-axis scores for every - // move, so claiming fMovesRescoredByIID would be a - // lie. This assert demanded the opposite of that - // documented contract; DO_IID is unconditionally - // compiled in (chess.h) so this could fire on any - // DEBUG build given an unlucky enough rescore -- - // ASSERT is a no-op in release, so this never - // crashed in production, but it made the DEBUG/ - // TEST harness itself unreliable at random. } } #endif |