diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-09-21 13:41:45 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2019-09-21 16:07:24 -0700 |
| commit | 9c48130d3717ca7e569bcf36fe753c90cc9dc3ea (patch) | |
| tree | f5f6066c1d47ae65dba1743ad435e2b24e6fdb07 /core/save-xml.c | |
| parent | e46e018e215539025ed6fa975e100486cc419506 (diff) | |
| download | subsurface-9c48130d3717ca7e569bcf36fe753c90cc9dc3ea.tar.gz | |
Cleanup: replace is_dive_site_used() by is_dive_site_selected()
is_dive_site_used() had a "selected" parameter. If true it would
return whether the given dive site had a selected dive. Turns
out all callers had this parameter set to true. Therefore, replace
by a simplified function without the "selected" parameter and
give the function an appropriate name.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/save-xml.c')
| -rw-r--r-- | core/save-xml.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/save-xml.c b/core/save-xml.c index 912350da2..c2e9828d1 100644 --- a/core/save-xml.c +++ b/core/save-xml.c @@ -608,7 +608,7 @@ static void save_dives_buffer(struct membuffer *b, const bool select_only, bool if (dive_site_is_empty(ds)) continue; /* Only write used dive sites when exporting selected dives */ - if (select_only && !is_dive_site_used(ds, true)) + if (select_only && !is_dive_site_selected(ds)) continue; put_format(b, "<site uuid='%8x'", ds->uuid); @@ -823,7 +823,7 @@ static void save_dive_sites_buffer(struct membuffer *b, const bool select_only, if (dive_site_is_empty(ds)) continue; /* Only write used dive sites when exporting selected dives */ - if (select_only && !is_dive_site_used(ds, true)) + if (select_only && !is_dive_site_selected(ds)) continue; put_format(b, "<site uuid='%8x'", ds->uuid); |