Enemy rando cleanup#6327
Conversation
garrettjoecox
left a comment
There was a problem hiding this comment.
Love to see this. For what it's worth ShouldActorInit already exists and you should be able to use that instead of your VB_SPAWN_ACTOR_ENTRY and most of the rest of the newly introduced VBs.
I have a proof of concept using this on 2ship here https://github.com/HarbourMasters/2ship2harkinian/pull/1564/changes
I think it's better to do it this way as it prevents future accidental interference with other features that spawn enemies. |
|
getting all the headers fixed will be painful if it's working on your system. I'll push fix later today |
Aside from a general cleanup, greatly reduces the intrusiveness of enemy randomizer (no more extra parameter in
Actor_Spawn+ smaller stuff) & fixes the door in the forest temple upper stalfos fight opening too soon.Build Artifacts