diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2011-11-13 15:29:07 -0200 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2011-11-13 15:29:07 -0200 |
commit | 97a1dd68306de17aa73be092d2bcf174e8158f62 (patch) | |
tree | 9639e9cf950a7c1d5ffbc51107b734675d254a77 /divelist.c | |
parent | 5c2fca262043992783551eceebc81c1f589b185b (diff) | |
download | subsurface-97a1dd68306de17aa73be092d2bcf174e8158f62.tar.gz |
Make sure SAC and OTU get recalculated after cylinder info was changed
Also fixed minor coding style issues (prevent useless code from being
executed).
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'divelist.c')
-rw-r--r-- | divelist.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/divelist.c b/divelist.c index eb8f23143..bf853ec0e 100644 --- a/divelist.c +++ b/divelist.c @@ -335,6 +335,14 @@ static int calculate_sac(struct dive *dive) return sac * 1000; } +void update_cylinder_related_info(struct dive *dive) +{ + if(dive != NULL) { + dive->sac = calculate_sac(dive); + dive->otu = calculate_otu(dive); + } +} + static void get_string(char **str, const char *s) { int len; @@ -452,8 +460,7 @@ static void fill_dive_list(void) for (i = 0; i < dive_table.nr; i++) { struct dive *dive = dive_table.dives[i]; - dive->otu = calculate_otu(dive); - dive->sac = calculate_sac(dive); + update_cylinder_related_info(dive); gtk_list_store_append(store, &iter); gtk_list_store_set(store, &iter, DIVE_INDEX, i, |