SDL_mixer icon indicating copy to clipboard operation
SDL_mixer copied to clipboard

BugFix: FIX MIX_CreateSineWaveAudio().

Open OvJat opened this issue 3 months ago • 1 comments

MIX_CreateSineWaveAudio() has a bug: MIX_LoadAudioWithProperties() require MIX_PROP_AUDIO_LOAD_IOSTREAM_POINTER, but MIX_CreateSineWaveAudio() not set it, MIX_LoadAudioWithProperties always fail.

I set a dummy SDL_IOStream to walk around it , and it works.

PS: The current Unit Test only cover a little code, shall we write high cover Unit Test using GoogleTest?

OvJat avatar Sep 21 '25 09:09 OvJat

Let me see if we can reasonably make this work without making a dummy iostream before we merge this.

icculus avatar Sep 21 '25 21:09 icculus