libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2025-01-22 10:17:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Ok, I'm stupid.

Browse Source 
The initial sample is, indeed, two bytes, just like the rest of them, but
it really, really helps if you read it from the correct position in the
file.

After fixing that, it turned out that my changing of signedness of the
sample was also wrong. Funny how those two bugs almost cancelled each other
out. Almost.

I've made a few other changes as well, but they're just to clean things up
a bit. The credits music works for me, and I've played the game up to
arriving in Quaramonte, with no obvious music-related problems.

svn-id: r10412
This commit is contained in:
Torbjörn Andersson 2003-09-26 06:26:18 +00:00
parent c03b99eab6
commit b7a5024daf
2 changed files with 36 additions and 45 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

74
sword2/driver/d_sound.cpp
View File

@ -181,26 +181,44 @@ int32 musicVolTable[17] = {
int16 MusicHandle::read() {
uint8 in;
uint16 delta, value;
uint16 delta;
int16 out;
if (!_streaming)
return 0;
if (_firstTime) {
_lastSample = fpMus.readUint16LE();
_filePos += 2;
_firstTime = false;
return _lastSample;
}
// Assume the file handle has been correctly positioned already.
in = fpMus.readByte();
delta = GetCompressedAmplitude(in) << GetCompressedShift(in);
if (GetCompressedSign(in))
value = _lastSample - delta;
out = _lastSample - delta;
else
value = _lastSample + delta;
out = _lastSample + delta;
_filePos++;
_lastSample = value;
_lastSample = out;
out = (int16) (value ^ 0x8000);
if (_looping) {
if (_filePos >= _fileEnd) {
_firstTime = true;
_filePos = _fileStart;
}
} else {
// Fade out at the end of the music. Is this really desirable
// behaviour?
if (_fileEnd - _filePos <= FADE_SAMPLES)
_fading = _fileEnd - _filePos;
}
if (_fading > 0) {
if (--_fading == 0) {
@ -879,6 +897,7 @@ int32 Sword2Sound::StreamCompMusic(const char *filename, uint32 musicId, bool lo
}
int32 Sword2Sound::StreamCompMusicFromLock(const char *filename, uint32 musicId, bool looping) {
uint32 len;
int32 primaryStream = -1;
int32 secondaryStream = -1;
@ -934,60 +953,29 @@ int32 Sword2Sound::StreamCompMusicFromLock(const char *filename, uint32 musicId,
music[secondaryStream]._fading = FADE_SAMPLES;
fpMus.seek((musicId + 1) * 8, SEEK_SET);
music[primaryStream]._filePos = fpMus.readUint32LE();
music[primaryStream]._fileEnd = fpMus.readUint32LE();
music[primaryStream]._fileStart = fpMus.readUint32LE();
len = fpMus.readUint32LE();
if (!music[primaryStream]._filePos || !music[primaryStream]._fileEnd)
if (!music[primaryStream]._fileStart || !len)
return RDERR_INVALIDID;
// Calculate the file position of the end of the music
music[primaryStream]._fileEnd += music[primaryStream]._filePos;
// We used to read two bytes for _lastSample, but doing that breaks
// some of the music, so I guess that was a bug. Maybe.
music[primaryStream]._lastSample = fpMus.readByte();
music[primaryStream]._filePos++;
music[primaryStream]._fileEnd = music[primaryStream]._fileStart + len;
music[primaryStream]._filePos = music[primaryStream]._fileStart;
music[primaryStream]._streaming = true;
music[primaryStream]._firstTime = true;
return RD_OK;
}
void Sword2Sound::UpdateCompSampleStreaming(int16 *data, uint len) {
uint32 i;
for (i = 0; i < MAXMUS; i++) {
for (int i = 0; i < MAXMUS; i++) {
if (!music[i]._streaming)
continue;
// Modify the volume according to the master volume and music
// mute state
byte volume = musicMuted ? 0 : musicVolTable[musicVol];
fpMus.seek(music[i]._filePos, SEEK_SET);
_converter->flow(music[i], data, len, volume, volume);
if (music[i].eos()) {
// End of the music so we need to start fading and
// start the music again
// FIXME: The original code faded the music here, but
// to do that we need to start before we reach the end
// of the file.
//
// On the other hand, do we want to fade out the end
// of the music?
// Fade the old music
// musFading[i] = -16;
// Loop if neccassary
if (music[i]._looping) {
music[i]._streaming = false;
StreamCompMusicFromLock(music[i]._fileName, music[i]._id, music[i]._looping);
}
}
}
// FIXME: We need to implement DipMusic()'s functionality, but since

7
sword2/driver/d_sound.h
View File

@ -56,10 +56,12 @@ class MusicHandle : public MusicStream {
public:
uint32 _id;
char _fileName[256];
bool _firstTime;
bool _streaming;
bool _paused;
bool _looping;
int32 _fading;
int32 _fileStart;
int32 _filePos;
int32 _fileEnd;
uint16 _lastSample;
@ -70,8 +72,9 @@ public:
int16 read();
bool eos() const;
MusicHandle() : MusicStream(), _streaming(false), _paused(false),
_looping(false), _fading(0), _filePos(0), _fileEnd(0),
MusicHandle() : MusicStream(), _firstTime(false), _streaming(false),
_paused(false), _looping(false), _fading(0),
_fileStart(0), _filePos(0), _fileEnd(0),
_lastSample(0) {
_fileName[0] = 0;
}