mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-30 22:20:21 +00:00
Code Issues Projects Releases Packages Wiki Activity

Use common helper function for the "no cylinder info" case

Browse source 
Miika fixed the statistics code that didn't properly check for the "no
cylinder info" case - this cleans it up and just uses the helper
function in equipment.c.

Rename the helper to be slightly better named while at it.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Linus Torvalds 2011-12-09 09:52:59 -08:00
parent fe532a585c
commit 9b1ea2ae6d
3 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
dive.h
View file

@ -87,6 +87,8 @@ typedef struct {
pressure_t start, end, sample_start, sample_end;
} cylinder_t;
extern int cylinder_none(cylinder_t *cyl);
extern int get_pressure_units(unsigned int mb, const char **units);
extern double get_depth_units(unsigned int mm, int *frac, const char **units);
extern double get_volume_units(unsigned int mm, int *frac, const char **units);

4
equipment.c
View file

@ -286,7 +286,7 @@ static void show_cylinder(cylinder_t *cyl, struct cylinder_widget *cylinder)
gtk_spin_button_set_value(GTK_SPIN_BUTTON(cylinder->o2), o2);
}
static int cyl_nothing(cylinder_t *cyl)
int cylinder_none(cylinder_t *cyl)
{
return !cyl->type.size.mliter &&
!cyl->type.workingpressure.mbar &&
@ -328,7 +328,7 @@ void show_dive_equipment(struct dive *dive)
do {
cylinder_t *cyl = &dive->cylinder[max-1];
if (!cyl_nothing(cyl))
if (!cylinder_none(cyl))
break;
} while (--max);

5
statistics.c
View file

@ -169,10 +169,7 @@ void show_dive_stats(struct dive *dive)
start = cyl->start.mbar ? : cyl->sample_start.mbar;
end = cyl->end.mbar ? : cyl->sample_end.mbar;
/* we assume that every valid cylinder has either a working pressure
* or a size; but for good measure let's also accept cylinders with
* a starting or ending pressure*/
if (cyl->type.workingpressure.mbar || cyl->type.size.mliter || cyl->gasmix.o2.permille || start || end) {
if (!cylinder_none(cyl)) {
/* 0% O2 strangely means air, so 21% - I don't like that at all */
int o2 = cyl->gasmix.o2.permille ? : 209;
if (offset > 0) {