M1 Map: Blessed and Broken failing

[MaddTheSane]

I encountered a lot of old Marathon 1 and M1A1 maps at https://archive.org/details/UltimateMarathonMaps specifically the file called "Marathon 1 + M1A1 Net + Solo Maps.zip" and have encountered a few that do not work Aleph one. Most of the errors are due to a "Detached polygon". Attached is the specific map mentioned, but there may be more: Blessed and Broken 1.0 - M1.zip

/Users/runner/work/alephone/alephone/Source_Files/GameWorld/flood_map.cpp:187: Assertion failed: !POLYGON_IS_DETACHED(polygon)

This does lead me to wonder how Aleph One should handle malformed maps that seemed to run fine under Marathon 1. Should we work around the buggy files or just flat-out refuse to run bad maps. How it handles it currently is an assertion and then a crash. Perhaps a warning about a bad map and going back to the main menu would be better?

[treellama]

Assertion failures aren't crashes, but yes, long term the goal should be to handle invalid maps with an error message and a return to the main screen. Many M1 maps were created in unofficial editors that set invalid flags, that's likely the case with Blessed and Broken.