diff options
| author |  Dirk Hohndel <dirk@hohndel.org> | 2013-02-09 06:17:25 -0800 |
|---|---|---|
| committer | Dirk Hohndel <dirk@hohndel.org> | 2013-02-09 06:17:25 -0800 |
| commit | 0755cc522b9d68580f79e567808770df5814f150 (patch) | |
| tree | 5cc688b9c24c74c74d36d295d0afedccc155badd /statistics.c | |
| parent | 9e66312d6a4e693dc094c3e17881e495f65f7d89 (diff) | |
| download | subsurface-0755cc522b9d68580f79e567808770df5814f150.tar.gz | |
Show the actice divecomputer's data in the Info notebook
While the profile switched between different divecomputers, the Dive Info
notebook always showed either information from the first divecomputer or
(with some of the recent changes) information that had been collected from
all divecomputers and somehow consolidated for the dive.
With this commit we now show the data from the same divecomputer that is
also shown in the profile (which means if some data is available from one
of the divecomputers and not from another that will be correctly reflected
in the Dive Info notebook as the user cycles through the divecomputers.
This does beg the question if we should have some kind of "best data
available, considering all divecomputers" mode - but that's definitely not
something I'll tackle prior to 3.0.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'statistics.c')
| -rw-r--r-- | statistics.c | 25 |
1 files changed, 17 insertions, 8 deletions
diff --git a/statistics.c b/statistics.c index 5c6777f8a..33789a93d 100644 --- a/statistics.c +++ b/statistics.c @@ -525,6 +525,9 @@ static char *get_time_string(int seconds, int maxdays) return buf; } +/* we try to show the data from the currently selected divecomputer + * right now for some values (e.g., surface pressure) we could fall back + * to dive data, but for consistency we don't. */ static void show_single_dive_stats(struct dive *dive) { char buf[80]; @@ -534,10 +537,12 @@ static void show_single_dive_stats(struct dive *dive) int idx, offset, gas_used, mbar; struct dive *prev_dive; struct tm tm; + struct divecomputer *dc; process_all_dives(dive, &prev_dive); if (!dive) return; + dc = select_dc(&dive->dc); utc_mkdate(dive->when, &tm); snprintf(buf, sizeof(buf), /*++GETTEXT 80 chars: weekday, monthname, day, year, hour, min */ @@ -554,28 +559,32 @@ static void show_single_dive_stats(struct dive *dive) get_time_string(dive->when - (prev_dive->when + get_duration_in_sec(prev_dive)), 4)); else set_label(single_w.surf_intv, _("unknown")); - value = get_depth_units(dive->maxdepth.mm, &decimals, &unit); + value = get_depth_units(dc->maxdepth.mm, &decimals, &unit); set_label(single_w.max_depth, "%.*f %s", decimals, value, unit); - value = get_depth_units(dive->dc.meandepth.mm, &decimals, &unit); + value = get_depth_units(dc->meandepth.mm, &decimals, &unit); set_label(single_w.avg_depth, "%.*f %s", decimals, value, unit); set_label(single_w.viz, star_strings[dive->visibility]); - if (dive->dc.watertemp.mkelvin) { - value = get_temp_units(dive->dc.watertemp.mkelvin, &unit); + if (dc->watertemp.mkelvin) { + value = get_temp_units(dc->watertemp.mkelvin, &unit); set_label(single_w.water_temp, "%.1f %s", value, unit); } else { set_label(single_w.water_temp, ""); } - if (dive->dc.airtemp.mkelvin) { - value = get_temp_units(dive->dc.airtemp.mkelvin, &unit); + if (dc->airtemp.mkelvin) { + value = get_temp_units(dc->airtemp.mkelvin, &unit); set_label(single_w.air_temp, "%.1f %s", value, unit); } else { set_label(single_w.air_temp, ""); } - mbar = get_surface_pressure_in_mbar(dive, FALSE); + mbar = dc->surface_pressure.mbar; + /* it would be easy to get dive data here: + * if (!mbar) + * mbar = get_surface_pressure_in_mbar(dive, FALSE); + */ if (mbar) { set_label(single_w.air_press, "%d mbar", mbar); } else { - set_label(single_w.air_press, _("unknown")); + set_label(single_w.air_press, ""); } value = get_volume_units(dive->sac, &decimals, &unit); if (value > 0) |