Skip to content

Conversation

@ryanbevins
Copy link
Contributor

@ryanbevins ryanbevins commented Feb 13, 2026

Summary

  • Decompile Animal/AnimalManager.cpp — 5/7 functions at 100% match
  • Add include/Animal/AnimalManager.hpp with TAnimalManagerBase and TMewManager class definitions

Notes

Mostly matching. Remaining issues seem unsolvable at the moment:

  • loadAfter: frame size +16 (likely from an unknown inline wrapper around createRandPlayVec)
  • load: frame size +40 (same class of issue)
  • clipEnemies: GPR register allocation shift (similar to known FPR f30/f31 swap pattern)

@decomp-dev
Copy link

decomp-dev bot commented Feb 13, 2026

Report for GMSJ01 (d0a2812 - c00aee1)

📈 Matched code: 26.64% (+0.03%, +1012 bytes)
📈 Matched data: 29.16% (+0.06%, +364 bytes)

✅ 8 new matches
Unit Item Bytes Before After
mario/Animal/AnimalManager __sinit_AnimalManager_cpp +764 0.00% 100.00%
mario/Animal/AnimalManager .bss +184 0.00% 100.00%
mario/Animal/AnimalManager .data +168 0.00% 100.00%
mario/Animal/AnimalManager TMewManager::~TMewManager() +116 0.00% 100.00%
mario/Animal/AnimalManager TAnimalManagerBase::TAnimalManagerBase(const char*) +80 0.00% 100.00%
mario/Animal/AnimalManager TMewManager::createModelData() +52 0.00% 100.00%
mario/Animal/AnimalManager .sdata2 +8 0.00% 100.00%
mario/Animal/AnimalManager .ctors +4 0.00% 100.00%
📈 4 improvements in unmatched items
Unit Item Bytes Before After
mario/Animal/AnimalManager .rodata +267 0.00% 95.52%
mario/Animal/AnimalManager TAnimalManagerBase::clipEnemies(JDrama::TGraphics*) +148 0.00% 57.81%
mario/Animal/AnimalManager TMewManager::load(JSUMemoryInputStream&) +95 0.00% 77.42%
mario/Animal/AnimalManager TMewManager::loadAfter() +40 0.00% 66.67%

Mostly matching - remaining issues (loadAfter frame +16, load frame +40,
clipEnemies register allocation shift) seem unsolvable at the moment.
@Mrkol Mrkol merged commit e7a9a50 into doldecomp:main Feb 13, 2026
2 checks 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