Removing unneeded null pointer checks (#3649)

mumble-voip · Apr 1, 2019 · 61be8b0 · 61be8b0
1 parent 193c01d
commit 61be8b0
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 12 deletions.
diff --git a/src/mumble/JackAudio.cpp b/src/mumble/JackAudio.cpp
@@ -86,10 +86,8 @@ class JackAudioInit : public DeferInit {
 		}
 
 		void destroy() {
-			if (airJackAudio)
-				delete airJackAudio;
-			if (aorJackAudio)
-				delete aorJackAudio;
+			delete airJackAudio;
+			delete aorJackAudio;
 			if (jasys) {
 				jasys->close_jack();
 				delete jasys;

diff --git a/src/murmur/MurmurGRPCImpl.cpp b/src/murmur/MurmurGRPCImpl.cpp
@@ -127,9 +127,7 @@ void GRPCStart() {
 }
 
 void GRPCStop() {
-	if (service) {
-		delete service;
-	}
+	delete service;
 }
 
 MurmurRPCImpl::MurmurRPCImpl(const QString &address, std::shared_ptr<::grpc::ServerCredentials> credentials) {

diff --git a/src/murmur/Server.cpp b/src/murmur/Server.cpp
@@ -621,11 +621,9 @@ void Server::initBonjour() {
 }
 
 void Server::removeBonjour() {
-	if (bsRegistration) {
-		delete bsRegistration;
-		bsRegistration = NULL;
-		log("Stopped announcing server via bonjour");
-	}
+	delete bsRegistration;
+	bsRegistration = NULL;
+	log("Stopped announcing server via bonjour");
 }
 #endif