Things like that should never be in a RMS but in a library and FeXoR had moved them in #3764.

Don't see any point with this in comparison to just working with the actual heightmap.

templates should be moved to the top and only one template per line, so that templte names can be exchanged without having to mess with code logic.

The array should be constructed in one statement, so that the result is immediately visible, rather than pushing one element after another with doing some stuff in between.

That magic number was hardcoded amongst many maps and only insiders know that the value 3 is hardcoded as a magic number in the pathfinder! #4034

Debug code must be and has been removed and replaced with a more general approach that works for every map, see #4950.

Terrible loop because it reinvents several wheels.
The following 160 lines can become 3 statements.

We have stayClasses to check for sufficient playerspace.
A HeightConstraint costs like 5 lines of code and is much more reusable than this manual check for heights.

Excluding minTerrainDistToBorder here is a mistake.
It is necessary to exclude only for entities, but not for terrain textures and actors!
It means you have water at the map border without water texture while one tile further away there is the water texture noticeable!

Bad duplication here, only the template is determined, the rest is the same! Fixed by rP21021

No reason to have the map square when you even add the code to place players only within the circular area!