From e6d6a71599419908afd39c704ccda0cf5e475dea Mon Sep 17 00:00:00 2001 From: tma Date: Wed, 16 Nov 2005 20:23:37 +0000 Subject: * Partial implementation of FS_Seek for files in pk3s * A couple of RIFF decoder tweaks/fixes git-svn-id: svn://svn.icculus.org/quake3/trunk@363 edf5b092-35ff-0310-97b2-ce42778d08ea --- code/client/snd_codec_wav.c | 53 +++++++++++++++++++++------------------------ 1 file changed, 25 insertions(+), 28 deletions(-) (limited to 'code/client') diff --git a/code/client/snd_codec_wav.c b/code/client/snd_codec_wav.c index 146d526..5a808d8 100644 --- a/code/client/snd_codec_wav.c +++ b/code/client/snd_codec_wav.c @@ -66,44 +66,38 @@ static int S_ReadChunkInfo(fileHandle_t f, char *name) return 0; len = FGetLittleLong(f); - if(len < 0 || len > 0xffffffff) + if( len < 0 ) { + Com_Printf( S_COLOR_YELLOW "WARNING: Negative chunk length\n" ); return 0; - - //FIXME: 11/11/05 - // I'm not sure I understand why this needs to be padded. - // Surely this results in reading past the end of the data? - //len = (len + 1 ) & ~1; // pad to word boundary + } return len; } /* ================= -S_FindWavChunk +S_FindRIFFChunk Returns the length of the data in the chunk, or 0 if not found ================= */ -static int S_FindWavChunk( fileHandle_t f, char *chunk ) { +static int S_FindRIFFChunk( fileHandle_t f, char *chunk ) { char name[5]; int len; - // This is a bit dangerous... - while(1) + while( ( len = S_ReadChunkInfo(f, name) ) ) { - len = S_ReadChunkInfo(f, name); - - // Read failure? - if(len == 0) - return 0; - // If this is the right chunk, return - if(!strcmp(name, chunk)) + if( !Q_strncmp( name, chunk, 4 ) ) return len; + len = (len + 1 ) & ~1; // pad to word boundary + // Not the right chunk - skip it FS_Seek( f, len, FS_SEEK_CUR ); } + + return 0; } /* @@ -131,10 +125,10 @@ static void S_ByteSwapRawSamples( int samples, int width, int s_channels, const /* ================= -S_ReadWavHeader +S_ReadRIFFHeader ================= */ -static qboolean S_ReadWavHeader(fileHandle_t file, snd_info_t *info) +static qboolean S_ReadRIFFHeader(fileHandle_t file, snd_info_t *info) { char dump[16]; int wav_format; @@ -144,9 +138,9 @@ static qboolean S_ReadWavHeader(fileHandle_t file, snd_info_t *info) FS_Read(dump, 12, file); // Scan for the format chunk - if((fmtlen = S_FindWavChunk(file, "fmt ")) == 0) + if((fmtlen = S_FindRIFFChunk(file, "fmt ")) == 0) { - Com_Printf("No fmt chunk\n"); + Com_Printf( S_COLOR_RED "ERROR: Couldn't find \"fmt\" chunk\n"); return qfalse; } @@ -167,9 +161,9 @@ static qboolean S_ReadWavHeader(fileHandle_t file, snd_info_t *info) } // Scan for the data chunk - if( (info->size = S_FindWavChunk(file, "data")) == 0) + if( (info->size = S_FindRIFFChunk(file, "data")) == 0) { - Com_Printf("No data chunk\n"); + Com_Printf( S_COLOR_RED "ERROR: Couldn't find \"data\" chunk\n"); return qfalse; } info->samples = (info->size / info->width) / info->channels; @@ -202,15 +196,17 @@ void *S_WAV_CodecLoad(const char *filename, snd_info_t *info) FS_FOpenFileRead(filename, &file, qtrue); if(!file) { - Com_Printf("Can't read sound file %s\n", filename); + Com_Printf( S_COLOR_RED "ERROR: Could not open \"%s\"\n", + filename); return NULL; } // Read the RIFF header - if(!S_ReadWavHeader(file, info)) + if(!S_ReadRIFFHeader(file, info)) { FS_FCloseFile(file); - Com_Printf("Can't understand wav file %s\n", filename); + Com_Printf( S_COLOR_RED "ERROR: Incorrect/unsupported format in \"%s\"\n", + filename); return NULL; } @@ -219,7 +215,8 @@ void *S_WAV_CodecLoad(const char *filename, snd_info_t *info) if(!buffer) { FS_FCloseFile(file); - Com_Printf("Out of memory reading %s\n", filename); + Com_Printf( S_COLOR_RED "ERROR: Out of memory reading \"%s\"\n", + filename); return NULL; } @@ -247,7 +244,7 @@ snd_stream_t *S_WAV_CodecOpenStream(const char *filename) return NULL; // Read the RIFF header - if(!S_ReadWavHeader(rv->file, &rv->info)) + if(!S_ReadRIFFHeader(rv->file, &rv->info)) { S_CodecUtilClose(rv); return NULL; -- cgit v1.2.3