Crash when burning enemies military sites
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
widelands |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
When a medium military building is conquered, small military buildings very close to it burn down without a fight.
When multiple small military buildings are burned down at once, the game crashes due to a failed assertion (when releasing the area of the small sites):
assert(
The simplest fix seems to be to remove the assert(). It doesn't seems to be required for the correct functioning of the method and after removing it the game seems to work fine.
The linked branch applies this "fix". I haven't created a merge request yet since I am unsure whether we really want to fix it that way, but feel free to do so.
Related branches
- hessenfarmer: Approve
-
Diff: 11 lines (+0/-1)1 file modifiedsrc/logic/editor_game_base.cc (+0/-1)
Changed in widelands: | |
status: | Confirmed → In Progress |
Changed in widelands: | |
status: | In Progress → Fix Committed |
Multiplayer savegame to reproduce. Play as red player and attack the Barrier.