Merge branch 'elfring-Remove_unnecessary_null_pointer_checks'

csmith-project · Apr 21, 2019 · c7e6a90 · c7e6a90
2 parents 4f540a9 + 52e36d8
commit c7e6a90
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 23 deletions.
diff --git a/src/DepthSpec.h b/src/DepthSpec.h
@@ -91,14 +91,14 @@ enum dType {
 
 #define DEPTH_GUARD_BY_DEPTH_RETURN_DEL1(d, t1, rv) \
 	if (DepthSpec::depth_guard_by_depth(d) != GOOD_DEPTH) { \
-		if (t1) delete t1; \
+		delete t1; \
 		return rv; \
 	}
 
 #define DEPTH_GUARD_BY_DEPTH_RETURN_DEL2(d, t1, t2, rv) \
 	if (DepthSpec::depth_guard_by_depth(d) != GOOD_DEPTH) { \
-		if (t1) delete t1; \
-		if (t2) delete t2; \
+		delete t1; \
+		delete t2; \
 		return rv; \
 	}
 

diff --git a/src/Enumerator.h b/src/Enumerator.h
@@ -148,8 +148,7 @@ Enumerator<Name>::~Enumerator()
 {
 	typename map<Name, EnumObject*>::iterator i;
 	for (i = objs_.begin(); i != objs_.end(); ++i) {
-		if ((*i).second != NULL)
-			delete (*i).second;
+		delete i->second;
 	}
 	objs_.clear();
 }

diff --git a/src/Function.cpp b/src/Function.cpp
@@ -874,10 +874,7 @@ OutputFunctions(std::ostream &out)
 int
 Function::deleteFunction(Function* func)
 {
-	if (func) {
-		delete func;
-		func = 0;
-	}
+	delete func;
 	return 0;
 }
 

diff --git a/src/Probabilities.cpp b/src/Probabilities.cpp
@@ -191,9 +191,7 @@ GroupProbElem::~GroupProbElem()
 {
 	std::map<ProbName, SingleProbElem*>::iterator i;
 	for (i = probs_.begin(); i != probs_.end(); ++i) {
-		SingleProbElem *elem = (*i).second;
-		assert(elem);
-		delete elem;
+		delete i->second;
 	}
 	probs_.clear();
 }
@@ -380,10 +378,8 @@ Probabilities::GetInstance()
 void
 Probabilities::DestroyInstance()
 {
-	if (Probabilities::instance_) {
-		delete Probabilities::instance_;
-		Probabilities::instance_ = NULL;
-	}
+	delete instance_;
+	instance_ = NULL;
 }
 
 void

diff --git a/src/RandomNumber.cpp b/src/RandomNumber.cpp
@@ -174,9 +174,7 @@ RandomNumber::doFinalization()
 	for (unsigned int i = 0; i < count; ++i) {
 		RNDNUM_GENERATOR rImpl = static_cast<RNDNUM_GENERATOR>(i);
 		generator = instance_->generators_[rImpl];
-		if (generator != NULL) {
-			delete generator;
-		}
+		delete generator;
 	}
 	delete instance_;
 }

diff --git a/src/Variable.cpp b/src/Variable.cpp
@@ -473,10 +473,7 @@ Variable::~Variable(void)
 	for(i = field_vars.begin(); i != field_vars.end(); ++i)
 		delete (*i);
 	field_vars.clear();
-	if (init) {
-		delete init;
-		init = NULL;
-	}
+	delete init;
 }
 
 // --------------------------------------------------------------