aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--scumm/imuse_digi/dimuse.cpp4
-rw-r--r--scumm/smush/smush_mixer.cpp2
-rw-r--r--sound/mixer.cpp95
-rw-r--r--sound/mixer.h14
-rw-r--r--sword2/driver/d_sound.cpp2
5 files changed, 65 insertions, 52 deletions
diff --git a/scumm/imuse_digi/dimuse.cpp b/scumm/imuse_digi/dimuse.cpp
index bcda26a452..24f4393172 100644
--- a/scumm/imuse_digi/dimuse.cpp
+++ b/scumm/imuse_digi/dimuse.cpp
@@ -132,7 +132,7 @@ void IMuseDigital::callback() {
_vm->_mixer->playInputStream(&_track[l].handle, _track[l].stream2, true, _track[l].vol / 1000, _track[l].pan, -1, false);
} else {
_vm->_mixer->setChannelVolume(_track[l].handle, _track[l].vol / 1000);
- _vm->_mixer->setChannelPan(_track[l].handle, pan);
+ _vm->_mixer->setChannelBalance(_track[l].handle, pan);
}
continue;
}
@@ -186,7 +186,7 @@ void IMuseDigital::callback() {
if (_vm->_mixer->isReady()) {
_vm->_mixer->setChannelVolume(_track[l].handle, _track[l].vol / 1000);
- _vm->_mixer->setChannelPan(_track[l].handle, pan);
+ _vm->_mixer->setChannelBalance(_track[l].handle, pan);
_track[l].stream->append(data, result);
_track[l].regionOffset += result;
_track[l].trackOffset += result;
diff --git a/scumm/smush/smush_mixer.cpp b/scumm/smush/smush_mixer.cpp
index 7f8ee10722..d329804f5f 100644
--- a/scumm/smush/smush_mixer.cpp
+++ b/scumm/smush/smush_mixer.cpp
@@ -122,7 +122,7 @@ bool SmushMixer::handleFrame() {
if (!_channels[i].handle.isActive())
_mixer->newStream(&_channels[i].handle, rate, flags, 400000);
_mixer->setChannelVolume(_channels[i].handle, vol);
- _mixer->setChannelPan(_channels[i].handle, pan);
+ _mixer->setChannelBalance(_channels[i].handle, pan);
_mixer->appendStream(_channels[i].handle, data, size);
}
free(data);
diff --git a/sound/mixer.cpp b/sound/mixer.cpp
index 667904a53a..b31c2a6f25 100644
--- a/sound/mixer.cpp
+++ b/sound/mixer.cpp
@@ -46,7 +46,7 @@ private:
bool _autofreeStream;
const bool _isMusic;
byte _volume;
- int8 _pan;
+ int8 _balance;
bool _paused;
int _id;
@@ -56,8 +56,8 @@ protected:
public:
- Channel(SoundMixer *mixer, PlayingSoundHandle *handle, bool isMusic, byte volume, int8 pan, int id = -1);
- Channel(SoundMixer *mixer, PlayingSoundHandle *handle, AudioStream *input, bool autofreeStream, bool isMusic, byte volume, int8 pan, bool reverseStereo = false, int id = -1);
+ Channel(SoundMixer *mixer, PlayingSoundHandle *handle, bool isMusic, int id = -1);
+ Channel(SoundMixer *mixer, PlayingSoundHandle *handle, AudioStream *input, bool autofreeStream, bool isMusic, bool reverseStereo = false, int id = -1);
virtual ~Channel();
void mix(int16 *data, uint len);
@@ -74,14 +74,11 @@ public:
bool isPaused() {
return _paused;
}
- void setChannelVolume(const byte volume) {
+ void setVolume(const byte volume) {
_volume = volume;
}
- void setChannelPan(const int8 pan) {
- _pan = pan;
- }
- int getVolume() const {
- return isMusicChannel() ? _mixer->getMusicVolume() : _mixer->getVolume();
+ void setBalance(const int8 balance) {
+ _balance = balance;
}
int getId() const {
return _id;
@@ -90,7 +87,7 @@ public:
class ChannelStream : public Channel {
public:
- ChannelStream(SoundMixer *mixer, PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size, byte volume, int8 pan);
+ ChannelStream(SoundMixer *mixer, PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size);
void append(void *sound, uint32 size);
void finish();
@@ -137,9 +134,13 @@ void SoundMixer::setupPremix(PremixProc *proc, void *param) {
_premixProc = proc;
}
-void SoundMixer::newStream(PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size, byte volume, int8 pan) {
+void SoundMixer::newStream(PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size, byte volume, int8 balance) {
Common::StackLock lock(_mutex);
- insertChannel(handle, new ChannelStream(this, handle, rate, flags, buffer_size, volume, pan));
+
+ Channel *chan = new ChannelStream(this, handle, rate, flags, buffer_size);
+ chan->setVolume(volume);
+ chan->setBalance(balance);
+ insertChannel(handle, chan);
}
void SoundMixer::appendStream(PlayingSoundHandle handle, void *sound, uint32 size) {
@@ -215,7 +216,7 @@ void SoundMixer::insertChannel(PlayingSoundHandle *handle, Channel *chan) {
handle->setIndex(index);
}
-void SoundMixer::playRaw(PlayingSoundHandle *handle, void *sound, uint32 size, uint rate, byte flags, int id, byte volume, int8 pan, uint32 loopStart, uint32 loopEnd) {
+void SoundMixer::playRaw(PlayingSoundHandle *handle, void *sound, uint32 size, uint rate, byte flags, int id, byte volume, int8 balance, uint32 loopStart, uint32 loopEnd) {
Common::StackLock lock(_mutex);
// Prevent duplicate sounds
@@ -242,27 +243,29 @@ void SoundMixer::playRaw(PlayingSoundHandle *handle, void *sound, uint32 size, u
}
// Create the channel
- Channel *chan = new Channel(this, handle, input, true, false, volume, pan, (flags & SoundMixer::FLAG_REVERSE_STEREO) != 0, id);
+ Channel *chan = new Channel(this, handle, input, true, false, (flags & SoundMixer::FLAG_REVERSE_STEREO) != 0, id);
+ chan->setVolume(volume);
+ chan->setBalance(balance);
insertChannel(handle, chan);
}
#ifdef USE_MAD
-void SoundMixer::playMP3(PlayingSoundHandle *handle, File *file, uint32 size, byte volume, int8 pan, int id) {
+void SoundMixer::playMP3(PlayingSoundHandle *handle, File *file, uint32 size, byte volume, int8 balance, int id) {
// Create the input stream
AudioStream *input = makeMP3Stream(file, size);
- playInputStream(handle, input, false, volume, pan, id);
+ playInputStream(handle, input, false, volume, balance, id);
}
#endif
#ifdef USE_VORBIS
-void SoundMixer::playVorbis(PlayingSoundHandle *handle, File *file, uint32 size, byte volume, int8 pan, int id) {
+void SoundMixer::playVorbis(PlayingSoundHandle *handle, File *file, uint32 size, byte volume, int8 balance, int id) {
// Create the input stream
AudioStream *input = makeVorbisStream(file, size);
- playInputStream(handle, input, false, volume, pan, id);
+ playInputStream(handle, input, false, volume, balance, id);
}
#endif
-void SoundMixer::playInputStream(PlayingSoundHandle *handle, AudioStream *input, bool isMusic, byte volume, int8 pan, int id, bool autofreeStream) {
+void SoundMixer::playInputStream(PlayingSoundHandle *handle, AudioStream *input, bool isMusic, byte volume, int8 balance, int id, bool autofreeStream) {
Common::StackLock lock(_mutex);
if (input == 0) {
@@ -281,7 +284,9 @@ void SoundMixer::playInputStream(PlayingSoundHandle *handle, AudioStream *input,
}
// Create the channel
- Channel *chan = new Channel(this, handle, input, autofreeStream, isMusic, volume, pan, false, id);
+ Channel *chan = new Channel(this, handle, input, autofreeStream, isMusic, false, id);
+ chan->setVolume(volume);
+ chan->setBalance(balance);
insertChannel(handle, chan);
}
@@ -368,10 +373,10 @@ void SoundMixer::setChannelVolume(PlayingSoundHandle handle, byte volume) {
}
if (_channels[index])
- _channels[index]->setChannelVolume(volume);
+ _channels[index]->setVolume(volume);
}
-void SoundMixer::setChannelPan(PlayingSoundHandle handle, int8 pan) {
+void SoundMixer::setChannelBalance(PlayingSoundHandle handle, int8 balance) {
Common::StackLock lock(_mutex);
if (!handle.isActive())
@@ -380,12 +385,12 @@ void SoundMixer::setChannelPan(PlayingSoundHandle handle, int8 pan) {
int index = handle.getIndex();
if ((index < 0) || (index >= NUM_CHANNELS)) {
- warning("soundMixer::setChannelVolume has invalid index %d", index);
+ warning("soundMixer::setChannelBalance has invalid index %d", index);
return;
}
if (_channels[index])
- _channels[index]->setChannelPan(pan);
+ _channels[index]->setBalance(balance);
}
void SoundMixer::pauseAll(bool paused) {
@@ -458,17 +463,16 @@ void SoundMixer::setMusicVolume(int volume) {
#pragma mark -
-Channel::Channel(SoundMixer *mixer, PlayingSoundHandle *handle, bool isMusic,
- byte volume, int8 pan, int id)
+Channel::Channel(SoundMixer *mixer, PlayingSoundHandle *handle, bool isMusic, int id)
: _mixer(mixer), _handle(handle), _autofreeStream(true), _isMusic(isMusic),
- _volume(volume), _pan(pan), _paused(false), _id(id), _converter(0), _input(0) {
+ _volume(255), _balance(0), _paused(false), _id(id), _converter(0), _input(0) {
assert(mixer);
}
Channel::Channel(SoundMixer *mixer, PlayingSoundHandle *handle, AudioStream *input,
- bool autofreeStream, bool isMusic, byte volume, int8 pan, bool reverseStereo, int id)
+ bool autofreeStream, bool isMusic, bool reverseStereo, int id)
: _mixer(mixer), _handle(handle), _autofreeStream(autofreeStream), _isMusic(isMusic),
- _volume(volume), _pan(pan), _paused(false), _id(id), _converter(0), _input(input) {
+ _volume(255), _balance(0), _paused(false), _id(id), _converter(0), _input(input) {
assert(mixer);
assert(input);
@@ -496,26 +500,35 @@ void Channel::mix(int16 *data, uint len) {
} else {
assert(_converter);
- // The pan value ranges from -127 to +127. That's 255 different values.
- // From the channel pan/volume and the global volume, we compute the
- // effective volume for the left and right channel.
- // Note the slightly odd divisor: the 255 reflects the fact that
- // the maximal value for _volume is 255, while the 254 is there
- // because the maximal left/right pan value is 2*127 = 254.
- // The value getVolume() returns is in the range 0 - 256.
+ // From the channel balance/volume and the global volume, we compute
+ // the effective volume for the left and right channel. Note the
+ // slightly odd divisor: the 255 reflects the fact that the maximal
+ // value for _volume is 255, while the 127 is there because the
+ // balance value ranges from -127 to 127. The mixer (music/sound)
+ // volume is in the range 0 - 256.
// Hence, the vol_l/vol_r values will be in that range, too
- int vol = getVolume() * _volume;
- st_volume_t vol_l = (127 - _pan) * vol / (255 * 254);
- st_volume_t vol_r = (127 + _pan) * vol / (255 * 254);
+ int vol = (isMusicChannel() ? _mixer->getMusicVolume() : _mixer->getVolume()) * _volume;
+ st_volume_t vol_l, vol_r;
+
+ if (_balance == 0) {
+ vol_l = vol / 255;
+ vol_r = vol / 255;
+ } else if (_balance < 0) {
+ vol_l = vol / 255;
+ vol_r = ((127 + _balance) * vol) / (255 * 127);
+ } else {
+ vol_l = ((127 - _balance) * vol) / (255 * 127);
+ vol_r = vol / 255;
+ }
_converter->flow(*_input, data, len, vol_l, vol_r);
}
}
ChannelStream::ChannelStream(SoundMixer *mixer, PlayingSoundHandle *handle,
- uint rate, byte flags, uint32 buffer_size, byte volume, int8 pan)
- : Channel(mixer, handle, true, volume, pan) {
+ uint rate, byte flags, uint32 buffer_size)
+ : Channel(mixer, handle, true) {
// Create the input stream
_input = makeAppendableAudioStream(rate, flags, buffer_size);
diff --git a/sound/mixer.h b/sound/mixer.h
index 2297bc8edc..377d299ee7 100644
--- a/sound/mixer.h
+++ b/sound/mixer.h
@@ -105,19 +105,19 @@ public:
// start playing a raw sound
void playRaw(PlayingSoundHandle *handle, void *sound, uint32 size, uint rate, byte flags,
- int id = -1, byte volume = 255, int8 pan = 0, uint32 loopStart = 0, uint32 loopEnd = 0);
+ int id = -1, byte volume = 255, int8 balance = 0, uint32 loopStart = 0, uint32 loopEnd = 0);
#ifdef USE_MAD
- void playMP3(PlayingSoundHandle *handle, File *file, uint32 size, byte volume = 255, int8 pan = 0, int id = -1);
+ void playMP3(PlayingSoundHandle *handle, File *file, uint32 size, byte volume = 255, int8 balance = 0, int id = -1);
#endif
#ifdef USE_VORBIS
- void playVorbis(PlayingSoundHandle *handle, File *file, uint32 size, byte volume = 255, int8 pan = 0, int id = -1);
+ void playVorbis(PlayingSoundHandle *handle, File *file, uint32 size, byte volume = 255, int8 balance = 0, int id = -1);
#endif
- void playInputStream(PlayingSoundHandle *handle, AudioStream *input, bool isMusic, byte volume = 255, int8 pan = 0, int id = -1, bool autofreeStream = true);
+ void playInputStream(PlayingSoundHandle *handle, AudioStream *input, bool isMusic, byte volume = 255, int8 balance = 0, int id = -1, bool autofreeStream = true);
/** Start a new stream. */
- void newStream(PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size, byte volume = 255, int8 pan = 0);
+ void newStream(PlayingSoundHandle *handle, uint rate, byte flags, uint32 buffer_size, byte volume = 255, int8 balance = 0);
/** Append to an existing stream. */
void appendStream(PlayingSoundHandle handle, void *sound, uint32 size);
@@ -151,8 +151,8 @@ public:
/** set the channel volume for the given handle (0 - 255) */
void setChannelVolume(PlayingSoundHandle handle, byte volume);
- /** set the channel pan for the given handle (-127 ... 0 ... 127) (left ... center ... right)*/
- void setChannelPan(PlayingSoundHandle handle, int8 pan);
+ /** set the channel balance for the given handle (-127 ... 0 ... 127) (left ... center ... right)*/
+ void setChannelBalance(PlayingSoundHandle handle, int8 balance);
/** Check whether any SFX channel is active.*/
bool hasActiveSFXChannel();
diff --git a/sword2/driver/d_sound.cpp b/sword2/driver/d_sound.cpp
index 41d7630b07..83af1fc3a2 100644
--- a/sword2/driver/d_sound.cpp
+++ b/sword2/driver/d_sound.cpp
@@ -952,7 +952,7 @@ int32 Sound::setFxIdVolumePan(int32 id, uint8 vol, int8 pan) {
if (!_fxMuted) {
_vm->_mixer->setChannelVolume(_fx[i]._handle, _fx[i]._volume * _fxVol);
- _vm->_mixer->setChannelPan(_fx[i]._handle, _panTable[pan + 16]);
+ _vm->_mixer->setChannelBalance(_fx[i]._handle, _panTable[pan + 16]);
}
return RD_OK;