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

Pokemon Emerald: Remove unnecessary code #4364

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

Zunawe
Copy link
Collaborator

@Zunawe Zunawe commented Dec 11, 2024

What is this fixing or adding?

#4329

Checked in the debugger, the key that it sets is some warp in a regi cave. It gets set to None on the first loop and then overridden later. So this code is effectively dead and has no intended function anyway.

How was this tested?

Generated after deleting the code.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Dec 11, 2024
@Zunawe Zunawe added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Dec 12, 2024
Copy link
Contributor

@nicholassaylor nicholassaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code is indeed dead, LGTM

@Berserker66 Berserker66 merged commit cacab68 into ArchipelagoMW:main Dec 16, 2024
16 checks passed
@Zunawe Zunawe deleted the emerald-rm-extra-lines branch December 16, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants