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

bug: fix manual cross-column movement logic

Browse source 
The original implementation didn't use references for the QList
instances, which meant they were getting copied, so the changes made
didn't mutate the actual values held by the class.
This commit is contained in:
ItsDrike 2024-12-06 17:08:32 +01:00
parent 5240949353
commit 18891308bc
Signed by: ItsDrike
GPG key ID: FA2745890B7048C0
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/gamestate.cpp
View file

@ -184,8 +184,8 @@ bool GameState::moveThrownCardToFoundation(int foundationId) {
bool GameState::moveColumnCardToColumn(int fromColumnId, int toColumnId, int fromCardIndex) {
assert(fromColumnId >= 0 && fromColumnId < 7);
assert(toColumnId >= 0 && toColumnId < 7);
auto fromColumnStack = m_columns[fromColumnId];
auto toColumnStack = m_columns[toColumnId];
auto& fromColumnStack = m_columns[fromColumnId];
auto& toColumnStack = m_columns[toColumnId];
if (fromColumnStack.isEmpty()) {
qWarning() << "Attempted to move card(s) to column from an empty column";