diff options
| author |  Lubomir I. Ivanov <neolit123@gmail.com> | 2015-10-07 12:59:13 +0300 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2015-10-07 13:46:23 +0100 |
| commit | 3b809f2cd45746bebdc844adec11d157ae4deea1 (patch) | |
| tree | 728a2d653e21c30c24d3b82fd3f2a50372337a7f | |
| parent | 43023fa51913b5f15a7e33275e3787c46ff74cc1 (diff) | |
| download | subsurface-3b809f2cd45746bebdc844adec11d157ae4deea1.tar.gz | |
dive.h: don't declare force_fixup_dive() in the header
The compiler complains that said static (not-inline) function
in declared in the header but has no definition.
The function is only used as a helper in dive.c so so this
simply acts as a forward declaration and we can safely remove
it from dive.h. The same forward declaration can be in dive.c.
Since dive.h is include in *many* translation units, this
causes a lot instances of the same warning to be show.
If the function becomes part of the API, the static keyword
needs to be removed and the declaration needs to be added in
the header again.
Signed-off-by: Lubomir I. Ivanov <neolit123@gmail.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
| -rw-r--r-- | dive.h | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -721,7 +721,6 @@ extern void fixup_dc_duration(struct divecomputer *dc); extern int dive_getUniqID(struct dive *d); extern unsigned int dc_airtemp(struct divecomputer *dc); extern unsigned int dc_watertemp(struct divecomputer *dc); -static void force_fixup_dive(struct dive *d); extern int split_dive(struct dive *); extern struct dive *merge_dives(struct dive *a, struct dive *b, int offset, bool prefer_downloaded); extern struct dive *try_to_merge(struct dive *a, struct dive *b, bool prefer_downloaded); |