-
Notifications
You must be signed in to change notification settings - Fork 106
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 struct cleanup #122
Conversation
…oThis # Conflicts: # src/battle_arcade_game_board.c # src/battle_command.c # src/overlay_94.c # src/scrcmd_move_relearner.c # src/scrcmd_party.c # src/scrcmd_pokemon_misc.c # src/use_item_on_mon.c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
generally speaking, I prefer pokemon instead of mon for the variable names, since it's more explicit.
there is a discussion to be had about typedef struct vs explicit struct, I think atm we're leaning towards the latter
The renaming is good though, all caps struct names just seem weird
…sDoThis # Conflicts: # include/field_blackthorn_tutors.h # include/get_egg.h # src/get_egg.c # src/hall_of_fame.c # src/scrcmd_move_relearner.c
Solved conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only thing left is mon -> pokemon, then I'm happy to merge
POKEMON
toPokemon
.MON_DATA
names.Pokemon
pointers.Pokemon
.AsparagusEduardo#6051