diff options
author | Tomaz Canabrava <tomaz.canabrava@intel.com> | 2014-05-21 12:06:02 -0300 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2014-05-22 07:28:27 +0900 |
commit | ef4705070ff67322386041529b7f10eaf09f6cdf (patch) | |
tree | faabcc792bd3d5e9bd3338b77ce6df4fc217ca77 /qt-ui | |
parent | 3750d55665e6cb940e22b5556a65078940169c5f (diff) | |
download | subsurface-ef4705070ff67322386041529b7f10eaf09f6cdf.tar.gz |
Create a 'remove_event' function that removes an event.
The logic of removing the event was in the UI, and this makes
the code harder to test because we need to take into account
also the events that the interface is receiving, instead of
only relying on the algorithm to test.
so, now it lives in dive.h/.c and a unittest is easyer to make.
Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'qt-ui')
-rw-r--r-- | qt-ui/profile/profilewidget2.cpp | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/qt-ui/profile/profilewidget2.cpp b/qt-ui/profile/profilewidget2.cpp index d95a32be8..4a599d711 100644 --- a/qt-ui/profile/profilewidget2.cpp +++ b/qt-ui/profile/profilewidget2.cpp @@ -830,13 +830,7 @@ void ProfileWidget2::removeEvent() tr("Remove the selected event?"), tr("%1 @ %2:%3").arg(event->name).arg(event->time.seconds / 60).arg(event->time.seconds % 60, 2, 10, QChar('0'))), QMessageBox::Ok | QMessageBox::Cancel) == QMessageBox::Ok) { - struct event **ep = ¤t_dc->events; - while (ep && *ep != event) - ep = &(*ep)->next; - if (ep) { - *ep = event->next; - free(event); - } + remove_event(event); mark_divelist_changed(true); replot(); } |