diff options
| author |  Linus Torvalds <torvalds@linux-foundation.org> | 2011-09-01 13:46:24 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-09-01 13:46:24 -0700 |
| commit | adc7280c025e68f6d55d779ad56712cb17aa3ae7 (patch) | |
| tree | 964e9f42539ec5861ea5cd6380ed0974f33bc0f1 | |
| parent | 97419131248aacc96278ba7022b39af32359c26c (diff) | |
| download | subsurface-adc7280c025e68f6d55d779ad56712cb17aa3ae7.tar.gz | |
libdivecomputer does crazy gas mixes too
Did I just say "In comparison, the libdivecomputer output is nice and
sane"?
It turns out that libdivecomputer has been doing some drugs too when it
comes to gas mixes. Like showing O2 percentages as 255.0% and N2
percentages as -155.0%.
Clearly libdivecomputer uses a 'unsigned char' for oxygen percentage,
and makes "-1" be "undefined". And then it prints that non-existing mix
out, and in the process does MATH on the damn thing ("100-O2") to
"calculate" the nitrogen percentage.
Christ.
Just make the parser silently ignore the craziness, because printing out
"Strange percentage reading -155.0" a few hundred times just doesn't
make anything any better.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | parse-xml.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/parse-xml.c b/parse-xml.c index 892e0fb20..e0231ce46 100644 --- a/parse-xml.c +++ b/parse-xml.c @@ -326,6 +326,9 @@ static void percent(char *buffer, void *_fraction) static void gasmix(char *buffer, void *_fraction) { + /* libdivecomputer does negative percentages. */ + if (*buffer == '-') + return; if (gasmix_index < MAX_MIXES) percent(buffer, _fraction); } |