equipment: clean up cylinder pressure spinbutton setup

[ext/subsurface.git] / equipment.c

diff --git a/equipment.c b/equipment.c
index 64a05d5b024d3500ffbc6ff666cd3726d3ce99aa..c2d05d2ba4d2304595acd175d487a8a913e999d0 100644 (file)
--- a/equipment.c
+++ b/equipment.c
@@ -71,12 +71,12 @@ static int convert_volume_pressure(int ml, int mbar, double *v, double *p)
        int decimals = 1;
        double volume, pressure;
 
+       volume = ml / 1000.0;
        if (mbar) {
                if (output_units.volume == CUFT) {
                        volume = ml_to_cuft(ml);
                        volume *= bar_to_atm(mbar / 1000.0);
-               } else
-                       volume = ml / 1000.0;
+               }
 
                if (output_units.pressure == PSI) {
                        pressure = mbar_to_PSI(mbar);
@@ -98,13 +98,13 @@ static void set_cylinder_type_spinbuttons(struct cylinder_widget *cylinder, int
        gtk_spin_button_set_value(cylinder->pressure, pressure);
 }
 
-static void set_cylinder_pressure_spinbuttons(struct cylinder_widget *cylinder, int start, int end)
+static void set_cylinder_pressure_spinbuttons(struct cylinder_widget *cylinder, cylinder_t *cyl)
 {
        double pressure;
 
-       convert_pressure(start, &pressure);
+       convert_pressure(cyl->start.mbar, &pressure);
        gtk_spin_button_set_value(cylinder->start, pressure);
-       convert_pressure(end, &pressure);
+       convert_pressure(cyl->end.mbar, &pressure);
        gtk_spin_button_set_value(cylinder->end, pressure);
 }
 
@@ -266,8 +266,7 @@ static void show_cylinder(cylinder_t *cyl, struct cylinder_widget *cylinder)
 
        set_cylinder_type_spinbuttons(cylinder,
                cyl->type.size.mliter, cyl->type.workingpressure.mbar);
-       set_cylinder_pressure_spinbuttons(cylinder,
-               cyl->start.mbar, cyl->end.mbar);
+       set_cylinder_pressure_spinbuttons(cylinder, cyl);
        o2 = cyl->gasmix.o2.permille / 10.0;
        gtk_widget_set_sensitive(cylinder->o2, !!o2);
        gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(cylinder->gasmix_button), !!o2);
@@ -283,18 +282,24 @@ static int cyl_nothing(cylinder_t *cyl)
                !cyl->type.description &&
                !cyl->gasmix.o2.permille &&
                !cyl->gasmix.he.permille &&
+               !cyl->sample_start.mbar &&
+               !cyl->sample_end.mbar &&
                !cyl->start.mbar &&
                !cyl->end.mbar;
 }
 
 static void set_one_cylinder(int index, cylinder_t *cyl, GtkListStore *model, GtkTreeIter *iter)
 {
+       unsigned int start, end;
+
+       start = cyl->start.mbar ? : cyl->sample_start.mbar;
+       end = cyl->end.mbar ? : cyl->sample_end.mbar;
        gtk_list_store_set(model, iter,
                CYL_DESC, cyl->type.description ? : "",
                CYL_SIZE, cyl->type.size.mliter,
                CYL_WORKP, cyl->type.workingpressure.mbar,
-               CYL_STARTP, cyl->start.mbar,
-               CYL_ENDP, cyl->end.mbar,
+               CYL_STARTP, start,
+               CYL_ENDP, end,
                CYL_O2, cyl->gasmix.o2.permille,
                CYL_HE, cyl->gasmix.he.permille,
                -1);
@@ -360,7 +365,7 @@ static void fill_cylinder_info(struct cylinder_widget *cylinder, cylinder_t *cyl
        }
 
        if (pressure && output_units.volume == CUFT) {
-               volume = cuft_to_ml(volume);
+               volume = cuft_to_l(volume);
                volume /= bar_to_atm(pressure);
        }
 
@@ -464,7 +469,7 @@ static void fill_tank_list(GtkListStore *store)
                /* Is it in cuft and psi? */
                if (psi) {
                        double bar = psi_to_bar(psi);
-                       double airvolume = cuft_to_ml(size);
+                       double airvolume = cuft_to_l(size) * 1000.0;
                        double atm = bar_to_atm(bar);
 
                        ml = airvolume / atm + 0.5;