Skip to content

Conversation

DavidJCobb
Copy link
Contributor

Named the task and sprite fields related to the battle animations for Hail, Rain Dance, and Sandstorm. Tested via make clean && make -j16 && make compare; the hash matched.

Description

  • Identified HailStruct::unk3 as a position type: fixed (i.e. always uses the position in the struct), negative (negative offset applied to a battler position, if battler is present; position in the struct otherwise), or positive. Renamed the field to type, and added an enum for the values.
  • Used #define and #undef for the task data and sprite data for these three moves.
  • Some integer constants, AFAIK, "should" be ARRAY_COUNT, but using that macro causes the build to fail to match, so I left the constants untouched and noted the equivalence in line comments.

Discord contact info

@davidjcobb

Named task and sprite fields related to the animations for Hail, Rain Dance, and Sandstorm.
@DavidJCobb
Copy link
Contributor Author

Yep, casting to int fixed it. Thanks for the tip!

@GriffinRichards GriffinRichards merged commit 263026e into pret:master May 2, 2025
1 check passed
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