Fix occasional crash from placing wall stairs in an invalid location

wheybags / freeablo

[ARCHIVED] Modern reimplementation of the Diablo 1 game engine

GNU General Public License v3.0

2.16k stars 195 forks source link

Fix occasional crash from placing wall stairs in an invalid location #426

Closed grantramsay closed 5 years ago

grantramsay commented 5 years ago

Dun::pointIsValid was not being checked when generating level stairs Added a helper placement constraint checking function

grantramsay commented 5 years ago

FYI: Also noticed a different random crash when changing level: ASSERTION FAILED: (!blocking) in ../apps/freeablo/faworld/gamelevel.cpp:118 I wasn't debugging at the time, maybe a monster walked onto the stairs?

wheybags commented 5 years ago

Great, thanks! As for the other crash, well, hopefully someone will run into it with a debugger attached and find a fix ¯\_(ツ)_/¯