diff options
| -rw-r--r-- | engines/simon/sound.cpp | 4 | ||||
| -rw-r--r-- | engines/sword2/sound.cpp | 7 | ||||
| -rw-r--r-- | sound/wave.cpp | 4 |
3 files changed, 6 insertions, 9 deletions
diff --git a/engines/simon/sound.cpp b/engines/simon/sound.cpp index f40a164a12..b3a7b54502 100644 --- a/engines/simon/sound.cpp +++ b/engines/simon/sound.cpp @@ -136,9 +136,11 @@ void WavSound::playSound(uint sound, Audio::SoundHandle *handle, byte flags) { error("playSound: Not a valid WAV file"); } + flags |= Audio::Mixer::FLAG_AUTOFREE | wavFlags; + byte *buffer = (byte *)malloc(size); _file->read(buffer, size); - _mixer->playRaw(handle, buffer, size, rate, flags | wavFlags); + _mixer->playRaw(handle, buffer, size, rate, flags); } void VocSound::playSound(uint sound, Audio::SoundHandle *handle, byte flags) { diff --git a/engines/sword2/sound.cpp b/engines/sword2/sound.cpp index 4e093d2a38..202a563fbb 100644 --- a/engines/sword2/sound.cpp +++ b/engines/sword2/sound.cpp @@ -270,13 +270,6 @@ int32 Sound::playFx(Audio::SoundHandle *handle, byte *data, uint32 len, uint8 vo return RDERR_INVALIDWAV; } - // The resource manager must have complete control over when resources - // are freed, or reference counting will break horribly. Besides, the - // data pointer is not valid for passing to free(). Why the hell is the - // AUTOFREE flag set by default anyway? - - flags &= ~Audio::Mixer::FLAG_AUTOFREE; - if (isReverseStereo()) flags |= Audio::Mixer::FLAG_REVERSE_STEREO; diff --git a/sound/wave.cpp b/sound/wave.cpp index f6ba3fed27..fdb358a1a9 100644 --- a/sound/wave.cpp +++ b/sound/wave.cpp @@ -111,7 +111,7 @@ bool loadWAVFromStream(Common::SeekableReadStream &stream, int &size, int &rate, // Prepare the return values. rate = samplesPerSec; - flags = Audio::Mixer::FLAG_AUTOFREE; + flags = 0; if (bitsPerSample == 8) // 8 bit data is unsigned flags |= Audio::Mixer::FLAG_UNSIGNED; else if (bitsPerSample == 16) // 16 bit data is signed little endian @@ -164,6 +164,8 @@ AudioStream *makeWAVStream(Common::SeekableReadStream &stream) { if (!loadWAVFromStream(stream, size, rate, flags, &type)) return 0; + flags |= Audio::Mixer::FLAG_AUTOFREE; + if (type == 17) // IMA ADPCM return makeADPCMStream(&stream, size, kADPCMIma, (flags & Audio::Mixer::FLAG_STEREO) ? 2 : 1); |