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

Flat Merge Friday (Weekly merge) #2286

Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

Description

Conflicts:

  • include/battle.h
    • field_52 -> scriptPartyIdx
  • include/battle_controllers.h
    • Deleted unused functions.
  • src/battle_ai_script_commands.c
    • Deleted incoming file, as we deleted it in favor of C AI.
  • src/battle_ai_switch_items.c, src/battle_controller_opponent.c, src/battle_controller_player.c, src/battle_main.c, src/battle_script_commands.c, src/battle_util.c
    • Changed instances of ^ BIT_SIDE and ^ BIT_FLANK to BATTLE_OPPOSITE and BATTLE_PARTNER
  • src/battle_controllers.c, src/battle_message.c
    • field_52 -> scriptPartyIdx
    • bakScriptPartyIdx -> scriptPartyIdx

Additionally, looked for the rest of the instances of ^ BIT_SIDE and ^ BIT_FLANK not handled upstream and updated them.
SHA1 before and after merge: AC8764900956D00715D3264A446F239176267008

Discord contact info

AsparagusEduardo#6051

AsparagusEduardo and others added 29 commits August 18, 2022 14:23
Incorporate commented bug fixes in contest AI scripts
Automatized the contents of some sideof in src/item.c
Updated a couple of sizeofs in src/battle_pyramid_bag.c
Standardize judgement vs judgment spelling
Renamed TM flags to keep consistency with the rest of the repo
…0902

# Conflicts:
#	include/battle.h
#	include/battle_controllers.h
#	src/battle_ai_script_commands.c
#	src/battle_ai_switch_items.c
#	src/battle_controller_opponent.c
#	src/battle_controller_player.c
#	src/battle_controllers.c
#	src/battle_main.c
#	src/battle_message.c
#	src/battle_script_commands.c
#	src/battle_util.c
@AsparagusEduardo
Copy link
Collaborator Author

Btw, when I merge from upstream after opening the PR, I make sure that it still matches.

@ghoulslash ghoulslash merged commit 971081b into rh-hideout:master Sep 6, 2022
@AsparagusEduardo AsparagusEduardo deleted the RHH/pr/pretMerge_20220902 branch September 6, 2022 19:55
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.

4 participants