libretro/scummvm
1
0
Fork 0
mirror of https://github.com/libretro/scummvm.git synced 2024-12-14 13:50:13 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

SWORD2: Fix some code analysis warnings (bug #3087857)

Browse Source 
svn-id: r53467
This commit is contained in:
Torbjörn Andersson 2010-10-15 06:12:23 +00:00
parent 6049a370b3
commit e5ec399c55
5 changed files with 19 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
engines/sword2/controls.cpp
View File

@ -180,12 +180,12 @@ void FontRendererGui::fetchText(uint32 textId, byte *buf) {
byte *data = _vm->fetchTextLine(_vm->_resman->openResource(textId / SIZE), textId & 0xffff); byte *data = _vm->fetchTextLine(_vm->_resman->openResource(textId / SIZE), textId & 0xffff);
int i; int i;
for (i = 0; data[i + 2]; i++) { if (buf) {
if (buf) for (i = 0; data[i + 2]; i++)
buf[i] = data[i + 2]; buf[i] = data[i + 2];
buf[i] = 0;
} }
buf[i] = 0;
_vm->_resman->closeResource(textId / SIZE); _vm->_resman->closeResource(textId / SIZE);
} }

13
engines/sword2/music.cpp
View File

@ -496,9 +496,16 @@ int Sound::readBuffer(int16 *buffer, const int numSamples) {
memset(buffer, 0, 2 * numSamples); memset(buffer, 0, 2 * numSamples);
if (!_mixBuffer || numSamples > _mixBufferLen) { if (!_mixBuffer || numSamples > _mixBufferLen) {
if (_mixBuffer) if (_mixBuffer) {
_mixBuffer = (int16 *)realloc(_mixBuffer, 2 * numSamples); int16 *newBuffer = (int16 *)realloc(_mixBuffer, 2 * numSamples);
else if (newBuffer) {
_mixBuffer = newBuffer;
} else {
// We can't use the old buffer any more. It's too small.
free(_mixBuffer);
_mixBuffer = 0;
}
} else
_mixBuffer = (int16 *)malloc(2 * numSamples); _mixBuffer = (int16 *)malloc(2 * numSamples);
_mixBufferLen = numSamples; _mixBufferLen = numSamples;

4
engines/sword2/protocol.cpp
View File

@ -412,8 +412,8 @@ byte *Sword2Engine::fetchPsxParallax(uint32 location, uint8 level) {
debug(2, "fetchPsxParallax() -> %s parallax, xRes: %u, yRes: %u", (level == 0) ? "Background" : "Foreground", plxXres, plxYres); debug(2, "fetchPsxParallax() -> %s parallax, xRes: %u, yRes: %u", (level == 0) ? "Background" : "Foreground", plxXres, plxYres);
// Calculate the number of tiles which compose the parallax grid. // Calculate the number of tiles which compose the parallax grid.
horTiles = plxXres % 64 ? (plxXres / 64) + 1 : plxXres / 64; horTiles = (plxXres % 64) ? (plxXres / 64) + 1 : plxXres / 64;
verTiles = plxYres % 16 ? (plxYres / 16) + 1 : plxYres / 16; verTiles = (plxYres % 16) ? (plxYres / 16) + 1 : plxYres / 16;
totSize = plxSize + horTiles * verTiles * 4 + 8; totSize = plxSize + horTiles * verTiles * 4 + 8;

4
engines/sword2/render.cpp
View File

@ -731,8 +731,8 @@ int32 Screen::initialisePsxParallaxLayer(byte *parallax) {
data = parallax + xTiles * yTiles * 4; data = parallax + xTiles * yTiles * 4;
_xBlocks[_layer] = xTiles; _xBlocks[_layer] = xTiles;
_yBlocks[_layer] = (yTiles / 2) + (yTiles % 2 ? 1 : 0); _yBlocks[_layer] = (yTiles / 2) + ((yTiles % 2) ? 1 : 0);
bool oddTiles = (yTiles % 2 ? true : false); bool oddTiles = ((yTiles % 2) ? true : false);
_blockSurfaces[_layer] = (BlockSurface **)calloc(_xBlocks[_layer] * _yBlocks[_layer], sizeof(BlockSurface *)); _blockSurfaces[_layer] = (BlockSurface **)calloc(_xBlocks[_layer] * _yBlocks[_layer], sizeof(BlockSurface *));
if (!_blockSurfaces[_layer]) if (!_blockSurfaces[_layer])

4
engines/sword2/resman.cpp
View File

@ -474,7 +474,7 @@ void ResourceManager::readCluIndex(uint16 fileNum, Common::File *file) {
file->seek(table_offset); file->seek(table_offset);
assert((tableSize % 8) == 0); assert((tableSize % 8) == 0);
_resFiles[fileNum].entryTab = (uint32*)malloc(tableSize); _resFiles[fileNum].entryTab = (uint32 *)malloc(tableSize);
_resFiles[fileNum].numEntries = tableSize / 8; _resFiles[fileNum].numEntries = tableSize / 8;
file->read(_resFiles[fileNum].entryTab, tableSize); file->read(_resFiles[fileNum].entryTab, tableSize);
if (file->eos() || file->err()) if (file->eos() || file->err())
@ -482,7 +482,7 @@ void ResourceManager::readCluIndex(uint16 fileNum, Common::File *file) {
#ifdef SCUMM_BIG_ENDIAN #ifdef SCUMM_BIG_ENDIAN
for (int tabCnt = 0; tabCnt < _resFiles[fileNum].numEntries * 2; tabCnt++) for (int tabCnt = 0; tabCnt < _resFiles[fileNum].numEntries * 2; tabCnt++)
_resFiles[fileNum].entryTab[tabCnt] = FROM_LE_32(_resFiles[fileNum].entryTab[tabCnt]); _resFiles[fileNum].entryTab[tabCnt] = FROM_LE_32(_resFiles[fileNum].entryTab[tabCnt]);
#endif #endif
} }