upstream/wesnoth
1
0
Fork 0
mirror of https://github.com/wesnoth/wesnoth synced 2025-05-18 17:03:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix crash in mp lobby

Browse Source 
Fixes #4792

Previously that line would call generator::set_item_shown which inherits that class since set_item_shown is virtual. This doesn't work, in particular  since generator::set_item_shown wouldn't do anything if the to-be-deleted selected item is already hidden. (which for some reason happens quite often in the mp lobby window).

This resulted in generator::delete_item deleting a selecting item,  so selected_item_count_ would become wrong.
This commit is contained in:
gfgtdf 2020-08-10 16:10:22 +02:00 committed by Charles Dang
parent e9f62bcd66
commit 7e353095c1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/gui/widgets/generator.cpp
View File

@ -84,7 +84,7 @@ bool one_item::deselect_item(const unsigned index)
void one_item::delete_item(const unsigned index)
{
// this needs the same logic for ensuring that at least one item is selected
set_item_shown(index, false);
one_item::set_item_shown(index, false);
}
void no_item::set_item_shown(const unsigned index, const bool show)