Fix issue#10: removed unnecessary null pointer checks

diff --git a/src/dhp_gc.cpp b/src/dhp_gc.cpp
index 375fb8b794d7ee7e8c6c1b611c7931fc5450e34c..ea41aa14fac9604bc73d1395b863f75cbbcf4a30 100644 (file)
--- 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 af15fa9324cb65f934c13358696ef5ec930913c6..a50aa37087773a6a20bf0d37cfe8ce2a6d88ccb7 100644 (file)
--- 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];