Skip to content

Comment more original names for audio functions from animal crossing#2721

Open
Thar0 wants to merge 2 commits intozeldaret:mainfrom
Thar0:audio-orig-names
Open

Comment more original names for audio functions from animal crossing#2721
Thar0 wants to merge 2 commits intozeldaret:mainfrom
Thar0:audio-orig-names

Conversation

@Thar0
Copy link
Collaborator

@Thar0 Thar0 commented Mar 10, 2026

A follow-on to #2513
Comments original names for more files that largely remained intact into animal crossing

}

/**
* original name: Nas_StartVoice
Copy link
Contributor

Choose a reason for hiding this comment

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

A bit off topic, but I haven't been following the audio portion of decomp nearly as close as other parts of decomp because I haven't particularly studied how digital sound works. Like I've studied a little bit of music but just nowhere near comfortable to understand all that I'm looking at.

But seeing this made me question what all is within the Note struct, and having looked through and hardly understood even half of it, to me it seems like there's way more detail than the typical definition of a musical note (pitch and duration).

}

/**
* original name possibly Na_StartOcarinaBgm
Copy link
Collaborator

Choose a reason for hiding this comment

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

Where does this name come from? I can't find it in the repo, nor in ACreTeam/ac-decomp, nor in zeldaret/af, nor in the AC map files

Copy link
Collaborator Author

@Thar0 Thar0 Mar 13, 2026

Choose a reason for hiding this comment

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

JP versions of Majora's Mask have strings no other versions seem to have

$ strings baseroms/n64-jp-1.1/baserom-decompressed.z64 | grep Na_StartOc
Na_StartOcarinaSinglePlayCheck2( message->ocarina_no );
Na_StartOcarinaBgm Na_StartOcarinaBgm Na_StartOcarinaBgm

}

/**
* original name possibly "Nap_SilenceCheck"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Where does this name come from?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Guess, inferred from the function it calls being called Nap_SilenceCheck_Inner, it's the only sensible-looking candidate for a "non-inner" variant

include/audio.h Outdated
/* 0x10 */ SoundEffect* soundEffects;
} SoundFont; // size = 0x14

// JAudio: "delayparam"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you know where do the struct names and struct member names come from in the AC decomp?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't. Matching the structs was informative for me, but I was reluctant to label them as "original"

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// JAudio: "delayparam"

It turns out this name is made up

@Thar0 Thar0 requested a review from Dragorn421 March 17, 2026 13:09
@Dragorn421
Copy link
Collaborator

About the JAudio struct names, they mostly come from the map file (C++ mangled names) but some are made up.
From the gc/wii decomp discord https://discord.com/channels/727908905392275526/1055890752284151928/1483667080342536373
image
image
image
image

@Dragorn421 Dragorn421 added the One approval Has one approval, can be merged in 48 hours if nothing else comes up label Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

One approval Has one approval, can be merged in 48 hours if nothing else comes up

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants