Skip to content

[Inagaki] Implement GameAudio::CircleSoundMgr - #77

Open
joakim-d wants to merge 1 commit into
doldecomp:mainfrom
joakim-d:reverse_circle_sound_mgr
Open

[Inagaki] Implement GameAudio::CircleSoundMgr#77
joakim-d wants to merge 1 commit into
doldecomp:mainfrom
joakim-d:reverse_circle_sound_mgr

Conversation

@joakim-d

@joakim-d joakim-d commented Aug 17, 2026

Copy link
Copy Markdown
Contributor
  • Implement GameAudio::CircleSoundMgr class

@decomp-dev

decomp-dev Bot commented Aug 17, 2026

Copy link
Copy Markdown

Report for MarioClub_us (55b9b9f - def5a21)

📈 Matched code: 46.70% (+0.09%, +2904 bytes)
📈 Linked code: 41.36% (+0.13%, +4096 bytes)
📈 Matched data: 24.47% (+0.04%, +336 bytes)
📈 Linked data: 31.86% (+0.06%, +548 bytes)

✅ 6 new matches
Unit Item Bytes Before After
main/Inagaki/CircleSoundMgr GameAudio::CircleSoundMgr::setSe(unsigned long) +1013 0.39% 100.00%
main/Inagaki/CircleSoundMgr GameAudio::CircleSoundMgr::calcPlayPos(unsigned long) +909 0.44% 100.00%
main/Inagaki/CircleSoundMgr GameAudio::CircleSoundMgr::loop() +608 0.65% 100.00%
main/Inagaki/CircleSoundMgr GameAudio::CircleSoundMgr::CircleSoundMgr(Vec*, float, float, float, JKRHeap*) +250 18.86% 100.00%
main/Inagaki/CircleSoundMgr .rodata +208 38.37% 100.00%
main/Inagaki/CircleSoundMgr GameAudio::CircleSoundMgr::frameWork() +53 7.14% 100.00%
📈 1 improvement in an unmatched item
Unit Item Bytes Before After
main/Inagaki/CircleSoundMgr .sdata2 +28 31.58% 90.91%

@joakim-d
joakim-d force-pushed the reverse_circle_sound_mgr branch from 970b6f6 to def5a21 Compare August 19, 2026 21:25

void setEcho(JAISoundHandle *handlePtr, f32 mix);
JAISoundHandle *startSoundCustom(u32 soundID, u32 p2);
virtual void loop();

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to move this method because of symbol order

@joakim-d joakim-d changed the title [DRAFT][Inagaki] Implement GameAudio::CircleSoundMgr [Inagaki] Implement GameAudio::CircleSoundMgr Aug 19, 2026
@joakim-d
joakim-d marked this pull request as ready for review August 19, 2026 21:28
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.

1 participant