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/uemis-downloader.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/uemis-downloader.c')
| -rw-r--r-- | core/uemis-downloader.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/core/uemis-downloader.c b/core/uemis-downloader.c index 5ad6b92be..08fa44542 100644 --- a/core/uemis-downloader.c +++ b/core/uemis-downloader.c @@ -1018,14 +1018,12 @@ static bool process_raw_buffer(device_data_t *devdata, uint32_t deviceid, char * if (done && ++bp < endptr && *bp != '{' && strstr(bp, "{{")) { done = false; record_dive_to_table(dive, devdata->download_table); - mark_divelist_changed(true); dive = uemis_start_dive(deviceid); } } if (is_log) { if (dive->dc.diveid) { record_dive_to_table(dive, devdata->download_table); - mark_divelist_changed(true); } else { /* partial dive */ free(dive); free(buf); |