diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-07-17 15:58:26 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-07-19 21:44:13 -0700 |
| commit | 94521c9958f5416311318826450825fcbaede4ce (patch) | |
| tree | 725dbf97b3feca84b8b3a67072668bacfbea9692 /core/dive.c | |
| parent | 726d08c2f71920acd8e5cc9b2d6850d1249466aa (diff) | |
| download | subsurface-94521c9958f5416311318826450825fcbaede4ce.tar.gz | |
Cleanup: remove bogus mark_divelist_changed() calls
The parsers / downloaders parse into a separate table and do
not directly change the divelist. Therefore, they shouldn't
call mark_divelist_changed().
Likewise split_dive_at() doesn't modify the dive list and
therefore shouldn't call this function.
Calling the function has the unwanted side-effect that undoing
the change will not clear the *-symbol in the title of the
main window.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/dive.c')
| -rw-r--r-- | core/dive.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/core/dive.c b/core/dive.c index 04247a682..fb1159564 100644 --- a/core/dive.c +++ b/core/dive.c @@ -3195,8 +3195,6 @@ static int split_dive_at(const struct dive *dive, int a, int b, struct dive **ou d2->number = 0; } - mark_divelist_changed(true); - *out1 = d1; *out2 = d2; return nr; |