From 9d1e32db698855b939c4bc5df6b6a82509dd381c Mon Sep 17 00:00:00 2001 From: Scott Gasch Date: Wed, 9 Sep 2026 13:10:43 -0700 Subject: Minor changes and file permissions. --- src/search.c | 16 ---------------- 1 file changed, 16 deletions(-) mode change 100755 => 100644 src/search.c (limited to 'src/search.c') diff --git a/src/search.c b/src/search.c old mode 100755 new mode 100644 index fdcb955..fae2567 --- 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 -- cgit v1.3