jirayu_sri/stockfish
1
0
Fork 0
mirror of https://github.com/official-stockfish/Stockfish.git synced 2026-07-22 12:47:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix material key computation error

Browse Source 
Fixes an error introduced in 278a755fb5 https://github.com/official-stockfish/Stockfish/pull/6891

Also enables more checking in debug builds

closes https://github.com/official-stockfish/Stockfish/pull/6894

No functional change
This commit is contained in:
anematode
2026-06-10 23:01:50 +02:00
committed by Joost VandeVondele
parent 5595cb20ea
commit 86f1df7134
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/position.cpp
+3 -3
View File
@@ -989,8 +989,8 @@ void Position::do_move(Move m,
// Update hash keys
// Zobrist::psq[pc][to] is zero, so we don't need to clear it
k ^= Zobrist::psq[promotion][to];
st->materialKey ^= Zobrist::psq[promotion][8 + pieceCount[promotion] - 1]
^ Zobrist::psq[pc][8 + pieceCount[pc]];
st->materialKey ^= Zobrist::psq[promotion][8 + pieceCount[promotion]]
^ Zobrist::psq[pc][8 + pieceCount[pc] - 1];
st->nonPawnKey[us] ^= Zobrist::psq[promotion][to];
if (pt <= BISHOP)
@@ -1625,7 +1625,7 @@ bool Position::material_key_is_ok() const { return compute_material_key() == st-
// This is meant to be helpful when debugging.
bool Position::pos_is_ok() const {
constexpr bool Fast = true; // Quick (default) or full check?
constexpr bool Fast = false; // fast or full check?
if ((sideToMove != WHITE && sideToMove != BLACK) || piece_on(square<KING>(WHITE)) != W_KING
|| piece_on(square<KING>(BLACK)) != B_KING