summaryrefslogtreecommitdiffstats
path: root/uemis.h
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2012-09-25 23:19:57 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2012-09-26 10:58:39 -0700
commitcb48db275e3877314c6da102990bed6c6980bd7f (patch)
tree93268c3621c14e18b146ece9e769986fa8be87b7 /uemis.h
parent43f122f9ff6eb933fcf831ac0b242f770204d2d5 (diff)
downloadsubsurface-cb48db275e3877314c6da102990bed6c6980bd7f.tar.gz
Fix stupid packing error on Windows
I guess no one has ever tried to import Uemis dive data under Windows. The glib-2 libraries for Windows (at least the ones that are part of the mingw package, but my guess is this is true for all of them), force the whole program to be compiled with Windows packing rules for structures. That broke the structure we use for decoding Uemis binary data. This commit changes the data structure to no longer use unaligned 16bit values but instead two 8bit values and assemble them in the actual code. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'uemis.h')
-rw-r--r--uemis.h10
1 files changed, 8 insertions, 2 deletions
diff --git a/uemis.h b/uemis.h
index ba9a23402..baa6783a4 100644
--- a/uemis.h
+++ b/uemis.h
@@ -24,8 +24,14 @@ typedef struct {
uint16_t hold_depth;
uint16_t hold_time;
uint8_t active_tank;
- uint16_t tank_pressure; // (in cbar)
- uint16_t consumption; // (units unclear)
+ // bloody glib, when compiled for Windows, forces the whole program to use
+ // the Windows packing rules. So to avoid problems on Windows (and since
+ // only tank_pressure is currently used and that exactly once) I give in and
+ // make this silly low byte / high byte 8bit entries
+ uint8_t tank_pressure_low; // (in cbar)
+ uint8_t tank_pressure_high;
+ uint8_t consumption_low; // (units unclear)
+ uint8_t consumption_high;
uint8_t rgt; // (remaining gas time in minutes)
uint8_t cns;
uint8_t flags[8];