Core/Misc: Multiple static analysis issues fixed (small optimizations and clear code)

2 files changed, 2 insertions, 5 deletions

diff --git a/src/server/worldserver/Main.cpp b/src/server/worldserver/Main.cpp
index 191e44b3e93..07101e61eb6 100644
--- a/src/server/worldserver/Main.cpp
+++ b/src/server/worldserver/Main.cpp
@@ -268,8 +268,7 @@ extern int main(int argc, char** argv)
         delete soapThread;
     }
 
-    if (raAcceptor != nullptr)
-        delete raAcceptor;
+    delete raAcceptor;
 
     ///- Clean database before leaving
     ClearOnlineAccounts();
diff --git a/src/server/worldserver/RemoteAccess/RASession.cpp b/src/server/worldserver/RemoteAccess/RASession.cpp
index 9892019ae5a..35149fff0b5 100644
--- a/src/server/worldserver/RemoteAccess/RASession.cpp
+++ b/src/server/worldserver/RemoteAccess/RASession.cpp
@@ -192,9 +192,7 @@ bool RASession::ProcessCommand(std::string& command)
     }
 
     // Obtain a new promise per command
-    if (_commandExecuting != nullptr)
-        delete _commandExecuting;
-
+    delete _commandExecuting;
     _commandExecuting = new std::promise<void>();
 
     CliCommandHolder* cmd = new CliCommandHolder(this, command.c_str(), &RASession::CommandPrint, &RASession::CommandFinished);