diff options
| author |  Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2018-08-17 00:58:30 +0200 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2018-08-23 05:16:38 -0700 |
| commit | 605e1e19ed0c52a16580c95e36ae79e71b539351 (patch) | |
| tree | 50895b0e390a1b446d220cefaad531880c13259b /core/statistics.c | |
| parent | 44f34d8cd72568a313e9a39907b65ed44bdbdeba (diff) | |
| download | subsurface-605e1e19ed0c52a16580c95e36ae79e71b539351.tar.gz | |
Cleanup: const-ify functions taking pointers to events
This is another entry in the series to make more things
"const-clean" with the ultimate goal of merge_dive() take
const pointers.
This concerns functions taking pointers to events and
the fallout from making these const.
The somewhat debatable part of this commit might be
that get_next_event() is split in a two distinct
(const and non-const) versions with different names,
since C doesn't allow overloading. The linker should
recognize that these functions are identical and remove
one of them.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/statistics.c')
| -rw-r--r-- | core/statistics.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/statistics.c b/core/statistics.c index bb32095cc..b518a9c7f 100644 --- a/core/statistics.c +++ b/core/statistics.c @@ -236,10 +236,10 @@ void process_selected_dives(void) #define SOME_GAS 5000 // 5bar drop in cylinder pressure makes cylinder used -bool has_gaschange_event(struct dive *dive, struct divecomputer *dc, int idx) +bool has_gaschange_event(const struct dive *dive, const struct divecomputer *dc, int idx) { bool first_gas_explicit = false; - struct event *event = get_next_event(dc->events, "gaschange"); + const struct event *event = get_next_event(dc->events, "gaschange"); while (event) { if (dc->sample && (event->time.seconds == 0 || (dc->samples && dc->sample[0].time.seconds == event->time.seconds))) |