Skip to content

Fix - Enemy Randomizer wrongly set flags#2253

Merged
briaguya0 merged 1 commit intoHarbourMasters:develop-bradleyfrom
aMannus:fix-enemizer-flags
Dec 24, 2022
Merged

Fix - Enemy Randomizer wrongly set flags#2253
briaguya0 merged 1 commit intoHarbourMasters:develop-bradleyfrom
aMannus:fix-enemizer-flags

Conversation

@aMannus
Copy link
Contributor

@aMannus aMannus commented Dec 23, 2022

Both redeads/gibdo's and Iron Knuckles were setting flags that overlapped with the flags already used by other objects in the game.

Notably, the web in Deku Tree would already be lit when killing an Iron Knuckle before, and killing a redead caused the "platforms dropping" cutscene when killing a redead in the central room. Spirit temple torches would be incorrectly lit, Fire Temple cracked floor would already be open before bombing it, all because of similar reasons.

Iron knuckles already were changed to use the "clear room" flag to properly despawn them in Enemy Rando, so removing this flag doesn't break anything.

I honestly couldn't even find anything in the game's code that uses this Redead flag, but they seem to work just fine in clear enemy rooms, so I'm not sure what's up with this one? Regardless, just to be safe, it's only prevented from being set in Enemy Randomizer.

Build Artifacts

@aMannus aMannus mentioned this pull request Dec 23, 2022
15 tasks
@briaguya0
Copy link
Contributor

assuming this should be pointed at bradley

@aMannus aMannus changed the base branch from develop to develop-bradley December 23, 2022 22:11
@aMannus
Copy link
Contributor Author

aMannus commented Dec 23, 2022

assuming this should be pointed at bradley

Woops, thanks.

@briaguya0 briaguya0 merged commit f6a7f3d into HarbourMasters:develop-bradley Dec 24, 2022
@aMannus aMannus deleted the fix-enemizer-flags branch December 24, 2022 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants