jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-07-04 04:23:55 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

remove break statement when closing views (#6801)

Browse source 
it's possible that multiple views need to be closed
this is an oversight of #4570
This commit is contained in:
ebbit1q 2026-04-12 02:37:43 +02:00 • committed by GitHub
parent d677e2bb70
commit 92fe406c22
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
cockatrice/src/game/player/player_event_handler.cpp
View file

@ -60,7 +60,6 @@ void PlayerEventHandler::eventShuffle(const Event_Shuffle &event)
// we want to close empty views as well // we want to close empty views as well
if (length == 0 || length > absStart) { // note this assumes views always start at the top of the library if (length == 0 || length > absStart) { // note this assumes views always start at the top of the library
view->close(); view->close();
break;
} }
} else { } else {
qWarning() << zone->getName() << "of" << player->getPlayerInfo()->getName() << "holds empty zoneview!"; qWarning() << zone->getName() << "of" << player->getPlayerInfo()->getName() << "holds empty zoneview!";