aboutsummaryrefslogtreecommitdiff
path: root/engines/m4/scene.cpp
diff options
context:
space:
mode:
authorTorbjörn Andersson2010-01-03 19:37:43 +0000
committerTorbjörn Andersson2010-01-03 19:37:43 +0000
commit72eb9ec9eab5efcb3aa99a962a80423e8c0a3232 (patch)
tree032c9107838aefca6a166465a46fa7318bb82566 /engines/m4/scene.cpp
parent910ffb53a0b6c74a965df9a1270cdfc3885252ec (diff)
downloadscummvm-rg350-72eb9ec9eab5efcb3aa99a962a80423e8c0a3232.tar.gz
scummvm-rg350-72eb9ec9eab5efcb3aa99a962a80423e8c0a3232.tar.bz2
scummvm-rg350-72eb9ec9eab5efcb3aa99a962a80423e8c0a3232.zip
Fixed a bunch of cppcheck warnings. Mostly about checking if a pointer is null
before freeing it, which isn't necessary. svn-id: r46941
Diffstat (limited to 'engines/m4/scene.cpp')
-rw-r--r--engines/m4/scene.cpp19
1 files changed, 5 insertions, 14 deletions
diff --git a/engines/m4/scene.cpp b/engines/m4/scene.cpp
index 905cb744c3..f61a2e88ce 100644
--- a/engines/m4/scene.cpp
+++ b/engines/m4/scene.cpp
@@ -73,21 +73,13 @@ Scene::~Scene() {
delete _backgroundSurface;
delete _codeSurface;
-
- if (_sceneSprites)
- delete _sceneSprites;
+ delete _sceneSprites;
// _vm->_palette->deleteAllRanges();
- if (_palData)
- delete _palData;
-
- if (_interfacePal)
- delete _interfacePal;
-
- if (_inverseColorTable)
- delete[] _inverseColorTable;
-
+ delete _palData;
+ delete _interfacePal;
+ delete[] _inverseColorTable;
}
void Scene::loadScene(int sceneNumber) {
@@ -284,8 +276,7 @@ void Scene::loadSceneInverseColorTable(int sceneNumber) {
if (_vm->isM4()) {
sprintf(filename, "%i.ipl", sceneNumber);
iplS = _vm->res()->openFile(filename);
- if (_inverseColorTable)
- delete[] _inverseColorTable;
+ delete[] _inverseColorTable;
_inverseColorTable = new byte[iplS->size()];
iplS->read(_inverseColorTable, iplS->size());
_vm->res()->toss(filename);