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

Implement the rest of ResultScreen #312

Merged
merged 15 commits into from
Nov 14, 2024

Conversation

Renzo904
Copy link
Member

Implements:

  • ResultScreen::OnDraw
  • ResultScreen::LinkScore
  • ResultScreen::FreeScore
  • ResultScreen::FreeAllScores
  • ResultScreen::ReleaseScoreDat
  • ResultScreen::LinkScoreEx
  • ResultScreen::DrawFinalStats
  • ResultScreen::HandleResultKeyboard
  • ResultScreen::HandleReplaySaveKeyboard
  • ResultScreen::WriteScore
  • ResultScreen::DeletedCallback
  • ResultScreen::GetHighScore

Plus, i reordered all the functions inside, so that they match against the original binary

@roblabla
Copy link
Member

There are some weird diffs in HandleReplaySaveKeyboard: https://github.com/happyhavoc/th06/actions/runs/11760974310#user-content-th06____resultscreen____handlereplaysavekeyboard

I'll merge this regardless, the diff can be fixed in a follow-up PR. Thanks for the contribution!

@roblabla roblabla merged commit 11d3cfb into happyhavoc:master Nov 14, 2024
6 checks passed
@Renzo904
Copy link
Member Author

There are some weird diffs in HandleReplaySaveKeyboard: https://github.com/happyhavoc/th06/actions/runs/11760974310#user-content-th06____resultscreen____handlereplaysavekeyboard

That's funny, that is not a diff, that is ghidra delinker(i think), treating 0x606060 as a memory address, instead of a RGB value, so it generates a relocation

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