Merge pull request #11961 from NotTsunami/master

Remove redundant null checks

src/base/bittorrent/session.cpp

@@ -977,7 +977,6 @@ Session::~Session()
 
     // We must delete FilterParserThread
     // before we delete lt::session
-    if (m_filterParser)
     delete m_filterParser;
 
     // We must delete PortForwarderImpl before
@@ -1002,11 +1001,9 @@ void Session::initInstance()
 
 void Session::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 Session *Session::instance()
 {
@@ -1605,7 +1602,6 @@ void Session::enableTracker(const bool enable)
         m_tracker->start();
     }
     else {
-        if (m_tracker)
         delete m_tracker;
     }
 }

src/base/iconprovider.cpp

@@ -46,11 +46,9 @@ void IconProvider::initInstance()
 
 void IconProvider::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 IconProvider *IconProvider::instance()
 {

src/base/logger.cpp

@@ -65,11 +65,9 @@ void Logger::initInstance()
 
 void Logger::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 void Logger::addMessage(const QString &message, const Log::MsgType &type)
 {

src/base/net/downloadmanager.cpp

@@ -152,11 +152,9 @@ void Net::DownloadManager::initInstance()
 
 void Net::DownloadManager::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 Net::DownloadManager *Net::DownloadManager::instance()
 {

src/base/net/geoipmanager.cpp

@@ -63,7 +63,6 @@ GeoIPManager::GeoIPManager()
 
 GeoIPManager::~GeoIPManager()
 {
-    if (m_geoIPDatabase)
     delete m_geoIPDatabase;
 }
 
@@ -75,11 +74,9 @@ void GeoIPManager::initInstance()
 
 void GeoIPManager::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 GeoIPManager *GeoIPManager::instance()
 {
@@ -88,10 +85,8 @@ GeoIPManager *GeoIPManager::instance()
 
 void GeoIPManager::loadDatabase()
 {
-    if (m_geoIPDatabase) {
     delete m_geoIPDatabase;
     m_geoIPDatabase = nullptr;
-    }
 
     const QString filepath = Utils::Fs::expandPathAbs(
         QString("%1%2/%3").arg(specialFolderLocation(SpecialFolder::Data), GEOIP_FOLDER, GEOIP_FILENAME));
@@ -398,7 +393,7 @@ void GeoIPManager::configure()
         if (m_enabled && !m_geoIPDatabase) {
             loadDatabase();
         }
-        else if (!m_enabled && m_geoIPDatabase) {
+        else if (!m_enabled) {
             delete m_geoIPDatabase;
             m_geoIPDatabase = nullptr;
         }
@@ -423,7 +418,6 @@ void GeoIPManager::downloadFinished(const DownloadResult &result)
     GeoIPDatabase *geoIPDatabase = GeoIPDatabase::load(data, error);
     if (geoIPDatabase) {
         if (!m_geoIPDatabase || (geoIPDatabase->buildEpoch() > m_geoIPDatabase->buildEpoch())) {
-            if (m_geoIPDatabase)
             delete m_geoIPDatabase;
             m_geoIPDatabase = geoIPDatabase;
             LogMsg(tr("GeoIP database loaded. Type: %1. Build time: %2.")

src/base/net/private/downloadhandlerimpl.cpp

@@ -63,7 +63,6 @@ DownloadHandlerImpl::DownloadHandlerImpl(const Net::DownloadRequest &downloadReq
 
 DownloadHandlerImpl::~DownloadHandlerImpl()
 {
-    if (m_reply)
     delete m_reply;
 }
 

src/base/net/proxyconfigurationmanager.cpp

@@ -84,11 +84,9 @@ void ProxyConfigurationManager::initInstance()
 
 void ProxyConfigurationManager::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 ProxyConfigurationManager *ProxyConfigurationManager::instance()
 {

src/base/preferences.cpp

@@ -77,11 +77,9 @@ void Preferences::initInstance()
 
 void Preferences::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 const QVariant Preferences::value(const QString &key, const QVariant &defaultValue) const
 {

src/base/scanfoldersmodel.cpp

@@ -69,11 +69,9 @@ bool ScanFoldersModel::initInstance(QObject *parent)
 
 void ScanFoldersModel::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 ScanFoldersModel *ScanFoldersModel::instance()
 {

src/base/search/searchpluginmanager.cpp

@@ -104,7 +104,6 @@ SearchPluginManager *SearchPluginManager::instance()
 
 void SearchPluginManager::freeInstance()
 {
-    if (m_instance)
     delete m_instance;
 }
 

src/gui/mainwindow.cpp

@@ -678,7 +678,7 @@ void MainWindow::displayRSSTab(bool enable)
 #endif
         }
     }
-    else if (m_rssWidget) {
+    else {
         delete m_rssWidget;
     }
 }
@@ -714,7 +714,7 @@ void MainWindow::displaySearchTab(bool enable)
                 tr("Search"));
         }
     }
-    else if (m_searchWidget) {
+    else {
         delete m_searchWidget;
     }
 }
@@ -1170,7 +1170,6 @@ void MainWindow::closeEvent(QCloseEvent *e)
     }
 
     // abort search if any
-    if (m_searchWidget)
     delete m_searchWidget;
 
     hide();
@@ -1875,7 +1874,7 @@ void MainWindow::on_actionExecutionLogs_triggered(bool checked)
         m_tabs->setTabIcon(indexTab, UIThemeManager::instance()->getIcon("view-calendar-journal"));
 #endif
     }
-    else if (m_executionLog) {
+    else {
         delete m_executionLog;
     }
 

src/gui/private/fspathedit_p.cpp

@@ -259,11 +259,9 @@ void Private::FileLineEdit::keyPressEvent(QKeyEvent *e)
         FileSystemPathValidator::TestResult lastTestResult = validator->lastTestResult();
         QValidator::State lastState = validator->lastValidationState();
         if (lastTestResult == FileSystemPathValidator::TestResult::OK) {
-            if (m_warningAction) {
             delete m_warningAction;
             m_warningAction = nullptr;
         }
-        }
         else {
             if (!m_warningAction) {
                 m_warningAction = new QAction(this);

src/gui/uithememanager.cpp

@@ -43,11 +43,9 @@ UIThemeManager *UIThemeManager::m_instance = nullptr;
 
 void UIThemeManager::freeInstance()
 {
-    if (m_instance) {
     delete m_instance;
     m_instance = nullptr;
 }
-}
 
 void UIThemeManager::initInstance()
 {

src/webui/webui.cpp

@@ -127,20 +127,14 @@ void WebUI::configure()
                 m_dnsUpdater->updateCredentials();
         }
         else {
-            if (m_dnsUpdater)
             delete m_dnsUpdater;
         }
     }
     else {
         Net::PortForwarder::instance()->deletePort(oldPort);
 
-        if (m_httpServer)
         delete m_httpServer;
-
-        if (m_webapp)
         delete m_webapp;
-
-        if (m_dnsUpdater)
         delete m_dnsUpdater;
     }
 }