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

"If you wish for it, you can fly" (pret merge 2023-12-10) #3675

Conversation

AsparagusEduardo
Copy link
Collaborator

Description

Due to linker changes on pret's side, the only main differences on agbcc are veneer changes (reordering of elements). Besides that and some label changes everything matches in agbcc.
image

agbcc SHA1 before merge: f9796c395d49742b8a279f3bee34314225c5a18c
agbcc SHA1 after merge: 2d55d8991dc04460ef2d7bc38010be16829900c5

modern SHA1 before merge: bac2955bb02d13e11cdef5cf3e2f5903769ff5e5
modern SHA1 after merge: fc553370a5724f1f9143b5eea72a647d7d27fce7

Discord contact info

AsparagusEduardo

GriffinRichards and others added 24 commits October 20, 2023 12:50
"SendMonToPC" implies that the Pokémon affected is actually sent over to the PC, but in reality the function simply copies the data of the Pokémon and then pastes it in the first available slot of the Pokémon Storage System.
…mands, the call to TypeCalc does not assign effectiveness flags properly, resulting in the check for immunity always failing
Added BUGFIX for "dual non-immunity" glitch.
Delete the redundant files generated after running build
Rename Undiscovered to No Eggs Discovered
Renamed SendMonToPC to CopyMonToPC instead
Rename HITMARKER_IGNORE_SAFEGUARD to HITMARKER_STATUS_ABILITY_EFFECT.
This flag is used exclusively by status-inflicting Abilities,
and has 3 main functions:
- Whether the effect bypasses Shield Dust
- Whether the effect bypasses Safeguard
- Which text string to display when the status condition is inflicted
   (i.e. whether it was inflicted by a move or Ability)

I believe this new name better represents what the flag actually does.
This name was structured to parallel HITMARKER_SYNCHRONISE_EFFECT.
Rename HITMARKER_IGNORE_SAFEGUARD
Moved files - spinda spots, jp fonts, redyellowgreen_frame.bin
[Build System Rewrite] Linkerscript now tracks RAM/ROM usage
Updated the size of the struct InGameTrade's otName variable
…_20231210

# Conflicts:
#	Makefile
#	data/battle_scripts_1.s
#	gflib/malloc.c
#	gflib/malloc.h
#	include/battle.h
#	ld_script.ld
#	ld_script_modern.ld
#	src/battle_ai_script_commands.c
#	src/battle_main.c
#	src/battle_script_commands.c
#	src/battle_util.c
#	src/data/pokemon/species_info.h
#	src/pokemon.c
# Conflicts:
#	src/data/pokemon/species_info.h
@AsparagusEduardo AsparagusEduardo merged commit 237c156 into rh-hideout:upcoming Dec 10, 2023
1 check passed
@AsparagusEduardo AsparagusEduardo deleted the RHH/pr/upcoming/pret_20231210 branch December 10, 2023 17:07
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.

9 participants