mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-01-19 14:25:27 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix reference tank information for LP steel tanks.

Browse source 
Commit f9bb3f7910 ("Clean up reference tank information table") had
cleaned up the tank info list so that you could sanely do tanks in
liters and with a working pressure in bar.

But the LP steel cylinders had somehow missed out on the ".psi =" part
of the equation, and as a result, what was supposed to be their working
pressure instead ended up being interpreted as their size in
milli-liters.

Oops.

Fix that, and also make the standard tank info filling code actually
verify the sanity of the reference tank table, so that if this happens
again, it will complain loudly instead of using nonsensical values.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Linus Torvalds 2012-04-01 15:38:52 -07:00
parent 9fad1cb50f
commit 3a7d577ff1
1 changed files with 21 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
equipment.c
View file

@ -664,10 +664,10 @@ static struct tank_info {
{ "AL100", .cuft = 100, .psi = 3300 },
/* Somewhat common LP steel cylinders */
{ "LP85", .cuft = 85, 2640 },
{ "LP95", .cuft = 95, 2640 },
{ "LP108", .cuft = 108, 2640 },
{ "LP121", .cuft = 121, 2640 },
{ "LP85", .cuft = 85, .psi = 2640 },
{ "LP95", .cuft = 95, .psi = 2640 },
{ "LP108", .cuft = 108, .psi = 2640 },
{ "LP121", .cuft = 121, .psi = 2640 },
/* Somewhat common HP steel cylinders */
{ "HP65", .cuft = 65, .psi = 3442 },
@ -696,23 +696,29 @@ static void fill_tank_list(GtkListStore *store)
GtkTreeIter iter;
struct tank_info *info = tank_info;
while (info->name) {
for (info = tank_info ; info->name; info++) {
int ml = info->ml;
int cuft = info->cuft;
int psi = info->psi;
int mbar;
double bar = info->bar;
if (psi && bar)
goto bad_tank_info;
if (ml && cuft)
goto bad_tank_info;
if (cuft && !psi)
goto bad_tank_info;
/* Is it in cuft and psi? */
if (psi) {
if (psi)
bar = psi_to_bar(psi);
if (cuft) {
double airvolume = cuft_to_l(cuft) * 1000.0;
double atm = bar_to_atm(bar);
if (cuft) {
double airvolume = cuft_to_l(cuft) * 1000.0;
double atm = bar_to_atm(bar);
ml = airvolume / atm + 0.5;
}
ml = airvolume / atm + 0.5;
}
mbar = bar * 1000 + 0.5;
@ -723,7 +729,10 @@ static void fill_tank_list(GtkListStore *store)
1, ml,
2, mbar,
-1);
info++;
continue;
bad_tank_info:
fprintf(stderr, "Bad tank info for '%s'\n", info->name);
}
}