diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2015-09-28 21:41:32 -0400 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2015-09-28 21:41:32 -0400 |
| commit | 55cc0ee9c158607378f8277143eab731c6f91e6c (patch) | |
| tree | 795e6a77565c1a860571b68e933d5d241f456e15 | |
| parent | b992ca22df4d8db6607bccca91e74a1d9b6294e3 (diff) | |
| download | subsurface-55cc0ee9c158607378f8277143eab731c6f91e6c.tar.gz | |
Fix incorrect and confusing warning message
In verbose mode we wanted to tell the user if they had a reference to a
non-existant dive site that we removed for consistency. Except the
condition was wrong and the warning was printed for every VALID dive site
reference. Oops.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
| -rw-r--r-- | save-git.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/save-git.c b/save-git.c index 1228b7762..9aa7bfd0e 100644 --- a/save-git.c +++ b/save-git.c @@ -405,7 +405,7 @@ static void create_dive_buffer(struct dive *dive, struct membuffer *b) save_tags(b, dive->tag_list); cond_put_format(dive->dive_site_uuid && get_dive_site_by_uuid(dive->dive_site_uuid), b, "divesiteid %08x\n", dive->dive_site_uuid); - if (verbose && dive->dive_site_uuid && get_dive_site_by_uuid(dive->dive_site_uuid)) + if (verbose && dive->dive_site_uuid && !get_dive_site_by_uuid(dive->dive_site_uuid)) fprintf(stderr, "removed reference to non-existant dive site with uuid %08x\n", dive->dive_site_uuid); save_overview(b, dive); save_cylinder_info(b, dive); |