diff options
| author | Scott Gasch <[email protected]> | 2026-10-14 12:24:20 -0700 |
|---|---|---|
| committer | Scott Gasch <[email protected]> | 2026-10-14 12:24:20 -0700 |
| commit | 2d7d182d4ab518d9f48c5b8cef0c5f664ed9ba3f (patch) | |
| tree | 4e11c867da523e01a165a5fe27c58ac4ec6cb49f | |
| parent | 78a5ea00ec507164e1cfc7fd12846e1b05d78e7e (diff) | |
Fix undeclared fPruneReduceEligible typo in dead FH_TABLE_COLLECTION block
Same class of bug as the just-removed PRUNE_VERIFY: search.c's
#ifdef FH_TABLE_COLLECTION block (never defined by any GNUmakefile
path) referenced fPruneReduceEligible, which doesn't exist -- the
real variable is fMoveIsPruneReduceEligible. Confirmed this block has
never compiled (clang -DFH_TABLE_COLLECTION failed before this fix,
succeeds after). Unlike PRUNE_VERIFY, this is a one-identifier typo
with an obvious correct fix (the real variable's name is a strict
superset, "Move" inserted), not a case of missing/invented semantics,
so fixed rather than removed. Two stale comment references to the old
name updated too.
Co-Authored-By: Claude Sonnet 5 <[email protected]>
| -rw-r--r-- | src/search.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/search.c b/src/search.c index 2d87035..a517a2d 100644 --- a/src/search.c +++ b/src/search.c @@ -102,7 +102,7 @@ extern FLAG g_fCanSplit[MAX_PLY_PER_SEARCH]; // EstimateHangingMaterialCp/HangCpBucket/GapCpBucket/MoveNumBucket/ // DepthBucket moved there too but are no longer file-static -- the // measurement/collection block further below (gated on -// fPruneReduceEligible alone, independent of the LMR decision -- this +// fMoveIsPruneReduceEligible alone, independent of the LMR decision -- this // is what's still generating the next round of g_uJointFHThousandths // data) still needs them, so they're declared in chess.h and called // from here unchanged in behavior, just by their new (un-underscored) @@ -1210,7 +1210,7 @@ Search(IN SEARCHER_THREAD_CONTEXT *ctx, INC(ctx->sCounters.tree.u64LeftoverTries); #ifdef FH_TABLE_COLLECTION - if (TRUE == fPruneReduceEligible) + if (TRUE == fMoveIsPruneReduceEligible) { SCORE iHangCp; @@ -1497,7 +1497,7 @@ Search(IN SEARCHER_THREAD_CONTEXT *ctx, if (TRUE == fIsLeftoverMove) { INC(ctx->sCounters.tree.u64LeftoverFH); - // Gated on fPruneReduceEligible now, + // Gated on fMoveIsPruneReduceEligible now, // matching the tries side -- must // match exactly, or this would reuse a // stale bucket left over from a |