From: khizmax <khizmax@gmail.com>
Date: Thu, 22 Jan 2015 11:32:53 +0000 (+0300)
Subject: Fix issue#10: removed unnecessary null pointer checks
X-Git-Tag: v2.1.0~305^2~100
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=ea48c7ab11c97b876fd016e72f71dbef2c718ed6;p=libcds.git

Fix issue#10: removed unnecessary null pointer checks
---

diff --git a/src/dhp_gc.cpp b/src/dhp_gc.cpp
index 375fb8b7..ea41aa14 100644
--- a/src/dhp_gc.cpp
+++ b/src/dhp_gc.cpp
@@ -147,10 +147,8 @@ namespace cds { namespace gc { namespace dhp {
 
     void CDS_STDCALL GarbageCollector::Destruct()
     {
-        if ( m_pManager ) {
-            delete m_pManager;
-            m_pManager = nullptr;
-        }
+        delete m_pManager;
+        m_pManager = nullptr;
     }
 
     GarbageCollector::GarbageCollector( size_t nLiberateThreshold, size_t nInitialThreadGuardCount )
diff --git a/tests/cppunit/thread.cpp b/tests/cppunit/thread.cpp
index af15fa93..a50aa370 100644
--- a/tests/cppunit/thread.cpp
+++ b/tests/cppunit/thread.cpp
@@ -42,10 +42,8 @@ namespace CppUnitMini {
 
     ThreadPool::~ThreadPool()
     {
-        if ( m_pBarrierStart )
-            delete m_pBarrierStart;
-        if ( m_pBarrierDone )
-            delete m_pBarrierDone;
+        delete m_pBarrierStart;
+        delete m_pBarrierDone;
 
         for ( size_t i = 0; i < m_arrThreads.size(); ++i )
             delete m_arrThreads[i];