aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTravis Howell2006-03-19 05:15:49 +0000
committerTravis Howell2006-03-19 05:15:49 +0000
commit6632a4df53bd6cd94d85057ceae10f14108a67e4 (patch)
treecb6f9737bd9b3f1abe88a0565b585d16de48b023
parent8ae4dd6caa4bcca13503d6a024e4ca5d75ff4808 (diff)
downloadscummvm-rg350-6632a4df53bd6cd94d85057ceae10f14108a67e4.tar.gz
scummvm-rg350-6632a4df53bd6cd94d85057ceae10f14108a67e4.tar.bz2
scummvm-rg350-6632a4df53bd6cd94d85057ceae10f14108a67e4.zip
Revert FLAG_AUTOFREE changes to loadWAVFromStreammakeWAVStream
svn-id: r21376
-rw-r--r--engines/simon/sound.cpp4
-rw-r--r--engines/sword2/sound.cpp7
-rw-r--r--sound/wave.cpp4
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);