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

Animation script cleanup and fixes - Part 1 #3185

Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

Description

Cleans up animation script macro usage to use ANIM_x defines and convert hex numbers to positive and negative decimals.
For these, I used shasum to make sure it didn't break anything.

  • visible
  • monbg
  • invisible
  • createvisualtaskontargets
    • AnimTask_ShakeMon
  • createvisualtask
    • SoundTask_PlayDoubleCry
    • SoundTask_PlayCryHighPitch
    • AnimTask_WindUpLunge
    • AnimTask_TranslateMonEllipticalRespectSide
    • AnimTask_SwayMon
    • AnimTask_Splash
    • AnimTask_ShockWaveProgressingBolt
    • AnimTask_ShakeMonInPlace

It also fixes the following bad uses of macros (original checksum: 7a05d0b5b2cd65346597a9a598c7f7cde1570475)

  • createvisualtask
    • SoundTask_PlayCryWithEcho (checksum changed to 35eae7f4357391b1ce38f0d79715f47f92c82087) in:
      • Move_RELIC_SONG
      • Move_BOOMBURST
    • AnimTask_Splash (checksum changed to 99dd16cb37b2e592c80ee16f740c672fa511336d) in:
      • Move_ROCK_CLIMB
    • AnimTask_ShockWaveProgressingBolt (checksum changed to 774f3ef2f9ec10c740b4e6f45fa64167c13baa8b) in:
      • Move_PLASMA_FISTS

Discord contact info

asparaguseduardo

@AsparagusEduardo
Copy link
Collaborator Author

(I recommend this to be squashed)

@AsparagusEduardo AsparagusEduardo changed the title Animation script cleanup and fixes- Part 1 Animation script cleanup and fixes - Part 1 Jul 31, 2023
@DizzyEggg DizzyEggg merged commit 15c0ea2 into rh-hideout:upcoming Jul 31, 2023
1 check passed
@AsparagusEduardo AsparagusEduardo mentioned this pull request Sep 27, 2023
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