Skip to content

Decomp snd_note - #192

Merged
Eletonyy merged 7 commits into
mainfrom
sound-snd-note
Oct 29, 2026
Merged

Eletonyy merged 7 commits into
mainfrom
sound-snd-note

Conversation

@roengstrom

Copy link
Copy Markdown
Owner

No description provided.

Adds the two bank entry structs and the instrument and stream fields they read, the Psy-Q GetRCnt prototype and RCntCNT2 define, and prototypes for the snd_note.c copy functions, whose snd_init.c callers now pass pointers.
func_8001A57C copies words from one buffer to another, and func_8001A5FC swaps the words of two buffers.
It steps the engine's slides: the CD volume, the stream pan, the two global fades, the music volume, and the slides on the sound effect tracks.
func_8001AD60, the root counter 2 event handler, runs the engine's ticks, and func_8001AA28 advances one config's tracks for each tick.
func_8001BAA8 runs a track's opcodes up to its next note and sets that note up, with func_8001B1F4, func_8001B42C, func_8001B690 and func_8001B820. Two statements in it do nothing at run time but are needed for the match, and each says so.
@roengstrom roengstrom self-assigned this Oct 29, 2026
@decomp-dev

decomp-dev Bot commented Oct 29, 2026

Copy link
Copy Markdown

Report for SLUS_008.92 (fc8ad38 - 35cb7ad)

📈 Matched code: 68.60% (+0.47%, +7020 bytes)
📈 Matched data: 74.23% (+0.73%, +216 bytes)

✅ 11 new matches
Unit Item Bytes Before After
src/snd_note func_8001BAA8 +1792 0.00% 100.00%
src/snd_note func_8001AD60 +1172 0.00% 100.00%
src/snd_note func_8001A674 +948 0.00% 100.00%
src/snd_note func_8001AA28 +676 0.00% 100.00%
src/snd_note func_8001B820 +648 0.00% 100.00%
src/snd_note func_8001B42C +612 0.00% 100.00%
src/snd_note func_8001B1F4 +524 0.00% 100.00%
src/snd_note func_8001B690 +400 0.00% 100.00%
src/snd_note .rodata +216 0.00% 100.00%
src/snd_note func_8001A57C +128 0.00% 100.00%
src/snd_note func_8001A5FC +120 0.00% 100.00%

@Eletonyy
Eletonyy merged commit 501ba69 into main Oct 29, 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.

3 participants