Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix - Enemy Randomizer wrongly set flags #2253

Merged

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
@briaguya-ai
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.

@briaguya-ai briaguya-ai 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.

None yet

2 participants