From 966192613164622da2ae8f0770aad8a6cde508ee Mon Sep 17 00:00:00 2001 From: Tomaz Canabrava Date: Fri, 11 Jul 2014 13:10:43 -0300 Subject: [PATCH] It's safe to delete NULL pointers Small code cleanup, it's safe to delete null pointers. Signed-off-by: Tomaz Canabrava Signed-off-by: Dirk Hohndel --- qt-ui/mainwindow.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/qt-ui/mainwindow.cpp b/qt-ui/mainwindow.cpp index 9f4c38f70..e589344e3 100644 --- a/qt-ui/mainwindow.cpp +++ b/qt-ui/mainwindow.cpp @@ -155,8 +155,7 @@ void MainWindow::refreshDisplay(bool doRecreateDiveList) WSInfoModel::instance()->updateInfo(); // refresh the yearly stats if the window has an instance if (yearlyStats) { - if (yearlyStatsModel) - delete yearlyStatsModel; + delete yearlyStatsModel; yearlyStatsModel = new YearlyStatisticsModel(); yearlyStats->setModel(yearlyStatsModel); } @@ -525,8 +524,7 @@ void MainWindow::on_actionYearlyStatistics_triggered() /* problem here is that without more MainWindow variables or a separate YearlyStatistics * class the user needs to close the window/widget and re-open it for it to update. */ - if (yearlyStatsModel) - delete yearlyStatsModel; + delete yearlyStatsModel; yearlyStatsModel = new YearlyStatisticsModel(); yearlyStats->setModel(yearlyStatsModel); yearlyStats->raise();