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

Removed pointless for loop in CB2_InitBattleInternal #3422

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

LOuroboros
Copy link
Collaborator

Description

Title.
Yesterday, I was looking at CB2_InitBattleInternal for reasons and I noticed that it was running 2 for loops to apply changes on the Pokémon in the Player's and the Opponent's parties.
However, it only really needs to run 1, as far as I can tell.
This PR corrects that.

Discord contact info

lunos4026

Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo left a comment

Choose a reason for hiding this comment

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

This doesn't particularly fix anything, just cleans it up, so I would move it to upcoming

@LOuroboros
Copy link
Collaborator Author

This doesn't particularly fix anything, just cleans it up, so I would move it to upcoming

It's a correction. Since it's not a new feature I figured Master was the right place for it?

Whatever, I don't care. Will rebase.

@AsparagusEduardo
Copy link
Collaborator

To put it on record, I see code cleanup as part of upcoming.

@LOuroboros LOuroboros changed the base branch from master to upcoming October 14, 2023 15:56
@AsparagusEduardo AsparagusEduardo merged commit 0f979ac into upcoming Oct 14, 2023
1 check passed
@AsparagusEduardo AsparagusEduardo deleted the LOuroboros-patch-1 branch October 14, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants