ap7mp/solitare
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Fix win simulation unnecessarily drawing same card

Browse source
This commit is contained in:
ItsDrike 2024-12-07 12:02:55 +01:00
parent 0bee5ddc12
commit 1eb72163b5
Signed by: ItsDrike
GPG key ID: FA2745890B7048C0
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/gamestate.cpp
View file

@ -703,7 +703,9 @@ std::optional<bool> GameState::canWinThroughSimulation(QSet<QString>& visitedSta
}
// Simulate draw pile move
if (!(m_drawPile.isEmpty() && m_throwawayPile.isEmpty())) {
// (The condition also handles the case where there's only one card in the throwaway pile,
// which means drawing would just result in flipping and re-drawing the same card.)
if (!(m_drawPile.isEmpty() && m_throwawayPile.size() <= 1)) {
GameState* clonedState = this->clone();
assert(clonedState->drawNextCard());
assert(clonedState->generateStateHash() != generateStateHash());