mirror of
https://github.com/sockspls/badfish
synced 2025-05-01 01:03:09 +00:00
94e41274bb
In this position we should have draw for repetition: position fen rnbqkbnr/2pppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1 moves g1f3 g8f6 f3g1 go infinite But latest patch broke it. Actually we had two(!) very subtle bugs, the first is that Position::set() clears the passed state and in particular 'previous' member, so that on passing setupStates, 'previous' pointer was reset. Second bug is even more subtle: SetupStates was based on std::vector as container, but when vector grows, std::vector copies all its contents to a new location invalidating all references to its entries. Because all StateInfo records are linked by 'previous' pointer, this made pointers go stale upon adding more element to setupStates. So revert to use a std::deque that ensures references are preserved when pushing back new elements. No functional change. |
||
|---|---|---|
| .. | ||
| syzygy | ||
| benchmark.cpp | ||
| bitbase.cpp | ||
| bitboard.cpp | ||
| bitboard.h | ||
| endgame.cpp | ||
| endgame.h | ||
| evaluate.cpp | ||
| evaluate.h | ||
| main.cpp | ||
| Makefile | ||
| material.cpp | ||
| material.h | ||
| misc.cpp | ||
| misc.h | ||
| movegen.cpp | ||
| movegen.h | ||
| movepick.cpp | ||
| movepick.h | ||
| pawns.cpp | ||
| pawns.h | ||
| position.cpp | ||
| position.h | ||
| psqt.cpp | ||
| search.cpp | ||
| search.h | ||
| thread.cpp | ||
| thread.h | ||
| thread_win32.h | ||
| timeman.cpp | ||
| timeman.h | ||
| tt.cpp | ||
| tt.h | ||
| types.h | ||
| uci.cpp | ||
| uci.h | ||
| ucioption.cpp | ||