From 0f55757c602c7fec723f73e5820ec936d24cd826 Mon Sep 17 00:00:00 2001
From: Björn Steinbrink <B.Steinbrink@gmx.de>
Date: Sun, 19 Mar 2006 21:40:21 +0000
Subject: Removed some useless null pointer checks.

---
 src/game.cpp                      | 7 ++-----
 src/gui/browserbox.cpp            | 6 ++----
 src/gui/gui.cpp                   | 5 +----
 src/gui/register.cpp              | 6 +-----
 src/lockedarray.h                 | 3 +--
 src/resources/resourcemanager.cpp | 6 ++----
 6 files changed, 9 insertions(+), 24 deletions(-)

(limited to 'src')

diff --git a/src/game.cpp b/src/game.cpp
index a780c2d7..f8812b81 100644
--- a/src/game.cpp
+++ b/src/game.cpp
@@ -308,14 +308,11 @@ Game::~Game()
 
     delete beingManager;
     delete floorItemManager;
+    delete joystick;
 
     beingManager = NULL;
     floorItemManager = NULL;
-
-    if (joystick != NULL)
-    {
-        delete joystick;
-    }
+    joystick = NULL;
 }
 
 bool saveScreenshot(SDL_Surface *screenshot)
diff --git a/src/gui/browserbox.cpp b/src/gui/browserbox.cpp
index 1f67670e..06660cd5 100644
--- a/src/gui/browserbox.cpp
+++ b/src/gui/browserbox.cpp
@@ -72,10 +72,8 @@ BrowserBox::~BrowserBox()
     if (instances == 0)
     {
         // Clean up static resource font
-        if (browserFont) {
-            delete browserFont;
-            browserFont = NULL;
-        }
+        delete browserFont;
+        browserFont = NULL;
     }
 #endif
 }
diff --git a/src/gui/gui.cpp b/src/gui/gui.cpp
index 7a57cd00..cb79e54b 100644
--- a/src/gui/gui.cpp
+++ b/src/gui/gui.cpp
@@ -193,10 +193,7 @@ Gui::~Gui()
     delete speechFont;
     delete mTop;
     delete mImageLoader;
-
-    if (mHostImageLoader) {
-        delete mHostImageLoader;
-    }
+    delete mHostImageLoader;
 
     delete guiInput;
 }
diff --git a/src/gui/register.cpp b/src/gui/register.cpp
index 39c918f7..c9b9daa7 100644
--- a/src/gui/register.cpp
+++ b/src/gui/register.cpp
@@ -198,11 +198,7 @@ RegisterDialog::action(const std::string& eventId)
                 mConfirmField->setText("");
             }
 
-            if (mWrongRegisterNotice)
-            {
-                delete mWrongRegisterNotice;
-                mWrongRegisterNotice = NULL;
-            }
+            delete mWrongRegisterNotice;
             mWrongRegisterNotice = new OkDialog("Error", errorMsg.str());
             mWrongRegisterNotice->addActionListener(mWrongDataNoticeListener);
         }
diff --git a/src/lockedarray.h b/src/lockedarray.h
index 13cbbda7..7ec2f9da 100644
--- a/src/lockedarray.h
+++ b/src/lockedarray.h
@@ -74,8 +74,7 @@ LockedArray<T>::LockedArray(unsigned int size):
 template<class T>
 LockedArray<T>::~LockedArray()
 {
-    if (mData)
-        delete [] mData;
+    delete [] mData;
 }
 
 template<class T>
diff --git a/src/resources/resourcemanager.cpp b/src/resources/resourcemanager.cpp
index 09102d2e..1cdf5a65 100644
--- a/src/resources/resourcemanager.cpp
+++ b/src/resources/resourcemanager.cpp
@@ -240,10 +240,8 @@ ResourceManager::getInstance()
 void
 ResourceManager::deleteInstance()
 {
-    if (instance != NULL) {
-        delete instance;
-        instance = NULL;
-    }
+    delete instance;
+    instance = NULL;
 }
 
 void*
-- 
cgit v1.2.3-70-g09d2