summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2013-01-16 14:44:11 -0800
committerGravatar Dirk Hohndel <dirk@hohndel.org>2013-01-16 14:44:11 -0800
commit2cee400ad1fd90815b7908a45618e2291a2ae5ec (patch)
treec190fab63aef00c2337e6e6e6f280a63b7910ab3
parent71c823441396d357b6f3bb329fa6bd43dfbab7e7 (diff)
downloadsubsurface-2cee400ad1fd90815b7908a45618e2291a2ae5ec.tar.gz
Correctly mark divelist changed after accepting dive plan
Commit 293126257b20 ("Mark divelist as changed if user accepts planned dive") had the correct commit message but an incorrect implementation. This moves the mark_divelist_changed() call into the GTK_RESPONSE_ACCEPT clause where it belongs. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
-rw-r--r--planner.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/planner.c b/planner.c
index 16813caed..64e95bbef 100644
--- a/planner.c
+++ b/planner.c
@@ -618,7 +618,6 @@ void plan(struct diveplan *diveplan, char **cached_datap, struct dive **divep)
add_plan_to_notes(diveplan, dive);
/* now make the dive visible in the dive list */
report_dives(FALSE, FALSE);
- mark_divelist_changed(TRUE);
show_and_select_dive(dive);
free(stoplevels);
free(gaschanges);
@@ -1172,6 +1171,7 @@ void input_plan()
gtk_widget_show_all(planner);
if (gtk_dialog_run(GTK_DIALOG(planner)) == GTK_RESPONSE_ACCEPT) {
plan(&diveplan, &cache_data, &planned_dive);
+ mark_divelist_changed(TRUE);
} else {
if (planned_dive) {
/* we have added a dive during the dynamic construction