From 72eb9ec9eab5efcb3aa99a962a80423e8c0a3232 Mon Sep 17 00:00:00 2001
From: Torbjörn Andersson
Date: Sun, 3 Jan 2010 19:37:43 +0000
Subject: Fixed a bunch of cppcheck warnings. Mostly about checking if a
 pointer is null before freeing it, which isn't necessary.

svn-id: r46941
---
 engines/cruise/detection.cpp | 2 +-
 engines/cruise/saveload.cpp  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'engines/cruise')

diff --git a/engines/cruise/detection.cpp b/engines/cruise/detection.cpp
index d6095092ac..5fc96ffff2 100644
--- a/engines/cruise/detection.cpp
+++ b/engines/cruise/detection.cpp
@@ -262,7 +262,7 @@ SaveStateList CruiseMetaEngine::listSaves(const char *target) const {
 				Cruise::CruiseSavegameHeader header;
 				Cruise::readSavegameHeader(in, header);
 				saveList.push_back(SaveStateDescriptor(slotNum, header.saveName));
-				if (header.thumbnail) delete header.thumbnail;
+				delete header.thumbnail;
 				delete in;
 			}
 		}
diff --git a/engines/cruise/saveload.cpp b/engines/cruise/saveload.cpp
index a7351ff700..92cccc7026 100644
--- a/engines/cruise/saveload.cpp
+++ b/engines/cruise/saveload.cpp
@@ -828,7 +828,7 @@ Common::Error loadSavegameData(int saveGameIdx) {
 	// Skip over the savegame header
 	CruiseSavegameHeader header;
 	readSavegameHeader(f, header);
-	if (header.thumbnail) delete header.thumbnail;
+	delete header.thumbnail;
 
 	// Synchronise the remaining data of the savegame
 	Common::Serializer s(f, NULL);
-- 
cgit v1.2.3