diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2012-09-25 23:19:57 -0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2012-09-26 10:58:39 -0700 |
commit | cb48db275e3877314c6da102990bed6c6980bd7f (patch) | |
tree | 93268c3621c14e18b146ece9e769986fa8be87b7 /uemis.h | |
parent | 43f122f9ff6eb933fcf831ac0b242f770204d2d5 (diff) | |
download | subsurface-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.h | 10 |
1 files changed, 8 insertions, 2 deletions
@@ -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]; |