audio: refactor audio asset loading, add crit sound #9

amr · 2026-04-06T21:53:12Z

amr commented

2026-04-06 21:53:12 +00:00

Created AudioAssets struct to hold the loaded sounds. It's part of the GameState to make it easy to access everywhere. If this is bad it can be refactored fairly easily.

amr added 1 commit

2026-04-06 21:53:12 +00:00

audio: refactor audio asset loading, add crit sound a315b14dd1

amr

2026-04-06 21:53:28 +00:00

added the
enhancement
label
requested review from NotAShelf

amr self-assigned this

2026-04-06 21:54:42 +00:00

NotAShelf reviewed

2026-04-07 17:52:46 +00:00

src/audio.c

					
				@ -95,3 +94,3 @@

				void audio_play_item_pickup(void) {

				void audio_play_item_pickup(GameState *gs) {

				  // High-pitched pickup sound

				  play_tone(800.0f, 0.15f, 0.4f);

Is the double sound playback intentional?

This plays both a procedural play_tone() and the loaded gs->sounds.pickup, simultaneously. Other refactored functions (audio_play_attack, audio_play_dodge) replaced procedural tones entirely with loaded sounds.

Is the double sound playback intentional? This plays both a procedural `play_tone()` and the loaded `gs->sounds.pickup`, simultaneously. Other refactored functions (`audio_play_attack`, `audio_play_dodge`) replaced procedural tones entirely with loaded sounds.