Skip to content

mkw: Match several KartObjectProxy methods - #15

Merged
em-eight merged 1 commit into
doldecomp:mainfrom
EricOhebshalom:kart-proxy-matches
Oct 16, 2026
Merged

em-eight merged 1 commit into
doldecomp:mainfrom
EricOhebshalom:kart-proxy-matches

Conversation

@EricOhebshalom

Copy link
Copy Markdown
Contributor

This PR matches four KartObjectProxy methods to 100.0%:

  • KartObjectProxy::isCpu() const (fix missing const qualifier on mangled symbol)
  • KartObjectProxy::getPlayerIdx() const
  • KartObjectProxy::getBodyForward(EGG::Vector3f& out)
  • KartObjectProxy::shouldVanishPlayer(bool vanish)

Also defines KART_FLAG_VANISH = 0x53 in KartFlags (KartState.hpp).

Verified with ninja and retail SHA-1 hash check:

  • build/RMCP01/main.dol: OK
  • build/RMCP01/StaticR/StaticR.rel: OK

- Match isCpu() const
- Match getPlayerIdx() const
- Match getBodyForward(EGG::Vector3f&)
- Match shouldVanishPlayer(bool)
- Define KART_FLAG_VANISH (0x53) in KartFlags
@decomp-dev

decomp-dev Bot commented Oct 15, 2026

Copy link
Copy Markdown

Report for RMCP01 (fa12730 - e0f5322)

📈 Matched code: 5.78% (+0.00%, +196 bytes)

✅ 4 new matches
Unit Item Bytes Before After
StaticR/kart/KartObjectProxy Kart::KartObjectProxy::shouldVanishPlayer(bool) +120 0.00% 100.00%
StaticR/kart/KartObjectProxy Kart::KartObjectProxy::getBodyForward(EGG::Vector3f&) +40 0.00% 100.00%
StaticR/kart/KartObjectProxy Kart::KartObjectProxy::isCpu() const +20 0.00% 100.00%
StaticR/kart/KartObjectProxy Kart::KartObjectProxy::getPlayerIdx() const +16 0.00% 100.00%

@em-eight

Copy link
Copy Markdown
Collaborator

Thanks! I will accept this PR, but as a future note unless we don't have any idea what a function does, we usually give a best-effort name to functions that are used

@em-eight
em-eight merged commit 8b9b5ee into doldecomp:main Oct 16, 2026
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