This is a small leftover from rewrites of old code. spawn_entities()
previously allowed its first parameter entities
to be nil
and just returned silently. I'm against such principle, that when a function is invoked, all conditions are correct. The main purpose of spawn_entities()
is ... to spawn entities. So at least one should be around.
I can rewrite this code part to only invoke that function as I saw one ruin in "base" ruin-set myself having no entities, but tiles. I have now expanded all these condition checks (variables first, then entities and tiles) by not only checking if it isn't nil
but also contains at least one item in it. 1.3.7 will address this issue.