diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2015-06-21 22:07:10 -0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2015-06-21 22:11:49 -0700 |
commit | 895d6e0000b824809a19d42aee0955df76af8c86 (patch) | |
tree | 930c9f93eeacd619decb5f2d360ff615808ceafc /qt-ui | |
parent | 2c0223fbb206b0552f137d0fb32d8bda848102b7 (diff) | |
download | subsurface-895d6e0000b824809a19d42aee0955df76af8c86.tar.gz |
Comparing the address of a member to 0 is pointless
Or... this code doesn't do what you think it does.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-ui')
-rw-r--r-- | qt-ui/profile/profilewidget2.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qt-ui/profile/profilewidget2.cpp b/qt-ui/profile/profilewidget2.cpp index cd6a7a157..be09fc5ea 100644 --- a/qt-ui/profile/profilewidget2.cpp +++ b/qt-ui/profile/profilewidget2.cpp @@ -1329,7 +1329,7 @@ void ProfileWidget2::hideEvents() if (QMessageBox::question(MainWindow::instance(), TITLE_OR_TEXT(tr("Hide events"), tr("Hide all %1 events?").arg(event->name)), QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Ok) { - if (event->name) { + if (!same_string(event->name, "")) { for (int i = 0; i < evn_used; i++) { if (same_string(event->name, ev_namelist[i].ev_name)) { ev_namelist[i].plot_ev = false; |