2011-08-28 23:58:26 +00:00
# include <stdio.h>
2011-08-30 00:51:54 +00:00
# include <ctype.h>
# include <string.h>
2011-08-30 23:23:47 +00:00
# include <stdlib.h>
# include <errno.h>
2011-11-05 20:51:37 +00:00
# include <unistd.h>
2013-01-31 00:08:36 +00:00
# include <assert.h>
2011-10-03 04:59:54 +00:00
# define __USE_XOPEN
2011-08-30 22:22:48 +00:00
# include <time.h>
2011-08-28 23:58:26 +00:00
# include <libxml/parser.h>
2013-03-15 17:02:14 +00:00
# include <libxml/parserInternals.h>
2011-08-28 23:58:26 +00:00
# include <libxml/tree.h>
2011-11-05 10:39:17 +00:00
# include <libxslt/transform.h>
2014-07-12 12:51:03 +00:00
# include <libdivecomputer/parser.h>
2013-10-06 15:55:58 +00:00
# include "gettext.h"
2011-08-31 01:23:59 +00:00
# include "dive.h"
Assemble the actual Suunto serial number
It turns out that the serial number returned by libdivecomputer isn't
really the serial number as interpreted by the vendor. Those tend to be
strings, but libdivecomputer gives us a 32bit number.
Some experimenting showed that for the Suunto devies tested the serial
number is encoded in that 32bit number:
It so happens that the Suunto serial number strings are strings that have
all numbers, but they aren't *one* number. They are four bytes
representing two numbers each, and the "23500027" string is actually the
four bytes 23 50 00 27 (0x17 0x32 0x00 0x1b). And libdivecomputer has
incorrectly parsed those four bytes as one number, not as the encoded
serial number string it is. So the value 389152795 is actually hex
0x1732001b, which is 0x17 0x32 0x00 0x1b, which is - 23 50 00 27.
This should be done by libdivecomputer, but hey, in the meantime this at
least shows the concept. And helps test the XML save/restore code.
It depends on the two patches that create the whole "device.c"
infrastructure, of course. With this, my dive file ends up having the
settings section look like this:
<divecomputerid model='Suunto Vyper Air' deviceid='d4629110'
serial='01201094' firmware='1.1.22'/>
<divecomputerid model='Suunto HelO2' deviceid='995dd566'
serial='23500027' firmware='1.0.4'/>
where the format of the firmware version is something I guessed at,
but it was the obvious choice (again, it's byte-based, I'm ignoring
the high byte that is zero for both of my Suuntos).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-10 00:14:21 +00:00
# include "device.h"
2011-08-30 23:42:05 +00:00
2013-11-02 19:00:16 +00:00
int verbose , quit ;
2014-02-15 06:36:50 +00:00
int metric = 1 ;
2011-08-30 20:48:05 +00:00
2014-03-14 18:26:07 +00:00
static xmlDoc * test_xslt_transforms ( xmlDoc * doc , const char * * params ) ;
2013-02-22 19:20:46 +00:00
2013-01-31 00:08:36 +00:00
/* the dive table holds the overall dive list; target table points at
* the table we are currently filling */
2011-08-31 01:40:25 +00:00
struct dive_table dive_table ;
2013-01-31 00:08:36 +00:00
struct dive_table * target_table = NULL ;
2013-02-22 19:20:46 +00:00
2011-08-31 01:40:25 +00:00
/*
* Add a dive into the dive_table array
*/
2013-01-31 00:08:36 +00:00
static void record_dive_to_table ( struct dive * dive , struct dive_table * table )
2011-08-31 00:18:33 +00:00
{
2013-01-31 00:08:36 +00:00
assert ( table ! = NULL ) ;
int nr = table - > nr , allocated = table - > allocated ;
struct dive * * dives = table - > dives ;
2011-08-31 00:18:33 +00:00
2011-08-31 01:40:25 +00:00
if ( nr > = allocated ) {
allocated = ( nr + 32 ) * 3 / 2 ;
dives = realloc ( dives , allocated * sizeof ( struct dive * ) ) ;
if ( ! dives )
exit ( 1 ) ;
2013-01-31 00:08:36 +00:00
table - > dives = dives ;
table - > allocated = allocated ;
2011-08-30 23:42:05 +00:00
}
2011-09-03 20:19:26 +00:00
dives [ nr ] = fixup_dive ( dive ) ;
2014-02-16 23:42:56 +00:00
table - > nr = nr + 1 ;
2011-08-31 00:18:33 +00:00
}
2013-01-31 00:08:36 +00:00
void record_dive ( struct dive * dive )
2012-04-03 02:19:01 +00:00
{
2013-01-31 00:08:36 +00:00
record_dive_to_table ( dive , & dive_table ) ;
2012-04-03 02:19:01 +00:00
}
2011-09-01 18:22:05 +00:00
static void start_match ( const char * type , const char * name , char * buffer )
2011-08-30 20:58:19 +00:00
{
2011-09-01 18:22:05 +00:00
if ( verbose > 2 )
printf ( " Matching %s '%s' (%s) \n " ,
2014-02-16 23:42:56 +00:00
type , name , buffer ) ;
2011-08-30 20:58:19 +00:00
}
2011-09-01 18:22:05 +00:00
static void nonmatch ( const char * type , const char * name , char * buffer )
2011-08-30 20:58:19 +00:00
{
2011-09-01 18:22:05 +00:00
if ( verbose > 1 )
printf ( " Unable to match %s '%s' (%s) \n " ,
2014-02-16 23:42:56 +00:00
type , name , buffer ) ;
2011-08-30 20:58:19 +00:00
}
2011-08-30 22:22:48 +00:00
typedef void ( * matchfn_t ) ( char * buffer , void * ) ;
2011-08-30 20:48:05 +00:00
2011-09-01 18:22:05 +00:00
static int match ( const char * pattern , int plen ,
2013-11-02 19:00:16 +00:00
const char * name ,
2011-09-01 18:22:05 +00:00
matchfn_t fn , char * buf , void * data )
2011-08-30 20:48:05 +00:00
{
2013-11-02 19:00:16 +00:00
switch ( name [ plen ] ) {
2014-02-16 23:42:56 +00:00
case ' \0 ' :
case ' . ' :
2013-11-02 19:00:16 +00:00
break ;
default :
2011-09-01 18:22:05 +00:00
return 0 ;
2013-11-02 19:00:16 +00:00
}
if ( memcmp ( pattern , name , plen ) )
2011-08-30 22:22:48 +00:00
return 0 ;
fn ( buf , data ) ;
return 1 ;
2011-08-30 20:48:05 +00:00
}
2011-09-07 02:07:17 +00:00
2013-01-11 01:26:10 +00:00
struct units xml_parsing_units ;
2012-12-10 17:20:57 +00:00
const struct units SI_units = SI_UNITS ;
const struct units IMPERIAL_units = IMPERIAL_UNITS ;
2011-09-07 02:07:17 +00:00
2011-08-30 20:48:05 +00:00
/*
* Dive info as it is being built up . .
*/
2012-11-25 02:50:21 +00:00
static struct divecomputer * cur_dc ;
2012-09-20 03:42:11 +00:00
static struct dive * cur_dive ;
static dive_trip_t * cur_trip = NULL ;
2012-01-05 16:16:08 +00:00
static struct sample * cur_sample ;
2014-06-09 00:42:15 +00:00
static struct picture * cur_picture ;
2011-09-23 01:02:54 +00:00
static struct {
int active ;
duration_t time ;
int type , flags , value ;
const char * name ;
2012-01-05 16:16:08 +00:00
} cur_event ;
2012-12-26 21:47:54 +00:00
static struct {
2014-02-16 23:42:56 +00:00
struct {
const char * model ;
uint32_t deviceid ;
const char * nickname , * serial_nr , * firmware ;
} dc ;
2012-12-26 21:47:54 +00:00
} cur_settings ;
2014-01-15 18:54:41 +00:00
static bool in_settings = false ;
2014-04-11 06:17:35 +00:00
static bool in_userid = false ;
2012-01-05 16:16:08 +00:00
static struct tm cur_tm ;
2012-03-24 04:07:53 +00:00
static int cur_cylinder_index , cur_ws_index ;
2012-12-31 02:11:01 +00:00
static int lastndl , laststoptime , laststopdepth , lastcns , lastpo2 , lastindeco ;
First step in cleaning up cylinder pressure sensor logic
This clarifies/changes the meaning of our "cylinderindex" entry in our
samples. It has been rather confused, because different dive computers
have done things differently, and the naming really hasn't helped.
There are two totally different - and independent - cylinder "indexes":
- the pressure sensor index, which indicates which cylinder the sensor
data is from.
- the "active cylinder" index, which indicates which cylinder we actually
breathe from.
These two values really are totally independent, and have nothing
what-so-ever to do with each other. The sensor index may well be fixed:
many dive computers only support a single pressure sensor (whether
wireless or wired), and the sensor index is thus always zero.
Other dive computers may support multiple pressure sensors, and the gas
switch event may - or may not - indicate that the sensor changed too. A
dive computer might give the sensor data for *all* cylinders it can read,
regardless of which one is the one we're actively breathing. In fact, some
dive computers might give sensor data for not just *your* cylinder, but
your buddies.
This patch renames "cylinderindex" in the samples as "sensor", making it
quite clear that it's about which sensor index the pressure data in the
sample is about.
The way we figure out which is the currently active gas is with an
explicit has change event. If a computer (like the Uemis Zurich) joins the
two concepts together, then a sensor change should also create a gas
switch event. This patch also changes the Uemis importer to do that.
Finally, it should be noted that the plot info works totally separately
from the sample data, and is about what we actually *display*, not about
the sample pressures etc. In the plot info, the "cylinderindex" does in
fact mean the currently active cylinder, and while it is initially set to
match the sensor information from the samples, we then walk the gas change
events and fix it up - and if the active cylinder differs from the sensor
cylinder, we clear the sensor data.
[Dirk Hohndel: this conflicted with some of my recent changes - I think
I merged things correctly...]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-12-31 04:00:51 +00:00
static int lastcylinderindex , lastsensor ;
2011-08-30 22:22:48 +00:00
2013-02-22 16:52:35 +00:00
/*
* If we don ' t have an explicit dive computer ,
* we use the implicit one that every dive has . .
*/
static struct divecomputer * get_dc ( void )
{
2014-02-16 23:42:56 +00:00
return cur_dc ? : & cur_dive - > dc ;
2013-02-22 16:52:35 +00:00
}
2011-09-05 20:45:14 +00:00
static enum import_source {
UNKNOWN ,
LIBDIVECOMPUTER ,
2011-09-05 21:29:08 +00:00
DIVINGLOG ,
2011-09-07 00:01:28 +00:00
UDDF ,
2011-09-05 20:45:14 +00:00
} import_source ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void divedate ( char * buffer , timestamp_t * when )
2011-08-30 22:22:48 +00:00
{
2014-02-16 23:42:56 +00:00
int d , m , y ;
int hh , mm , ss ;
2013-01-29 05:07:52 +00:00
2014-02-16 23:42:56 +00:00
hh = 0 ;
mm = 0 ;
ss = 0 ;
2013-01-29 05:07:52 +00:00
if ( sscanf ( buffer , " %d.%d.%d %d:%d:%d " , & d , & m , & y , & hh , & mm , & ss ) > = 3 ) {
/* This is ok, and we got at least the date */
} else if ( sscanf ( buffer , " %d-%d-%d %d:%d:%d " , & y , & m , & d , & hh , & mm , & ss ) > = 3 ) {
/* This is also ok */
2011-09-02 00:13:39 +00:00
} else {
fprintf ( stderr , " Unable to parse date '%s' \n " , buffer ) ;
2013-01-29 05:07:52 +00:00
return ;
2011-08-30 22:22:48 +00:00
}
2013-01-29 05:07:52 +00:00
cur_tm . tm_year = y ;
2014-02-16 23:42:56 +00:00
cur_tm . tm_mon = m - 1 ;
2013-01-29 05:07:52 +00:00
cur_tm . tm_mday = d ;
cur_tm . tm_hour = hh ;
cur_tm . tm_min = mm ;
cur_tm . tm_sec = ss ;
* when = utc_mktime ( & cur_tm ) ;
2011-08-30 22:22:48 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void divetime ( char * buffer , timestamp_t * when )
2011-08-30 22:22:48 +00:00
{
2014-02-16 23:42:56 +00:00
int h , m , s = 0 ;
2011-08-30 22:22:48 +00:00
if ( sscanf ( buffer , " %d:%d:%d " , & h , & m , & s ) > = 2 ) {
2012-01-05 16:16:08 +00:00
cur_tm . tm_hour = h ;
cur_tm . tm_min = m ;
cur_tm . tm_sec = s ;
2013-01-29 05:07:52 +00:00
* when = utc_mktime ( & cur_tm ) ;
2011-08-30 22:22:48 +00:00
}
}
2011-08-30 23:59:03 +00:00
/* Libdivecomputer: "2011-03-20 10:22:38" */
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void divedatetime ( char * buffer , timestamp_t * when )
2011-08-30 23:59:03 +00:00
{
2014-02-16 23:42:56 +00:00
int y , m , d ;
int hr , min , sec ;
2011-08-30 23:59:03 +00:00
if ( sscanf ( buffer , " %d-%d-%d %d:%d:%d " ,
2014-02-16 23:42:56 +00:00
& y , & m , & d , & hr , & min , & sec ) = = 6 ) {
2012-01-05 16:16:08 +00:00
cur_tm . tm_year = y ;
2014-02-16 23:42:56 +00:00
cur_tm . tm_mon = m - 1 ;
2012-01-05 16:16:08 +00:00
cur_tm . tm_mday = d ;
cur_tm . tm_hour = hr ;
cur_tm . tm_min = min ;
cur_tm . tm_sec = sec ;
* when = utc_mktime ( & cur_tm ) ;
2011-08-30 23:59:03 +00:00
}
}
2014-02-16 23:42:56 +00:00
enum ParseState {
FINDSTART ,
FINDEND
} ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void divetags ( char * buffer , struct tag_entry * * tags )
2013-04-09 20:06:30 +00:00
{
2013-11-02 01:12:42 +00:00
int i = 0 , start = 0 , end = 0 ;
enum ParseState state = FINDEND ;
2013-12-13 00:12:04 +00:00
int len = buffer ? strlen ( buffer ) : 0 ;
2014-02-16 23:42:56 +00:00
while ( i < len ) {
2013-11-02 01:12:42 +00:00
if ( buffer [ i ] = = ' , ' ) {
if ( state = = FINDSTART ) {
/* Detect empty tags */
} else if ( state = = FINDEND ) {
/* Found end of tag */
2013-12-13 00:12:04 +00:00
if ( i > 0 & & buffer [ i - 1 ] ! = ' \\ ' ) {
2014-02-16 23:42:56 +00:00
buffer [ i ] = ' \0 ' ;
state = FINDSTART ;
taglist_add_tag ( tags , buffer + start ) ;
2013-11-02 01:12:42 +00:00
} else {
2014-02-16 23:42:56 +00:00
state = FINDSTART ;
2013-04-09 20:06:30 +00:00
}
}
2013-11-02 01:12:42 +00:00
} else if ( buffer [ i ] = = ' ' ) {
/* Handled */
} else {
/* Found start of tag */
if ( state = = FINDSTART ) {
state = FINDEND ;
start = i ;
} else if ( state = = FINDEND ) {
2013-12-13 00:12:04 +00:00
end = i ;
2013-11-02 01:12:42 +00:00
}
2013-04-09 20:06:30 +00:00
}
2013-11-02 01:12:42 +00:00
i + + ;
2013-12-11 20:21:52 +00:00
}
if ( state = = FINDEND ) {
if ( end < start )
2013-12-13 00:12:04 +00:00
end = len - 1 ;
if ( len > 0 ) {
buffer [ end + 1 ] = ' \0 ' ;
taglist_add_tag ( tags , buffer + start ) ;
2013-12-11 20:21:52 +00:00
}
}
2013-04-09 20:06:30 +00:00
}
2011-08-30 23:23:47 +00:00
enum number_type {
NEITHER ,
FLOAT
} ;
2014-01-08 06:51:22 +00:00
static enum number_type parse_float ( const char * buffer , double * res , const char * * endp )
2011-08-30 23:23:47 +00:00
{
2013-02-23 00:18:39 +00:00
double val ;
2014-01-15 18:54:41 +00:00
static bool first_time = true ;
2011-08-30 23:23:47 +00:00
2013-01-23 20:09:29 +00:00
errno = 0 ;
2013-10-05 07:29:09 +00:00
val = ascii_strtod ( buffer , endp ) ;
2013-02-23 00:18:39 +00:00
if ( errno | | * endp = = buffer )
return NEITHER ;
2013-03-07 19:43:51 +00:00
if ( * * endp = = ' , ' ) {
2014-01-21 21:03:08 +00:00
if ( IS_FP_SAME ( val , rint ( val ) ) ) {
2013-03-07 19:43:51 +00:00
/* we really want to send an error if this is a Subsurface native file
* as this is likely indication of a bug - but right now we don ' t have
* that information available */
if ( first_time ) {
fprintf ( stderr , " Floating point value with decimal comma (%s)? \n " , buffer ) ;
2014-01-15 18:54:41 +00:00
first_time = false ;
2013-03-07 19:43:51 +00:00
}
2014-01-08 06:51:22 +00:00
/* Try again in permissive mode*/
val = strtod_flags ( buffer , endp , 0 ) ;
2013-03-07 19:43:51 +00:00
}
}
2013-02-23 00:18:39 +00:00
* res = val ;
return FLOAT ;
}
union int_or_float {
double fp ;
} ;
2011-08-30 23:23:47 +00:00
2013-02-23 00:18:39 +00:00
static enum number_type integer_or_float ( char * buffer , union int_or_float * res )
{
2014-01-08 06:51:22 +00:00
const char * end ;
2013-02-23 00:18:39 +00:00
return parse_float ( buffer , & res - > fp , & end ) ;
2011-08-30 23:23:47 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void pressure ( char * buffer , pressure_t * pressure )
2011-08-30 23:23:47 +00:00
{
2013-12-20 17:37:56 +00:00
double mbar = 0.0 ;
2011-08-30 23:23:47 +00:00
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2011-09-02 21:06:26 +00:00
/* Just ignore zero values */
if ( ! val . fp )
break ;
2013-01-11 01:26:10 +00:00
switch ( xml_parsing_units . pressure ) {
2011-09-07 00:01:28 +00:00
case PASCAL :
mbar = val . fp / 100 ;
break ;
2011-09-02 20:59:39 +00:00
case BAR :
/* Assume mbar, but if it's really small, it's bar */
mbar = val . fp ;
2014-07-17 09:00:06 +00:00
if ( fabs ( mbar ) < 5000 )
2011-09-02 20:59:39 +00:00
mbar = mbar * 1000 ;
break ;
case PSI :
2013-11-26 22:11:30 +00:00
mbar = psi_to_mbar ( val . fp ) ;
2011-08-30 23:23:47 +00:00
break ;
}
2014-07-17 09:00:06 +00:00
if ( fabs ( mbar ) > 5 & & fabs ( mbar ) < 5000000 ) {
2014-02-12 22:19:53 +00:00
pressure - > mbar = rint ( mbar ) ;
2011-08-30 23:23:47 +00:00
break ;
}
2011-09-02 20:59:39 +00:00
/* fallthrough */
2011-08-30 23:23:47 +00:00
default :
printf ( " Strange pressure reading %s \n " , buffer ) ;
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void salinity ( char * buffer , int * salinity )
2012-11-12 19:57:49 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2014-02-12 22:19:53 +00:00
* salinity = rint ( val . fp * 10.0 ) ;
2012-11-12 19:57:49 +00:00
break ;
default :
printf ( " Strange salinity reading %s \n " , buffer ) ;
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void depth ( char * buffer , depth_t * depth )
2011-08-30 23:23:47 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2013-01-11 01:26:10 +00:00
switch ( xml_parsing_units . length ) {
2011-09-02 20:59:39 +00:00
case METERS :
2014-02-12 22:19:53 +00:00
depth - > mm = rint ( val . fp * 1000 ) ;
2011-09-02 20:59:39 +00:00
break ;
case FEET :
2013-11-26 22:11:30 +00:00
depth - > mm = feet_to_mm ( val . fp ) ;
2011-09-02 20:59:39 +00:00
break ;
}
2011-08-30 23:23:47 +00:00
break ;
default :
printf ( " Strange depth reading %s \n " , buffer ) ;
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void weight ( char * buffer , weight_t * weight )
2011-12-24 03:41:16 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2013-01-11 01:26:10 +00:00
switch ( xml_parsing_units . weight ) {
2011-12-24 03:41:16 +00:00
case KG :
2014-02-12 22:19:53 +00:00
weight - > grams = rint ( val . fp * 1000 ) ;
2011-12-24 03:41:16 +00:00
break ;
case LBS :
2013-11-26 22:11:30 +00:00
weight - > grams = lbs_to_grams ( val . fp ) ;
2011-12-24 03:41:16 +00:00
break ;
}
break ;
default :
2012-11-24 10:12:16 +00:00
printf ( " Strange weight reading %s \n " , buffer ) ;
2011-12-24 03:41:16 +00:00
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void temperature ( char * buffer , temperature_t * temperature )
2011-08-30 23:23:47 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2013-01-11 01:26:10 +00:00
switch ( xml_parsing_units . temperature ) {
2011-09-07 00:01:28 +00:00
case KELVIN :
temperature - > mkelvin = val . fp * 1000 ;
break ;
2011-09-02 20:59:39 +00:00
case CELSIUS :
2013-11-26 22:11:30 +00:00
temperature - > mkelvin = C_to_mkelvin ( val . fp ) ;
2011-08-30 23:23:47 +00:00
break ;
2011-09-02 20:59:39 +00:00
case FAHRENHEIT :
2013-11-26 22:11:30 +00:00
temperature - > mkelvin = F_to_mkelvin ( val . fp ) ;
2011-08-30 23:23:47 +00:00
break ;
}
break ;
default :
printf ( " Strange temperature reading %s \n " , buffer ) ;
}
2013-03-08 19:52:10 +00:00
/* temperatures outside -40C .. +70C should be ignored */
if ( temperature - > mkelvin < ZERO_C_IN_MKELVIN - 40000 | |
2014-02-16 23:42:56 +00:00
temperature - > mkelvin > ZERO_C_IN_MKELVIN + 70000 )
2013-03-08 19:52:10 +00:00
temperature - > mkelvin = 0 ;
2011-08-30 23:23:47 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void sampletime ( char * buffer , duration_t * time )
2011-08-30 23:23:47 +00:00
{
2011-08-31 00:45:03 +00:00
int i ;
int min , sec ;
2011-08-30 23:23:47 +00:00
2011-08-31 00:45:03 +00:00
i = sscanf ( buffer , " %d:%d " , & min , & sec ) ;
switch ( i ) {
case 1 :
sec = min ;
min = 0 ;
/* fallthrough */
case 2 :
2014-02-16 23:42:56 +00:00
time - > seconds = sec + min * 60 ;
2011-08-30 23:23:47 +00:00
break ;
default :
printf ( " Strange sample time reading %s \n " , buffer ) ;
}
}
2014-07-08 19:29:06 +00:00
static void offsettime ( char * buffer , offset_t * time )
{
duration_t uoffset ;
int sign = 1 ;
if ( * buffer = = ' - ' ) {
sign = - 1 ;
buffer + + ;
}
/* yes, this could indeed fail if we have an offset > 34yrs
* - too bad */
sampletime ( buffer , & uoffset ) ;
time - > seconds = sign * uoffset . seconds ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void duration ( char * buffer , duration_t * time )
2011-08-31 00:45:03 +00:00
{
2013-02-25 22:19:16 +00:00
/* DivingLog 5.08 (and maybe other versions) appear to sometimes
* store the dive time as 44.00 instead of 44 : 00 ;
* This attempts to parse this in a fairly robust way */
2014-02-16 23:42:56 +00:00
if ( ! strchr ( buffer , ' : ' ) & & strchr ( buffer , ' . ' ) ) {
2013-02-25 22:19:16 +00:00
char * mybuffer = strdup ( buffer ) ;
2014-02-16 23:42:56 +00:00
char * dot = strchr ( mybuffer , ' . ' ) ;
2013-02-25 22:19:16 +00:00
* dot = ' : ' ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
sampletime ( mybuffer , time ) ;
2013-12-10 23:53:30 +00:00
free ( mybuffer ) ;
2013-02-25 22:19:16 +00:00
} else {
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
sampletime ( buffer , time ) ;
2013-02-25 22:19:16 +00:00
}
2011-08-31 00:45:03 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void percent ( char * buffer , fraction_t * fraction )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
{
2013-02-23 00:18:39 +00:00
double val ;
2014-01-08 06:51:22 +00:00
const char * end ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
2013-02-23 00:18:39 +00:00
switch ( parse_float ( buffer , & val , & end ) ) {
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
case FLOAT :
2013-02-23 00:18:39 +00:00
/* Turn fractions into percent unless explicit.. */
if ( val < = 1.0 ) {
2013-10-05 07:29:09 +00:00
while ( isspace ( * end ) )
2013-02-23 00:18:39 +00:00
end + + ;
if ( * end ! = ' % ' )
val * = 100 ;
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
2013-02-23 00:18:39 +00:00
/* Then turn percent into our integer permille format */
if ( val > = 0 & & val < = 100.0 ) {
2014-02-12 22:19:53 +00:00
fraction - > permille = rint ( val * 10 ) ;
2013-02-23 00:18:39 +00:00
break ;
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
default :
2014-02-16 23:42:56 +00:00
printf ( translate ( " gettextFromC " , " Strange percentage reading %s \n " ) , buffer ) ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
break ;
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void gasmix ( char * buffer , fraction_t * fraction )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
{
2011-09-01 20:46:24 +00:00
/* libdivecomputer does negative percentages. */
if ( * buffer = = ' - ' )
return ;
2012-01-05 16:16:08 +00:00
if ( cur_cylinder_index < MAX_CYLINDERS )
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
percent ( buffer , fraction ) ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void gasmix_nitrogen ( char * buffer , struct gasmix * gasmix )
2011-09-01 23:26:11 +00:00
{
/* Ignore n2 percentages. There's no value in them. */
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void cylindersize ( char * buffer , volume_t * volume )
2011-09-04 03:31:18 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2014-02-12 22:19:53 +00:00
volume - > mliter = rint ( val . fp * 1000 ) ;
2011-09-04 03:31:18 +00:00
break ;
default :
printf ( " Strange volume reading %s \n " , buffer ) ;
break ;
}
}
2014-06-11 17:48:48 +00:00
/* Trim a character string by removing leading and trailing white space characters.
* Parameter : a pointer to a null - terminated character string ( buffer ) ;
* Return value : length of the trimmed string , excluding the terminal 0x0 byte
* The original pointer ( buffer ) remains valid after this function has been called
* and points to the trimmed string */
int trimspace ( char * buffer ) {
int i , size , start , end ;
size = strlen ( buffer ) ;
for ( start = 0 ; isspace ( buffer [ start ] ) ; start + + )
if ( start > = size ) return 0 ; // Find 1st character following leading whitespace
for ( end = size - 1 ; isspace ( buffer [ end ] ) ; end - - ) // Find last character before trailing whitespace
if ( end < = 0 ) return 0 ;
for ( i = start ; i < = end ; i + + ) // Move the nonspace characters to the start of the string
buffer [ i - start ] = buffer [ i ] ;
size = end - start + 1 ;
buffer [ size ] = 0x0 ; // then terminate the string
return size ; // return string length
}
2011-09-02 02:56:04 +00:00
static void utf8_string ( char * buffer , void * _res )
{
2014-06-12 02:02:20 +00:00
char * * res = _res ;
2014-06-11 17:48:48 +00:00
int size ;
size = trimspace ( buffer ) ;
2014-06-12 02:02:20 +00:00
if ( size )
* res = strdup ( buffer ) ;
2011-09-02 02:56:04 +00:00
}
2014-06-11 17:48:48 +00:00
/* Extract the dive computer type from the xml text buffer */
static void get_dc_type ( char * buffer , enum dive_comp_type * i )
{
if ( ( trimspace ( buffer ) ) & & ( strcmp ( buffer , " CCR " ) = = 0 ) )
* i = CCR ; // if the xml string = "CCR", set dc-type to CCR
} // otherwise the default dc-type is used (OC)
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
# define MATCH(pattern, fn, dest) ({ \
/* Silly type compatibility test */ \
if ( 0 ) ( fn ) ( " test " , dest ) ; \
match ( pattern , strlen ( pattern ) , name , ( matchfn_t ) ( fn ) , buf , dest ) ; } )
2011-09-01 18:22:05 +00:00
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void get_index ( char * buffer , int * i )
2011-09-02 22:01:53 +00:00
{
* i = atoi ( buffer ) ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void get_uint8 ( char * buffer , uint8_t * i )
{
* i = atoi ( buffer ) ;
}
static void get_bearing ( char * buffer , bearing_t * bearing )
{
bearing - > degrees = atoi ( buffer ) ;
}
static void get_rating ( char * buffer , int * i )
2013-01-29 21:30:02 +00:00
{
int j = atoi ( buffer ) ;
if ( j > = 0 & & j < = 5 ) {
* i = j ;
}
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void double_to_o2pressure ( char * buffer , o2pressure_t * i )
2012-12-08 04:08:29 +00:00
{
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
i - > mbar = rint ( ascii_strtod ( buffer , NULL ) * 1000.0 ) ;
2012-12-08 04:08:29 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void hex_value ( char * buffer , uint32_t * i )
2012-11-25 19:44:27 +00:00
{
2013-11-22 21:31:52 +00:00
* i = strtoul ( buffer , NULL , 16 ) ;
2012-11-25 19:44:27 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void get_tripflag ( char * buffer , tripflag_t * tf )
2012-08-22 05:04:24 +00:00
{
2012-11-26 22:52:07 +00:00
* tf = strcmp ( buffer , " NOTRIP " ) ? TF_NONE : NO_TRIP ;
2012-08-22 05:04:24 +00:00
}
2011-09-05 21:29:08 +00:00
/*
* Divinglog is crazy . The temperatures are in celsius . EXCEPT
* for the sample temperatures , that are in Fahrenheit .
* WTF ?
2011-09-11 20:16:23 +00:00
*
* Oh , and I think Diving Log * internally * probably kept them
* in celsius , because I ' m seeing entries like
*
* < Temp > 32.0 < / Temp >
*
* in there . Which is freezing , aka 0 degC . I bet the " 0 " is
* what Diving Log uses for " no temperature " .
*
* So throw away crap like that .
2012-10-10 09:14:55 +00:00
*
* It gets worse . Sometimes the sample temperatures are in
* Celsius , which apparently happens if you are in a SI
* locale . So we now do :
*
* - temperatures < 32.0 = = Celsius
* - temperature = = 32.0 - > garbage , it ' s a missing temperature ( zero converted from C to F )
* - temperatures > 32.0 = = Fahrenheit
2011-09-05 21:29:08 +00:00
*/
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void fahrenheit ( char * buffer , temperature_t * temperature )
2011-09-05 21:29:08 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2014-01-21 21:03:08 +00:00
if ( IS_FP_SAME ( val . fp , 32.0 ) )
2011-09-11 20:16:23 +00:00
break ;
2012-10-10 09:14:55 +00:00
if ( val . fp < 32.0 )
temperature - > mkelvin = C_to_mkelvin ( val . fp ) ;
else
temperature - > mkelvin = F_to_mkelvin ( val . fp ) ;
2011-09-05 21:29:08 +00:00
break ;
default :
fprintf ( stderr , " Crazy Diving Log temperature reading %s \n " , buffer ) ;
}
}
2011-09-11 19:24:57 +00:00
/*
* Did I mention how bat - shit crazy divinglog is ? The sample
* pressures are in PSI . But the tank working pressure is in
* bar . WTF ^ 2 ?
*
2011-09-15 16:43:14 +00:00
* Crazy stuff like this is why subsurface has everything in
2011-09-11 19:24:57 +00:00
* these inconvenient typed structures , and you have to say
* " pressure->mbar " to get the actual value . Exactly so that
* you can never have unit confusion .
2012-10-10 09:14:55 +00:00
*
* It gets worse : sometimes apparently the pressures are in
* bar , sometimes in psi . Dirk suspects that this may be a
* DivingLog Uemis importer bug , and that they are always
* supposed to be in bar , but that the importer got the
* sample importing wrong .
*
* Sadly , there ' s no way to really tell . So I think we just
* have to have some arbitrary cut - off point where we assume
* that smaller values mean bar . . Not good .
2011-09-11 19:24:57 +00:00
*/
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void psi_or_bar ( char * buffer , pressure_t * pressure )
2011-09-11 19:24:57 +00:00
{
union int_or_float val ;
switch ( integer_or_float ( buffer , & val ) ) {
case FLOAT :
2012-10-10 09:14:55 +00:00
if ( val . fp > 400 )
pressure - > mbar = psi_to_mbar ( val . fp ) ;
else
2014-02-12 22:19:53 +00:00
pressure - > mbar = rint ( val . fp * 1000 ) ;
2011-09-11 19:24:57 +00:00
break ;
default :
fprintf ( stderr , " Crazy Diving Log PSI reading %s \n " , buffer ) ;
}
}
2013-11-02 19:00:16 +00:00
static int divinglog_fill_sample ( struct sample * sample , const char * name , char * buf )
2011-09-05 21:29:08 +00:00
{
2014-02-16 23:42:56 +00:00
return MATCH ( " time.p " , sampletime , & sample - > time ) | |
MATCH ( " depth.p " , depth , & sample - > depth ) | |
MATCH ( " temp.p " , fahrenheit , & sample - > temperature ) | |
MATCH ( " press1.p " , psi_or_bar , & sample - > cylinderpressure ) | |
0 ;
2011-09-05 21:29:08 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void uddf_gasswitch ( char * buffer , struct sample * sample )
2013-02-22 16:52:35 +00:00
{
int idx = atoi ( buffer ) ;
int seconds = sample - > time . seconds ;
struct dive * dive = cur_dive ;
struct divecomputer * dc = get_dc ( ) ;
add_gas_switch_event ( dive , dc , seconds , idx ) ;
}
2013-11-02 19:00:16 +00:00
static int uddf_fill_sample ( struct sample * sample , const char * name , char * buf )
2011-09-07 00:33:52 +00:00
{
2014-02-16 23:42:56 +00:00
return MATCH ( " divetime " , sampletime , & sample - > time ) | |
MATCH ( " depth " , depth , & sample - > depth ) | |
MATCH ( " temperature " , temperature , & sample - > temperature ) | |
MATCH ( " tankpressure " , pressure , & sample - > cylinderpressure ) | |
MATCH ( " ref.switchmix " , uddf_gasswitch , sample ) | |
0 ;
2011-09-07 00:33:52 +00:00
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void eventtime ( char * buffer , duration_t * duration )
2011-09-23 01:02:54 +00:00
{
sampletime ( buffer , duration ) ;
2012-01-05 16:16:08 +00:00
if ( cur_sample )
duration - > seconds + = cur_sample - > time . seconds ;
2011-09-23 01:02:54 +00:00
}
2013-01-02 01:29:38 +00:00
static void try_to_match_autogroup ( const char * name , char * buf )
{
int autogroupvalue ;
start_match ( " autogroup " , name , buf ) ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " state.autogroup " , get_index , & autogroupvalue ) ) {
2013-01-02 01:29:38 +00:00
set_autogroup ( autogroupvalue ) ;
return ;
}
nonmatch ( " autogroup " , name , buf ) ;
}
2014-06-11 17:48:48 +00:00
void add_gas_switch_event ( struct dive * dive , struct divecomputer * dc , int seconds , int idx )
{
/* The gas switch event format is insane. It will be fixed, I think */
int o2 = get_o2 ( & dive - > cylinder [ idx ] . gasmix ) ;
int he = get_he ( & dive - > cylinder [ idx ] . gasmix ) ;
int value ;
o2 = ( o2 + 5 ) / 10 ;
he = ( he + 5 ) / 10 ;
value = o2 + ( he < < 16 ) ;
2014-07-12 12:51:03 +00:00
add_event ( dc , seconds , he ? SAMPLE_EVENT_GASCHANGE2 : SAMPLE_EVENT_GASCHANGE , 0 , value , " gaschange " ) ;
2014-06-11 17:48:48 +00:00
}
static void get_cylinderindex ( char * buffer , uint8_t * i )
{
* i = atoi ( buffer ) ;
if ( lastcylinderindex ! = * i ) {
add_gas_switch_event ( cur_dive , get_dc ( ) , cur_sample - > time . seconds , * i ) ;
lastcylinderindex = * i ;
}
}
static void get_sensor ( char * buffer , uint8_t * i )
{
* i = atoi ( buffer ) ;
lastsensor = * i ;
}
2012-12-26 21:47:54 +00:00
static void try_to_fill_dc_settings ( const char * name , char * buf )
{
start_match ( " divecomputerid " , name , buf ) ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " model.divecomputerid " , utf8_string , & cur_settings . dc . model ) )
2012-12-26 21:47:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " deviceid.divecomputerid " , hex_value , & cur_settings . dc . deviceid ) )
2012-12-26 21:47:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " nickname.divecomputerid " , utf8_string , & cur_settings . dc . nickname ) )
2012-12-26 21:47:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " serial.divecomputerid " , utf8_string , & cur_settings . dc . serial_nr ) )
Assemble the actual Suunto serial number
It turns out that the serial number returned by libdivecomputer isn't
really the serial number as interpreted by the vendor. Those tend to be
strings, but libdivecomputer gives us a 32bit number.
Some experimenting showed that for the Suunto devies tested the serial
number is encoded in that 32bit number:
It so happens that the Suunto serial number strings are strings that have
all numbers, but they aren't *one* number. They are four bytes
representing two numbers each, and the "23500027" string is actually the
four bytes 23 50 00 27 (0x17 0x32 0x00 0x1b). And libdivecomputer has
incorrectly parsed those four bytes as one number, not as the encoded
serial number string it is. So the value 389152795 is actually hex
0x1732001b, which is 0x17 0x32 0x00 0x1b, which is - 23 50 00 27.
This should be done by libdivecomputer, but hey, in the meantime this at
least shows the concept. And helps test the XML save/restore code.
It depends on the two patches that create the whole "device.c"
infrastructure, of course. With this, my dive file ends up having the
settings section look like this:
<divecomputerid model='Suunto Vyper Air' deviceid='d4629110'
serial='01201094' firmware='1.1.22'/>
<divecomputerid model='Suunto HelO2' deviceid='995dd566'
serial='23500027' firmware='1.0.4'/>
where the format of the firmware version is something I guessed at,
but it was the obvious choice (again, it's byte-based, I'm ignoring
the high byte that is zero for both of my Suuntos).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-10 00:14:21 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " firmware.divecomputerid " , utf8_string , & cur_settings . dc . firmware ) )
Assemble the actual Suunto serial number
It turns out that the serial number returned by libdivecomputer isn't
really the serial number as interpreted by the vendor. Those tend to be
strings, but libdivecomputer gives us a 32bit number.
Some experimenting showed that for the Suunto devies tested the serial
number is encoded in that 32bit number:
It so happens that the Suunto serial number strings are strings that have
all numbers, but they aren't *one* number. They are four bytes
representing two numbers each, and the "23500027" string is actually the
four bytes 23 50 00 27 (0x17 0x32 0x00 0x1b). And libdivecomputer has
incorrectly parsed those four bytes as one number, not as the encoded
serial number string it is. So the value 389152795 is actually hex
0x1732001b, which is 0x17 0x32 0x00 0x1b, which is - 23 50 00 27.
This should be done by libdivecomputer, but hey, in the meantime this at
least shows the concept. And helps test the XML save/restore code.
It depends on the two patches that create the whole "device.c"
infrastructure, of course. With this, my dive file ends up having the
settings section look like this:
<divecomputerid model='Suunto Vyper Air' deviceid='d4629110'
serial='01201094' firmware='1.1.22'/>
<divecomputerid model='Suunto HelO2' deviceid='995dd566'
serial='23500027' firmware='1.0.4'/>
where the format of the firmware version is something I guessed at,
but it was the obvious choice (again, it's byte-based, I'm ignoring
the high byte that is zero for both of my Suuntos).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-10 00:14:21 +00:00
return ;
2012-12-26 21:47:54 +00:00
nonmatch ( " divecomputerid " , name , buf ) ;
}
2011-09-23 01:02:54 +00:00
static void try_to_fill_event ( const char * name , char * buf )
{
start_match ( " event " , name , buf ) ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " event " , utf8_string , & cur_event . name ) )
2011-09-23 01:02:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " name " , utf8_string , & cur_event . name ) )
2011-09-23 01:02:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " time " , eventtime , & cur_event . time ) )
2011-09-23 01:02:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " type " , get_index , & cur_event . type ) )
2011-09-23 01:02:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " flags " , get_index , & cur_event . flags ) )
2011-09-23 01:02:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " value " , get_index , & cur_event . value ) )
2011-10-01 04:55:51 +00:00
return ;
2011-09-23 01:02:54 +00:00
nonmatch ( " event " , name , buf ) ;
}
2013-11-02 19:00:16 +00:00
static int match_dc_data_fields ( struct divecomputer * dc , const char * name , char * buf )
2013-01-23 18:25:31 +00:00
{
2013-11-02 19:00:16 +00:00
if ( MATCH ( " maxdepth " , depth , & dc - > maxdepth ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " meandepth " , depth , & dc - > meandepth ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " max.depth " , depth , & dc - > maxdepth ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " mean.depth " , depth , & dc - > meandepth ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " duration " , duration , & dc - > duration ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " divetime " , duration , & dc - > duration ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " divetimesec " , duration , & dc - > duration ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " surfacetime " , duration , & dc - > surfacetime ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " airtemp " , temperature , & dc - > airtemp ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " watertemp " , temperature , & dc - > watertemp ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " air.temperature " , temperature , & dc - > airtemp ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " water.temperature " , temperature , & dc - > watertemp ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " pressure.surface " , pressure , & dc - > surface_pressure ) )
2013-01-23 18:25:31 +00:00
return 1 ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " salinity.water " , salinity , & dc - > salinity ) )
2013-01-23 18:25:31 +00:00
return 1 ;
return 0 ;
}
2012-11-25 02:50:21 +00:00
/* We're in the top-level dive xml. Try to convert whatever value to a dive value */
static void try_to_fill_dc ( struct divecomputer * dc , const char * name , char * buf )
{
start_match ( " divecomputer " , name , buf ) ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " date " , divedate , & dc - > when ) )
2012-11-25 02:50:21 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " time " , divetime , & dc - > when ) )
2012-11-25 02:50:21 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " model " , utf8_string , & dc - > model ) )
2012-11-25 19:44:27 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " deviceid " , hex_value , & dc - > deviceid ) )
2012-11-25 02:50:21 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " diveid " , hex_value , & dc - > diveid ) )
2012-11-25 02:50:21 +00:00
return ;
2014-06-11 17:48:48 +00:00
if ( MATCH ( " dctype " , get_dc_type , & dc - > dctype ) )
return ;
if ( MATCH ( " no_o2sensors " , get_sensor , & dc - > no_o2sensors ) )
return ;
2013-11-02 19:00:16 +00:00
if ( match_dc_data_fields ( dc , name , buf ) )
2013-01-23 18:25:31 +00:00
return ;
2012-11-25 02:50:21 +00:00
nonmatch ( " divecomputer " , name , buf ) ;
}
2011-08-30 22:22:48 +00:00
/* We're in samples - try to convert the random xml value to something useful */
static void try_to_fill_sample ( struct sample * sample , const char * name , char * buf )
{
2012-12-31 02:11:01 +00:00
int in_deco ;
2011-08-30 23:23:47 +00:00
2011-09-01 18:22:05 +00:00
start_match ( " sample " , name , buf ) ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " pressure.sample " , pressure , & sample - > cylinderpressure ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " cylpress.sample " , pressure , & sample - > cylinderpressure ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " cylinderindex.sample " , get_cylinderindex , & sample - > sensor ) )
First step in cleaning up cylinder pressure sensor logic
This clarifies/changes the meaning of our "cylinderindex" entry in our
samples. It has been rather confused, because different dive computers
have done things differently, and the naming really hasn't helped.
There are two totally different - and independent - cylinder "indexes":
- the pressure sensor index, which indicates which cylinder the sensor
data is from.
- the "active cylinder" index, which indicates which cylinder we actually
breathe from.
These two values really are totally independent, and have nothing
what-so-ever to do with each other. The sensor index may well be fixed:
many dive computers only support a single pressure sensor (whether
wireless or wired), and the sensor index is thus always zero.
Other dive computers may support multiple pressure sensors, and the gas
switch event may - or may not - indicate that the sensor changed too. A
dive computer might give the sensor data for *all* cylinders it can read,
regardless of which one is the one we're actively breathing. In fact, some
dive computers might give sensor data for not just *your* cylinder, but
your buddies.
This patch renames "cylinderindex" in the samples as "sensor", making it
quite clear that it's about which sensor index the pressure data in the
sample is about.
The way we figure out which is the currently active gas is with an
explicit has change event. If a computer (like the Uemis Zurich) joins the
two concepts together, then a sensor change should also create a gas
switch event. This patch also changes the Uemis importer to do that.
Finally, it should be noted that the plot info works totally separately
from the sample data, and is about what we actually *display*, not about
the sample pressures etc. In the plot info, the "cylinderindex" does in
fact mean the currently active cylinder, and while it is initially set to
match the sensor information from the samples, we then walk the gas change
events and fix it up - and if the active cylinder differs from the sensor
cylinder, we clear the sensor data.
[Dirk Hohndel: this conflicted with some of my recent changes - I think
I merged things correctly...]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-12-31 04:00:51 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " sensor.sample " , get_sensor , & sample - > sensor ) )
2011-10-19 17:06:11 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " depth.sample " , depth , & sample - > depth ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " temp.sample " , temperature , & sample - > temperature ) )
2011-09-01 23:41:10 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " temperature.sample " , temperature , & sample - > temperature ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " sampletime.sample " , sampletime , & sample - > time ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " time.sample " , sampletime , & sample - > time ) )
2011-08-30 23:23:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " ndl.sample " , sampletime , & sample - > ndl ) )
2012-12-01 21:02:30 +00:00
return ;
2014-07-09 20:13:36 +00:00
if ( MATCH ( " tts.sample " , sampletime , & sample - > tts ) )
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " in_deco.sample " , get_index , & in_deco ) ) {
2012-12-31 02:11:01 +00:00
sample - > in_deco = ( in_deco = = 1 ) ;
return ;
}
2013-11-02 19:00:16 +00:00
if ( MATCH ( " stoptime.sample " , sampletime , & sample - > stoptime ) )
2012-12-01 21:02:30 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " stopdepth.sample " , depth , & sample - > stopdepth ) )
2012-12-01 21:02:30 +00:00
return ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
if ( MATCH ( " cns.sample " , get_uint8 , & sample - > cns ) )
2012-12-11 21:40:07 +00:00
return ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
if ( MATCH ( " po2.sample " , double_to_o2pressure , & sample - > po2 ) )
2012-12-11 21:40:07 +00:00
return ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
if ( MATCH ( " heartbeat " , get_uint8 , & sample - > heartbeat ) )
2014-01-17 22:00:28 +00:00
return ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
if ( MATCH ( " bearing " , get_bearing , & sample - > bearing ) )
2014-01-17 22:00:28 +00:00
return ;
2011-08-30 23:23:47 +00:00
2011-09-05 20:45:14 +00:00
switch ( import_source ) {
2011-09-05 21:29:08 +00:00
case DIVINGLOG :
2013-11-02 19:00:16 +00:00
if ( divinglog_fill_sample ( sample , name , buf ) )
2011-09-05 21:29:08 +00:00
return ;
break ;
2011-09-07 00:33:52 +00:00
case UDDF :
2013-11-02 19:00:16 +00:00
if ( uddf_fill_sample ( sample , name , buf ) )
2011-09-07 00:33:52 +00:00
return ;
break ;
2011-09-05 20:45:14 +00:00
default :
break ;
2011-09-02 18:32:48 +00:00
}
2011-09-01 18:22:05 +00:00
nonmatch ( " sample " , name , buf ) ;
2011-08-30 22:22:48 +00:00
}
2014-04-11 06:17:35 +00:00
void try_to_fill_userid ( const char * name , char * buf )
{
2014-04-17 14:34:21 +00:00
if ( prefs . save_userid_local )
2014-04-11 06:17:35 +00:00
set_userid ( buf ) ;
}
2011-09-05 21:29:08 +00:00
static const char * country , * city ;
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void divinglog_place ( char * place , char * * location )
2011-09-05 21:29:08 +00:00
{
2013-03-04 01:53:43 +00:00
char buffer [ 1024 ] , * p ;
2011-09-05 21:29:08 +00:00
int len ;
len = snprintf ( buffer , sizeof ( buffer ) ,
2014-02-16 23:42:56 +00:00
" %s%s%s%s%s " ,
place ,
city ? " , " : " " ,
city ? city : " " ,
country ? " , " : " " ,
country ? country : " " ) ;
p = malloc ( len + 1 ) ;
memcpy ( p , buffer , len + 1 ) ;
2011-09-05 21:29:08 +00:00
* location = p ;
city = NULL ;
country = NULL ;
}
2013-11-02 19:00:16 +00:00
static int divinglog_dive_match ( struct dive * dive , const char * name , char * buf )
{
2014-02-16 23:42:56 +00:00
return MATCH ( " divedate " , divedate , & dive - > when ) | |
MATCH ( " entrytime " , divetime , & dive - > when ) | |
MATCH ( " divetime " , duration , & dive - > dc . duration ) | |
MATCH ( " depth " , depth , & dive - > dc . maxdepth ) | |
MATCH ( " depthavg " , depth , & dive - > dc . meandepth ) | |
MATCH ( " tanktype " , utf8_string , & dive - > cylinder [ 0 ] . type . description ) | |
MATCH ( " tanksize " , cylindersize , & dive - > cylinder [ 0 ] . type . size ) | |
MATCH ( " presw " , pressure , & dive - > cylinder [ 0 ] . type . workingpressure ) | |
MATCH ( " press " , pressure , & dive - > cylinder [ 0 ] . start ) | |
MATCH ( " prese " , pressure , & dive - > cylinder [ 0 ] . end ) | |
MATCH ( " comments " , utf8_string , & dive - > notes ) | |
MATCH ( " names.buddy " , utf8_string , & dive - > buddy ) | |
MATCH ( " name.country " , utf8_string , & country ) | |
MATCH ( " name.city " , utf8_string , & city ) | |
MATCH ( " name.place " , divinglog_place , & dive - > location ) | |
0 ;
2011-09-05 21:29:08 +00:00
}
2011-09-07 00:33:52 +00:00
/*
* Uddf specifies ISO 8601 time format .
*
* There are many variations on that . This handles the useful cases .
*/
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void uddf_datetime ( char * buffer , timestamp_t * when )
2011-09-07 00:33:52 +00:00
{
char c ;
2014-02-16 23:42:56 +00:00
int y , m , d , hh , mm , ss ;
2011-09-07 00:33:52 +00:00
struct tm tm = { 0 } ;
int i ;
i = sscanf ( buffer , " %d-%d-%d%c%d:%d:%d " , & y , & m , & d , & c , & hh , & mm , & ss ) ;
if ( i = = 7 )
goto success ;
ss = 0 ;
if ( i = = 6 )
goto success ;
i = sscanf ( buffer , " %04d%02d%02d%c%02d%02d%02d " , & y , & m , & d , & c , & hh , & mm , & ss ) ;
if ( i = = 7 )
goto success ;
ss = 0 ;
if ( i = = 6 )
goto success ;
bad_date :
printf ( " Bad date time %s \n " , buffer ) ;
return ;
success :
if ( c ! = ' T ' & & c ! = ' ' )
goto bad_date ;
tm . tm_year = y ;
tm . tm_mon = m - 1 ;
tm . tm_mday = d ;
tm . tm_hour = hh ;
tm . tm_min = mm ;
tm . tm_sec = ss ;
* when = utc_mktime ( & tm ) ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
# define uddf_datedata(name, offset) \
static void uddf_ # # name ( char * buffer , timestamp_t * when ) \
{ \
cur_tm . tm_ # # name = atoi ( buffer ) + offset ; \
* when = utc_mktime ( & cur_tm ) ; \
2014-02-16 23:42:56 +00:00
}
2013-02-22 16:52:35 +00:00
uddf_datedata ( year , 0 )
uddf_datedata ( mon , - 1 )
uddf_datedata ( mday , 0 )
uddf_datedata ( hour , 0 )
uddf_datedata ( min , 0 )
2013-11-02 19:00:16 +00:00
static int uddf_dive_match ( struct dive * dive , const char * name , char * buf )
2011-09-07 00:33:52 +00:00
{
2014-02-16 23:42:56 +00:00
return MATCH ( " datetime " , uddf_datetime , & dive - > when ) | |
MATCH ( " diveduration " , duration , & dive - > dc . duration ) | |
MATCH ( " greatestdepth " , depth , & dive - > dc . maxdepth ) | |
MATCH ( " year.date " , uddf_year , & dive - > when ) | |
MATCH ( " month.date " , uddf_mon , & dive - > when ) | |
MATCH ( " day.date " , uddf_mday , & dive - > when ) | |
MATCH ( " hour.time " , uddf_hour , & dive - > when ) | |
MATCH ( " minute.time " , uddf_min , & dive - > when ) | |
0 ;
2011-09-07 00:33:52 +00:00
}
2012-12-05 17:59:52 +00:00
/*
* This parses " floating point " into micro - degrees .
* We don ' t do exponentials etc , if somebody does
* gps locations in that format , they are insane .
*/
2014-03-09 19:19:41 +00:00
degrees_t parse_degrees ( char * buf , char * * end )
2012-12-05 17:59:52 +00:00
{
int sign = 1 , decimals = 6 , value = 0 ;
degrees_t ret ;
2013-10-05 07:29:09 +00:00
while ( isspace ( * buf ) )
2012-12-05 17:59:52 +00:00
buf + + ;
switch ( * buf ) {
case ' - ' :
sign = - 1 ;
2014-02-16 23:42:56 +00:00
/* fallthrough */
2012-12-05 17:59:52 +00:00
case ' + ' :
buf + + ;
}
while ( isdigit ( * buf ) ) {
2014-02-16 23:42:56 +00:00
value = 10 * value + * buf - ' 0 ' ;
2012-12-05 17:59:52 +00:00
buf + + ;
}
/* Get the first six decimals if they exist */
if ( * buf = = ' . ' )
buf + + ;
do {
value * = 10 ;
if ( isdigit ( * buf ) ) {
value + = * buf - ' 0 ' ;
buf + + ;
}
} while ( - - decimals ) ;
/* Rounding */
switch ( * buf ) {
case ' 5 ' . . . ' 9 ' :
value + + ;
}
while ( isdigit ( * buf ) )
buf + + ;
* end = buf ;
ret . udeg = value * sign ;
return ret ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void gps_lat ( char * buffer , struct dive * dive )
2013-01-23 19:53:42 +00:00
{
char * end ;
dive - > latitude = parse_degrees ( buffer , & end ) ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void gps_long ( char * buffer , struct dive * dive )
2013-01-23 19:53:42 +00:00
{
char * end ;
dive - > longitude = parse_degrees ( buffer , & end ) ;
}
Make parse-xml callbacks be type-safe
.. and fix the type breakage brought in by commit eaf6d564874a ("CCR code:
Change to sample structure")
The XML parsing callbacks pass a "void *" around, because the helper
function that matches the XML node names ("match()") does so for all the
different dive/sample/dc member nodes that all have different types.
But that also hid the fact that it very much depended on the various types
being regular "int" etc, rather than the denser types that were introduced
so that the CCR data wouldn't expand memory use excessively. As a result,
XML loading would overwrite other members, and possibly even the
allocation, when it wrote an "int" value to something that only was a
8-bit allocation.
I left the "utf8_string()" without type checking - so it still uses
"void *_res" for the result type, with the cast happening inside the
function.
That's because the result destination ends up being a bit mixed-up wrt
"const char **" and just plain "char **". Note that the thing we modify
itself isn't const (it's not "char *const *"), but the pointer, but we
basically sometimes assign a "const char *", and sometimes a "char *".
I considered making two different versions of the callback, but it just
wasn't worth it. So "utf8_string()" users still aren't type-checked, and
you'd better give it a pointer to something that is some kind of "char *"
This patch doesn't really change the calling convention of the matching
function itself, but it makes the wrapper macro ("MATCH()") take a
properly type-checked function pointer instead (with a dummy call to do
type checking), and then casts the pointer to the "void *" type for the
actual real call.
The function pointer call is not really portable (although it works on
all sane architectures, particularly since the cast only changes one
argument from one type of pointer to another), and to make matters worse
uses the gcc statement-expression extension. But all the compilers we use
seem to support that gcc'ism, so in practice this gives us type-safety
with no downsides.
(If we ever want to use MSVC to compile subsurface, I suspect we'll have
to ifdef out the statement expression use and not type-check things. Or
perhaps re-write the thing as a ternary expression instead, or something).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-06-07 21:41:07 +00:00
static void gps_location ( char * buffer , struct dive * dive )
2011-09-16 01:16:07 +00:00
{
2012-11-18 17:55:41 +00:00
char * end ;
2011-09-16 01:16:07 +00:00
2012-12-05 17:59:52 +00:00
dive - > latitude = parse_degrees ( buffer , & end ) ;
dive - > longitude = parse_degrees ( end , & end ) ;
2011-09-16 01:16:07 +00:00
}
2014-06-09 00:42:15 +00:00
static void gps_picture_location ( char * buffer , struct picture * pic )
{
char * end ;
pic - > latitude = parse_degrees ( buffer , & end ) ;
pic - > longitude = parse_degrees ( end , & end ) ;
}
2011-08-30 22:22:48 +00:00
/* We're in the top-level dive xml. Try to convert whatever value to a dive value */
2012-11-24 02:05:38 +00:00
static void try_to_fill_dive ( struct dive * dive , const char * name , char * buf )
2011-08-30 22:22:48 +00:00
{
2011-09-01 18:22:05 +00:00
start_match ( " dive " , name , buf ) ;
2011-09-05 21:29:08 +00:00
switch ( import_source ) {
case DIVINGLOG :
2013-11-02 19:00:16 +00:00
if ( divinglog_dive_match ( dive , name , buf ) )
2011-09-05 21:29:08 +00:00
return ;
break ;
2011-09-07 00:33:52 +00:00
case UDDF :
2013-11-02 19:00:16 +00:00
if ( uddf_dive_match ( dive , name , buf ) )
2011-09-07 00:33:52 +00:00
return ;
break ;
2011-09-05 21:29:08 +00:00
default :
break ;
}
2013-11-02 19:00:16 +00:00
if ( MATCH ( " number " , get_index , & dive - > number ) )
2011-09-11 18:36:33 +00:00
return ;
Get rid of crazy empty tag_list element at the start
So this is totally unrelated to the git repository format, except for
the fact that I noticed it while writing the git saving code.
The subsurface divetag list handling is being stupid, and has a
initial dummy entry at the head of the list for no good reason.
I say "no good reason", because there *is* a reason for it: it allows
code to avoid the special case of empty list and adding entries to
before the first entry etc etc. But that reason is a really *bad*
reason, because it's valid only because people don't understand basic
list manipulation and pointers to pointers.
So get rid of the dummy element, and do things right instead - by
passing a *pointer* to the list, instead of the list. And then when
traversing the list and looking for a place to insert things, don't go
to the next entry - just update the "pointer to pointer" to point to
the address of the next entry. Each entry in a C linked list is no
different than the list itself, so you can use the pointer to the
pointer to the next entry as a pointer to the list.
This is a pet peeve of mine. The real beauty of pointers can never be
understood unless you understand the indirection they allow. People
who grew up with Pascal and were corrupted by that mindset are
mentally stunted. Niklaus Wirth has a lot to answer for!
But never fear. You too can overcome that mental limitation, it just
needs some brain exercise. Reading this patch may help. In particular,
contemplate the new "taglist_add_divetag()".
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-03-10 17:18:13 +00:00
if ( MATCH ( " tags " , divetags , & dive - > tag_list ) )
2013-04-09 15:54:36 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " tripflag " , get_tripflag , & dive - > tripflag ) )
2012-08-22 05:04:24 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " date " , divedate , & dive - > when ) )
2011-08-30 22:22:48 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " time " , divetime , & dive - > when ) )
2011-08-30 22:22:48 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " datetime " , divedatetime , & dive - > when ) )
2011-08-30 23:59:03 +00:00
return ;
2013-01-23 18:25:31 +00:00
/*
* Legacy format note : per - dive depths and duration get saved
* in the first dive computer entry
*/
2013-11-02 19:00:16 +00:00
if ( match_dc_data_fields ( & dive - > dc , name , buf ) )
2012-11-12 19:57:49 +00:00
return ;
2013-01-23 18:25:31 +00:00
2014-06-09 00:42:15 +00:00
if ( MATCH ( " filename.picture " , utf8_string , & cur_picture - > filename ) )
return ;
2014-07-08 19:29:06 +00:00
if ( MATCH ( " offset.picture " , offsettime , & cur_picture - > offset ) )
2014-06-09 00:42:15 +00:00
return ;
if ( MATCH ( " gps.picture " , gps_picture_location , cur_picture ) )
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " cylinderstartpressure " , pressure , & dive - > cylinder [ 0 ] . start ) )
2011-08-31 00:45:03 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " cylinderendpressure " , pressure , & dive - > cylinder [ 0 ] . end ) )
2011-08-31 00:45:03 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " gps " , gps_location , dive ) )
2011-09-16 01:16:07 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " Place " , gps_location , dive ) )
2013-02-25 07:20:58 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " latitude " , gps_lat , dive ) )
2013-01-23 19:53:42 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " sitelat " , gps_lat , dive ) )
2013-01-29 05:11:01 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " lat " , gps_lat , dive ) )
2013-02-25 07:20:58 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " longitude " , gps_long , dive ) )
2013-01-23 19:53:42 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " sitelon " , gps_long , dive ) )
2013-01-29 05:11:01 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " lon " , gps_long , dive ) )
2013-02-25 07:20:58 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " location " , utf8_string , & dive - > location ) )
2011-09-02 02:56:04 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " name.dive " , utf8_string , & dive - > location ) )
2013-01-23 19:53:42 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " suit " , utf8_string , & dive - > suit ) )
2012-08-14 23:07:25 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " divesuit " , utf8_string , & dive - > suit ) )
2012-08-18 03:22:37 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " notes " , utf8_string , & dive - > notes ) )
2011-09-02 02:56:04 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " divemaster " , utf8_string , & dive - > divemaster ) )
2011-09-13 21:58:06 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " buddy " , utf8_string , & dive - > buddy ) )
2011-09-13 21:58:06 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " rating.dive " , get_rating , & dive - > rating ) )
2011-12-07 19:58:16 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " visibility.dive " , get_rating , & dive - > visibility ) )
2012-10-28 22:49:02 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " size.cylinder " , cylindersize , & dive - > cylinder [ cur_cylinder_index ] . type . size ) )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " workpressure.cylinder " , pressure , & dive - > cylinder [ cur_cylinder_index ] . type . workingpressure ) )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " description.cylinder " , utf8_string , & dive - > cylinder [ cur_cylinder_index ] . type . description ) )
2011-09-04 22:07:47 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " start.cylinder " , pressure , & dive - > cylinder [ cur_cylinder_index ] . start ) )
2011-09-05 16:12:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " end.cylinder " , pressure , & dive - > cylinder [ cur_cylinder_index ] . end ) )
2011-09-05 16:12:54 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " description.weightsystem " , utf8_string , & dive - > weightsystem [ cur_ws_index ] . description ) )
2011-12-24 03:41:16 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " weight.weightsystem " , weight , & dive - > weightsystem [ cur_ws_index ] . weight ) )
2011-12-24 03:41:16 +00:00
return ;
2012-03-24 04:07:53 +00:00
if ( MATCH ( " weight " , weight , & dive - > weightsystem [ cur_ws_index ] . weight ) )
2011-12-24 03:41:16 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " o2 " , gasmix , & dive - > cylinder [ cur_cylinder_index ] . gasmix . o2 ) )
2011-09-04 03:31:18 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " o2percent " , gasmix , & dive - > cylinder [ cur_cylinder_index ] . gasmix . o2 ) )
2013-01-31 21:54:14 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " n2 " , gasmix_nitrogen , & dive - > cylinder [ cur_cylinder_index ] . gasmix ) )
2011-09-04 03:31:18 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " he " , gasmix , & dive - > cylinder [ cur_cylinder_index ] . gasmix . he ) )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " air.divetemperature " , temperature , & dive - > airtemp ) )
2013-02-14 17:44:18 +00:00
return ;
2014-06-03 23:01:48 +00:00
if ( MATCH ( " water.divetemperature " , temperature , & dive - > watertemp ) )
return ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
2011-09-01 18:22:05 +00:00
nonmatch ( " dive " , name , buf ) ;
2011-08-30 22:22:48 +00:00
}
2011-08-30 20:48:05 +00:00
2012-08-22 05:04:24 +00:00
/* We're in the top-level trip xml. Try to convert whatever value to a trip value */
2012-09-20 03:42:11 +00:00
static void try_to_fill_trip ( dive_trip_t * * dive_trip_p , const char * name , char * buf )
2012-08-22 05:04:24 +00:00
{
start_match ( " trip " , name , buf ) ;
2012-09-20 03:42:11 +00:00
dive_trip_t * dive_trip = * dive_trip_p ;
2012-08-22 05:04:24 +00:00
2013-11-02 19:00:16 +00:00
if ( MATCH ( " date " , divedate , & dive_trip - > when ) )
2012-08-30 00:24:15 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " time " , divetime , & dive_trip - > when ) )
2012-08-22 05:04:24 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " location " , utf8_string , & dive_trip - > location ) )
2012-08-22 05:04:24 +00:00
return ;
2013-11-02 19:00:16 +00:00
if ( MATCH ( " notes " , utf8_string , & dive_trip - > notes ) )
2012-08-22 05:04:24 +00:00
return ;
nonmatch ( " trip " , name , buf ) ;
}
2011-08-30 04:32:27 +00:00
/*
2012-09-16 01:32:15 +00:00
* While in some formats file boundaries are dive boundaries , in many
* others ( as for example in our native format ) there are
2011-08-30 04:32:27 +00:00
* multiple dives per file , so there can be other events too that
* trigger a " new dive " marker and you may get some nesting due
* to that . Just ignore nesting levels .
2012-09-16 01:32:15 +00:00
* On the flipside it is possible that we start an XML file that ends
* up having no dives in it at all - don ' t create a bogus empty dive
* for those . It ' s not entirely clear what is the minimum set of data
* to make a dive valid , but if it has no location , no date and no
* samples I ' m pretty sure it ' s useless .
2011-08-30 04:32:27 +00:00
*/
2013-10-05 07:29:09 +00:00
static bool is_dive ( void )
2012-09-16 01:32:15 +00:00
{
return ( cur_dive & &
2012-11-24 02:51:27 +00:00
( cur_dive - > location | | cur_dive - > when | | cur_dive - > dc . samples ) ) ;
2012-09-16 01:32:15 +00:00
}
2012-12-19 20:36:56 +00:00
static void reset_dc_info ( struct divecomputer * dc )
{
2012-12-31 02:11:01 +00:00
lastcns = lastpo2 = lastndl = laststoptime = laststopdepth = lastindeco = 0 ;
First step in cleaning up cylinder pressure sensor logic
This clarifies/changes the meaning of our "cylinderindex" entry in our
samples. It has been rather confused, because different dive computers
have done things differently, and the naming really hasn't helped.
There are two totally different - and independent - cylinder "indexes":
- the pressure sensor index, which indicates which cylinder the sensor
data is from.
- the "active cylinder" index, which indicates which cylinder we actually
breathe from.
These two values really are totally independent, and have nothing
what-so-ever to do with each other. The sensor index may well be fixed:
many dive computers only support a single pressure sensor (whether
wireless or wired), and the sensor index is thus always zero.
Other dive computers may support multiple pressure sensors, and the gas
switch event may - or may not - indicate that the sensor changed too. A
dive computer might give the sensor data for *all* cylinders it can read,
regardless of which one is the one we're actively breathing. In fact, some
dive computers might give sensor data for not just *your* cylinder, but
your buddies.
This patch renames "cylinderindex" in the samples as "sensor", making it
quite clear that it's about which sensor index the pressure data in the
sample is about.
The way we figure out which is the currently active gas is with an
explicit has change event. If a computer (like the Uemis Zurich) joins the
two concepts together, then a sensor change should also create a gas
switch event. This patch also changes the Uemis importer to do that.
Finally, it should be noted that the plot info works totally separately
from the sample data, and is about what we actually *display*, not about
the sample pressures etc. In the plot info, the "cylinderindex" does in
fact mean the currently active cylinder, and while it is initially set to
match the sensor information from the samples, we then walk the gas change
events and fix it up - and if the active cylinder differs from the sensor
cylinder, we clear the sensor data.
[Dirk Hohndel: this conflicted with some of my recent changes - I think
I merged things correctly...]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-12-31 04:00:51 +00:00
lastsensor = lastcylinderindex = 0 ;
2012-12-19 20:36:56 +00:00
}
2012-12-26 21:47:54 +00:00
static void reset_dc_settings ( void )
{
free ( ( void * ) cur_settings . dc . model ) ;
free ( ( void * ) cur_settings . dc . nickname ) ;
Assemble the actual Suunto serial number
It turns out that the serial number returned by libdivecomputer isn't
really the serial number as interpreted by the vendor. Those tend to be
strings, but libdivecomputer gives us a 32bit number.
Some experimenting showed that for the Suunto devies tested the serial
number is encoded in that 32bit number:
It so happens that the Suunto serial number strings are strings that have
all numbers, but they aren't *one* number. They are four bytes
representing two numbers each, and the "23500027" string is actually the
four bytes 23 50 00 27 (0x17 0x32 0x00 0x1b). And libdivecomputer has
incorrectly parsed those four bytes as one number, not as the encoded
serial number string it is. So the value 389152795 is actually hex
0x1732001b, which is 0x17 0x32 0x00 0x1b, which is - 23 50 00 27.
This should be done by libdivecomputer, but hey, in the meantime this at
least shows the concept. And helps test the XML save/restore code.
It depends on the two patches that create the whole "device.c"
infrastructure, of course. With this, my dive file ends up having the
settings section look like this:
<divecomputerid model='Suunto Vyper Air' deviceid='d4629110'
serial='01201094' firmware='1.1.22'/>
<divecomputerid model='Suunto HelO2' deviceid='995dd566'
serial='23500027' firmware='1.0.4'/>
where the format of the firmware version is something I guessed at,
but it was the obvious choice (again, it's byte-based, I'm ignoring
the high byte that is zero for both of my Suuntos).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-10 00:14:21 +00:00
free ( ( void * ) cur_settings . dc . serial_nr ) ;
free ( ( void * ) cur_settings . dc . firmware ) ;
2012-12-26 21:47:54 +00:00
cur_settings . dc . model = NULL ;
cur_settings . dc . nickname = NULL ;
Assemble the actual Suunto serial number
It turns out that the serial number returned by libdivecomputer isn't
really the serial number as interpreted by the vendor. Those tend to be
strings, but libdivecomputer gives us a 32bit number.
Some experimenting showed that for the Suunto devies tested the serial
number is encoded in that 32bit number:
It so happens that the Suunto serial number strings are strings that have
all numbers, but they aren't *one* number. They are four bytes
representing two numbers each, and the "23500027" string is actually the
four bytes 23 50 00 27 (0x17 0x32 0x00 0x1b). And libdivecomputer has
incorrectly parsed those four bytes as one number, not as the encoded
serial number string it is. So the value 389152795 is actually hex
0x1732001b, which is 0x17 0x32 0x00 0x1b, which is - 23 50 00 27.
This should be done by libdivecomputer, but hey, in the meantime this at
least shows the concept. And helps test the XML save/restore code.
It depends on the two patches that create the whole "device.c"
infrastructure, of course. With this, my dive file ends up having the
settings section look like this:
<divecomputerid model='Suunto Vyper Air' deviceid='d4629110'
serial='01201094' firmware='1.1.22'/>
<divecomputerid model='Suunto HelO2' deviceid='995dd566'
serial='23500027' firmware='1.0.4'/>
where the format of the firmware version is something I guessed at,
but it was the obvious choice (again, it's byte-based, I'm ignoring
the high byte that is zero for both of my Suuntos).
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-10 00:14:21 +00:00
cur_settings . dc . serial_nr = NULL ;
cur_settings . dc . firmware = NULL ;
2012-12-26 21:47:54 +00:00
cur_settings . dc . deviceid = 0 ;
}
2013-01-02 01:29:38 +00:00
static void settings_start ( void )
2012-12-26 21:47:54 +00:00
{
2014-01-15 18:54:41 +00:00
in_settings = true ;
2013-01-02 01:29:38 +00:00
}
static void settings_end ( void )
{
2014-01-15 18:54:41 +00:00
in_settings = false ;
2013-01-02 01:29:38 +00:00
}
static void dc_settings_start ( void )
{
2012-12-26 21:47:54 +00:00
reset_dc_settings ( ) ;
}
static void dc_settings_end ( void )
{
2013-06-17 22:58:26 +00:00
create_device_node ( cur_settings . dc . model , cur_settings . dc . deviceid , cur_settings . dc . serial_nr ,
cur_settings . dc . firmware , cur_settings . dc . nickname ) ;
2012-12-26 21:47:54 +00:00
reset_dc_settings ( ) ;
}
2011-08-30 04:32:27 +00:00
static void dive_start ( void )
{
2012-01-05 16:16:08 +00:00
if ( cur_dive )
2011-08-30 22:22:48 +00:00
return ;
2012-01-05 16:16:08 +00:00
cur_dive = alloc_dive ( ) ;
2012-12-19 20:36:56 +00:00
reset_dc_info ( & cur_dive - > dc ) ;
2012-01-05 16:16:08 +00:00
memset ( & cur_tm , 0 , sizeof ( cur_tm ) ) ;
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
if ( cur_trip ) {
2012-11-10 18:51:03 +00:00
add_dive_to_trip ( cur_dive , cur_trip ) ;
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
cur_dive - > tripflag = IN_TRIP ;
}
2011-08-30 04:32:27 +00:00
}
static void dive_end ( void )
{
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
if ( ! cur_dive )
2011-08-30 20:48:05 +00:00
return ;
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
if ( ! is_dive ( ) )
free ( cur_dive ) ;
else
2013-01-31 00:08:36 +00:00
record_dive_to_table ( cur_dive , target_table ) ;
2012-01-05 16:16:08 +00:00
cur_dive = NULL ;
2012-11-25 02:50:21 +00:00
cur_dc = NULL ;
2012-01-05 16:16:08 +00:00
cur_cylinder_index = 0 ;
2012-03-24 04:07:53 +00:00
cur_ws_index = 0 ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
}
2012-08-22 05:04:24 +00:00
static void trip_start ( void )
{
if ( cur_trip )
return ;
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
dive_end ( ) ;
2014-02-10 15:04:37 +00:00
cur_trip = calloc ( 1 , sizeof ( dive_trip_t ) ) ;
2012-08-22 05:04:24 +00:00
memset ( & cur_tm , 0 , sizeof ( cur_tm ) ) ;
}
static void trip_end ( void )
{
if ( ! cur_trip )
return ;
2012-09-05 20:54:22 +00:00
insert_trip ( & cur_trip ) ;
2012-08-22 05:04:24 +00:00
cur_trip = NULL ;
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
static void event_start ( void )
{
2012-01-05 16:16:08 +00:00
memset ( & cur_event , 0 , sizeof ( cur_event ) ) ;
cur_event . active = 1 ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
}
static void event_end ( void )
{
2012-11-25 02:50:21 +00:00
struct divecomputer * dc = get_dc ( ) ;
2012-12-24 01:51:39 +00:00
if ( cur_event . name ) {
2014-06-09 16:21:26 +00:00
if ( strcmp ( cur_event . name , " surface " ) ! = 0 ) {
/* 123 is a magic event that we used for a while to encode images in dives */
if ( cur_event . type = = 123 ) {
struct picture * pic = alloc_picture ( ) ;
pic - > filename = strdup ( cur_event . name ) ;
2014-07-08 19:29:06 +00:00
/* theoretically this could fail - but we didn't support multi year offsets */
pic - > offset . seconds = cur_event . time . seconds ;
2014-06-09 16:21:26 +00:00
dive_add_picture ( cur_dive , pic ) ;
} else {
2014-07-12 10:12:24 +00:00
/* At some point gas change events did not have any type. Thus we need to add
* one on import , if we encounter the type one missing .
*/
if ( cur_event . type = = 0 & & strcmp ( cur_event . name , " gaschange " ) = = 0 )
2014-07-12 12:51:03 +00:00
cur_event . type = cur_event . value > > 16 > 0 ? SAMPLE_EVENT_GASCHANGE2 : SAMPLE_EVENT_GASCHANGE ;
2014-07-12 10:12:24 +00:00
2014-06-09 16:21:26 +00:00
add_event ( dc , cur_event . time . seconds ,
cur_event . type , cur_event . flags ,
cur_event . value , cur_event . name ) ;
}
}
2012-12-24 01:51:39 +00:00
free ( ( void * ) cur_event . name ) ;
}
2012-01-05 16:16:08 +00:00
cur_event . active = 0 ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
}
2014-06-09 00:42:15 +00:00
static void picture_start ( void )
{
cur_picture = alloc_picture ( ) ;
}
static void picture_end ( void )
{
dive_add_picture ( cur_dive , cur_picture ) ;
cur_picture = NULL ;
}
2011-09-04 03:31:18 +00:00
static void cylinder_start ( void )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
{
}
2011-09-04 03:31:18 +00:00
static void cylinder_end ( void )
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
{
2012-01-05 16:16:08 +00:00
cur_cylinder_index + + ;
2011-08-30 04:32:27 +00:00
}
2011-12-24 03:41:16 +00:00
static void ws_start ( void )
{
}
static void ws_end ( void )
{
2012-03-24 04:07:53 +00:00
cur_ws_index + + ;
2011-12-24 03:41:16 +00:00
}
2011-08-30 04:32:27 +00:00
static void sample_start ( void )
{
2012-11-25 02:50:21 +00:00
cur_sample = prepare_sample ( get_dc ( ) ) ;
2012-12-01 21:02:30 +00:00
cur_sample - > ndl . seconds = lastndl ;
2012-12-31 02:11:01 +00:00
cur_sample - > in_deco = lastindeco ;
2012-12-01 21:02:30 +00:00
cur_sample - > stoptime . seconds = laststoptime ;
cur_sample - > stopdepth . mm = laststopdepth ;
2012-12-08 04:08:29 +00:00
cur_sample - > cns = lastcns ;
2014-06-03 17:21:41 +00:00
cur_sample - > po2 . mbar = lastpo2 ;
First step in cleaning up cylinder pressure sensor logic
This clarifies/changes the meaning of our "cylinderindex" entry in our
samples. It has been rather confused, because different dive computers
have done things differently, and the naming really hasn't helped.
There are two totally different - and independent - cylinder "indexes":
- the pressure sensor index, which indicates which cylinder the sensor
data is from.
- the "active cylinder" index, which indicates which cylinder we actually
breathe from.
These two values really are totally independent, and have nothing
what-so-ever to do with each other. The sensor index may well be fixed:
many dive computers only support a single pressure sensor (whether
wireless or wired), and the sensor index is thus always zero.
Other dive computers may support multiple pressure sensors, and the gas
switch event may - or may not - indicate that the sensor changed too. A
dive computer might give the sensor data for *all* cylinders it can read,
regardless of which one is the one we're actively breathing. In fact, some
dive computers might give sensor data for not just *your* cylinder, but
your buddies.
This patch renames "cylinderindex" in the samples as "sensor", making it
quite clear that it's about which sensor index the pressure data in the
sample is about.
The way we figure out which is the currently active gas is with an
explicit has change event. If a computer (like the Uemis Zurich) joins the
two concepts together, then a sensor change should also create a gas
switch event. This patch also changes the Uemis importer to do that.
Finally, it should be noted that the plot info works totally separately
from the sample data, and is about what we actually *display*, not about
the sample pressures etc. In the plot info, the "cylinderindex" does in
fact mean the currently active cylinder, and while it is initially set to
match the sensor information from the samples, we then walk the gas change
events and fix it up - and if the active cylinder differs from the sensor
cylinder, we clear the sensor data.
[Dirk Hohndel: this conflicted with some of my recent changes - I think
I merged things correctly...]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-12-31 04:00:51 +00:00
cur_sample - > sensor = lastsensor ;
2011-08-30 04:32:27 +00:00
}
static void sample_end ( void )
{
2012-01-05 16:16:08 +00:00
if ( ! cur_dive )
2011-08-30 20:48:05 +00:00
return ;
2011-08-31 21:36:53 +00:00
2012-11-25 02:50:21 +00:00
finish_sample ( get_dc ( ) ) ;
2012-12-01 21:02:30 +00:00
lastndl = cur_sample - > ndl . seconds ;
2012-12-31 02:11:01 +00:00
lastindeco = cur_sample - > in_deco ;
2012-12-01 21:02:30 +00:00
laststoptime = cur_sample - > stoptime . seconds ;
laststopdepth = cur_sample - > stopdepth . mm ;
2012-12-08 04:08:29 +00:00
lastcns = cur_sample - > cns ;
2014-06-03 17:21:41 +00:00
lastpo2 = cur_sample - > po2 . mbar ;
2012-01-05 16:16:08 +00:00
cur_sample = NULL ;
2011-08-30 04:32:27 +00:00
}
2012-11-25 02:50:21 +00:00
static void divecomputer_start ( void )
{
struct divecomputer * dc ;
/* Start from the previous dive computer */
dc = & cur_dive - > dc ;
while ( dc - > next )
dc = dc - > next ;
/* Did we already fill that in? */
2012-11-25 19:44:27 +00:00
if ( dc - > samples | | dc - > model | | dc - > when ) {
2012-11-25 02:50:21 +00:00
struct divecomputer * newdc = calloc ( 1 , sizeof ( * newdc ) ) ;
if ( newdc ) {
dc - > next = newdc ;
dc = newdc ;
}
}
/* .. this is the one we'll use */
cur_dc = dc ;
2012-12-19 20:36:56 +00:00
reset_dc_info ( dc ) ;
2012-11-25 02:50:21 +00:00
}
static void divecomputer_end ( void )
{
if ( ! cur_dc - > when )
cur_dc - > when = cur_dive - > when ;
cur_dc = NULL ;
}
2014-04-11 06:17:35 +00:00
static void userid_start ( void )
{
in_userid = true ;
set_save_userid_local ( true ) ; //if the xml contains userid, keep saving it.
}
static void userid_stop ( void )
{
in_userid = false ;
}
2012-12-28 16:18:23 +00:00
static void entry ( const char * name , char * buf )
2011-08-30 04:32:27 +00:00
{
2014-04-11 06:17:35 +00:00
if ( in_userid ) {
try_to_fill_userid ( name , buf ) ;
return ;
}
2012-12-26 21:47:54 +00:00
if ( in_settings ) {
try_to_fill_dc_settings ( name , buf ) ;
2013-01-02 01:29:38 +00:00
try_to_match_autogroup ( name , buf ) ;
2012-12-26 21:47:54 +00:00
return ;
}
2012-01-05 16:16:08 +00:00
if ( cur_event . active ) {
2011-09-23 01:02:54 +00:00
try_to_fill_event ( name , buf ) ;
return ;
}
2012-01-05 16:16:08 +00:00
if ( cur_sample ) {
try_to_fill_sample ( cur_sample , name , buf ) ;
2011-08-30 20:48:05 +00:00
return ;
}
2012-11-25 02:50:21 +00:00
if ( cur_dc ) {
try_to_fill_dc ( cur_dc , name , buf ) ;
return ;
}
2012-01-05 16:16:08 +00:00
if ( cur_dive ) {
2012-11-24 02:05:38 +00:00
try_to_fill_dive ( cur_dive , name , buf ) ;
2011-08-30 20:48:05 +00:00
return ;
}
New XML format for saving dives
This patch makes the trips nest, and it also fixes the fact that you never
saved the trip notes (you could edit it, but saving would throw it away).
I did *not* change the indentation of the dives, so the trip stuff shows
up the the beginning of the line, at the same level as the <dive> and
<dives> thing. I think it's fairly readable xml, though, and we haven't
really had proper "indentation shows nesting" anyway, since the top-level
"<dives>" thing also didn't indent stuff inside of it.
Anyway, the way I wrote it, it still parses your old "INTRIP" stuff etc,
so as far as I know, it should happily read the old-style XML too. At
least it seemed to work with your xml file that already had the old-style
one (I haven't committed my divetrips, exactly because I didn't like the
new format).
It always saves in the new style, though.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2012-09-30 19:36:18 +00:00
if ( cur_trip ) {
try_to_fill_trip ( & cur_trip , name , buf ) ;
return ;
}
2011-08-30 04:32:27 +00:00
}
2011-08-30 00:51:54 +00:00
static const char * nodename ( xmlNode * node , char * buf , int len )
2011-08-28 23:58:26 +00:00
{
2013-11-02 19:00:16 +00:00
int levels = 2 ;
char * p = buf ;
2011-08-30 21:38:39 +00:00
if ( ! node | | ! node - > name )
return " root " ;
2011-08-30 00:51:54 +00:00
2013-11-02 19:00:16 +00:00
if ( node - > parent & & ! strcmp ( node - > name , " text " ) )
node = node - > parent ;
/* Make sure it's always NUL-terminated */
p [ - - len ] = 0 ;
2011-08-30 00:51:54 +00:00
2014-02-16 23:42:56 +00:00
for ( ; ; ) {
2011-08-30 00:51:54 +00:00
const char * name = node - > name ;
2013-11-02 19:00:16 +00:00
char c ;
while ( ( c = * name + + ) ! = 0 ) {
/* Cheaper 'tolower()' for ASCII */
c = ( c > = ' A ' & & c < = ' Z ' ) ? c - ' A ' + ' a ' : c ;
* p + + = c ;
2011-08-30 00:51:54 +00:00
if ( ! - - len )
return buf ;
}
2013-11-02 19:00:16 +00:00
* p = 0 ;
2011-08-30 00:51:54 +00:00
node = node - > parent ;
if ( ! node | | ! node - > name )
return buf ;
2013-11-02 19:00:16 +00:00
* p + + = ' . ' ;
2011-08-30 00:51:54 +00:00
if ( ! - - len )
return buf ;
2013-11-02 19:00:16 +00:00
if ( ! - - levels )
return buf ;
2011-08-30 00:51:54 +00:00
}
}
2013-11-02 19:00:16 +00:00
# define MAXNAME 32
2011-08-30 00:51:54 +00:00
2011-08-30 04:32:27 +00:00
static void visit_one_node ( xmlNode * node )
2011-08-30 00:51:54 +00:00
{
2012-12-28 16:18:23 +00:00
char * content ;
2013-11-28 20:13:02 +00:00
static char buffer [ MAXNAME ] ;
2011-08-30 00:51:54 +00:00
const char * name ;
content = node - > content ;
2012-12-28 16:18:23 +00:00
if ( ! content | | xmlIsBlankNode ( node ) )
2011-08-30 00:51:54 +00:00
return ;
name = nodename ( node , buffer , sizeof ( buffer ) ) ;
2011-08-28 23:58:26 +00:00
2012-12-28 16:18:23 +00:00
entry ( name , content ) ;
2011-08-28 23:58:26 +00:00
}
2011-09-01 18:22:05 +00:00
static void traverse ( xmlNode * root ) ;
static void traverse_properties ( xmlNode * node )
{
xmlAttr * p ;
for ( p = node - > properties ; p ; p = p - > next )
traverse ( p - > children ) ;
}
static void visit ( xmlNode * n )
{
visit_one_node ( n ) ;
traverse_properties ( n ) ;
traverse ( n - > children ) ;
}
2011-09-05 21:29:08 +00:00
static void DivingLog_importer ( void )
{
import_source = DIVINGLOG ;
/*
* Diving Log units are really strange .
*
* Temperatures are in C , except in samples ,
* when they are in Fahrenheit . Depths are in
2011-09-11 19:24:57 +00:00
* meters , an dpressure is in PSI in the samples ,
* but in bar when it comes to working pressure .
*
* Crazy f * % ^ morons .
2011-09-05 21:29:08 +00:00
*/
2013-01-11 01:26:10 +00:00
xml_parsing_units = SI_units ;
2011-09-05 21:29:08 +00:00
}
2011-09-07 00:01:28 +00:00
static void uddf_importer ( void )
{
import_source = UDDF ;
2013-01-11 01:26:10 +00:00
xml_parsing_units = SI_units ;
xml_parsing_units . pressure = PASCAL ;
xml_parsing_units . temperature = KELVIN ;
2011-09-07 00:01:28 +00:00
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
/*
* I ' m sure this could be done as some fancy DTD rules .
* It ' s just not worth the headache .
*/
static struct nesting {
const char * name ;
void ( * start ) ( void ) , ( * end ) ( void ) ;
} nesting [ ] = {
2014-02-16 23:42:56 +00:00
{ " divecomputerid " , dc_settings_start , dc_settings_end } ,
{ " settings " , settings_start , settings_end } ,
{ " dive " , dive_start , dive_end } ,
{ " Dive " , dive_start , dive_end } ,
{ " trip " , trip_start , trip_end } ,
{ " sample " , sample_start , sample_end } ,
{ " waypoint " , sample_start , sample_end } ,
{ " SAMPLE " , sample_start , sample_end } ,
{ " reading " , sample_start , sample_end } ,
{ " event " , event_start , event_end } ,
{ " mix " , cylinder_start , cylinder_end } ,
{ " gasmix " , cylinder_start , cylinder_end } ,
{ " cylinder " , cylinder_start , cylinder_end } ,
{ " weightsystem " , ws_start , ws_end } ,
{ " divecomputer " , divecomputer_start , divecomputer_end } ,
{ " P " , sample_start , sample_end } ,
2014-04-11 06:17:35 +00:00
{ " userid " , userid_start , userid_stop } ,
2014-06-09 00:42:15 +00:00
{ " picture " , picture_start , picture_end } ,
2014-02-16 23:42:56 +00:00
/* Import type recognition */
{ " Divinglog " , DivingLog_importer } ,
{ " uddf " , uddf_importer } ,
{ NULL , }
} ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
2011-09-01 18:22:05 +00:00
static void traverse ( xmlNode * root )
2011-08-28 23:58:26 +00:00
{
xmlNode * n ;
2011-09-01 18:22:05 +00:00
for ( n = root ; n ; n = n - > next ) {
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
struct nesting * rule = nesting ;
2011-08-30 04:32:27 +00:00
parse-xml: allow XML nodes with empty tag names
They happen for CDATA content, where libxml2 turns the CDATA fields into
a child of the parent entry, but without a name.
Now, of course, any sane person would just want to use the CDATA as the
string value of the parent itself, but libxml2 probably does this
insanity for a reason. And the reason is probably that some misguided
people want to *write* XML using libxml2, and then the stupid child node
actually acts as a "now I want you to write this data as CDATA".
Whatever the reason, let's just ignore it. We will just traverse such a
nameless child and be happy, and we'll give the nameless child the name
of the parent. Our XML node matching logic will then never see this
insane nameless child at all, and doesn't have to care.
Our whole XML parsing rule-of-thumb is to take the whole "be strict in
what you output, but generous in what you accept" to its logical
conclusion. Because we will literally accept almost anything, in any
format. You can mix tags or attributes wildly, and youc an use CDATA or
not as you see fit. We just don't care.
We're the honeybadger of the divelog world.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2013-01-29 05:12:32 +00:00
if ( ! n - > name ) {
visit ( n ) ;
continue ;
}
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
do {
if ( ! strcmp ( rule - > name , n - > name ) )
break ;
rule + + ;
} while ( rule - > name ) ;
2011-08-30 04:32:27 +00:00
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
if ( rule - > start )
rule - > start ( ) ;
2011-09-01 18:22:05 +00:00
visit ( n ) ;
Start parsing gas mixes
The suunto xml is just completely crazy. What's the helium percentage
companion to "o2pct"? Would it be "hepct"? No. It's "hepct_0".
Ok, so they didn't number the first o2pct, which could be seen as sane:
that's the only mix value that should always exist. And they clearly
started their indexing with 0. So with multiple mixes, you'd then
expect "o2pct_1" and "hepct_1", right?
Wrong! Because XML people are crazy, the second O2 mix percentage is
obviously "o2pct_2". So the O2 percentages are one-based, with an
implicit one. But the He percentages are zero-based with an explicit
zero. So the second mix is "o2pct_2" and "hepct_1".
I'd like to ask what drugs Suunto people are on, but hey, it's a Finnish
company. No need to ask. Vodka explains everything. LOTS AND LOTS OF
VODKA.
In comparison, the libdivecomputer output is nice and sane, and uses a
'gasmix' node. Of course, now we have so many different XML nesting
nodes to check that I just made it an array of different noces. That
also allows me to mark the suunto case, so that we only do the "check
for crazy alcoholic xml entries" when it's a suunto file.
The "type of file" thing is probably a good idea for deciding on default
units too. Some day.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2011-09-01 20:32:52 +00:00
if ( rule - > end )
rule - > end ( ) ;
2011-08-28 23:58:26 +00:00
}
}
2011-09-02 18:32:48 +00:00
/* Per-file reset */
static void reset_all ( void )
{
/*
* We reset the units for each file . You ' d think it was
* a per - dive property , but I ' m not going to trust people
* to do per - dive setup . If the xml does have per - dive
* data within one file , we might have to reset it per
* dive for that format .
*/
2013-01-11 01:26:10 +00:00
xml_parsing_units = SI_units ;
2011-09-05 20:45:14 +00:00
import_source = UNKNOWN ;
2011-09-02 18:32:48 +00:00
}
2013-03-15 17:02:14 +00:00
/* divelog.de sends us xml files that claim to be iso-8859-1
* but once we decode the HTML encoded characters they turn
* into UTF - 8 instead . So skip the incorrect encoding
* declaration and decode the HTML encoded characters */
const char * preprocess_divelog_de ( const char * buffer )
{
char * ret = strstr ( buffer , " <DIVELOGSDATA> " ) ;
if ( ret ) {
xmlParserCtxtPtr ctx ;
char buf [ ] = " " ;
2013-03-26 19:59:22 +00:00
int i ;
for ( i = 0 ; i < strlen ( ret ) ; + + i )
if ( ! isascii ( ret [ i ] ) )
return buffer ;
2013-03-15 17:02:14 +00:00
ctx = xmlCreateMemoryParserCtxt ( buf , sizeof ( buf ) ) ;
2014-02-16 23:42:56 +00:00
ret = xmlStringLenDecodeEntities ( ctx , ret , strlen ( ret ) , XML_SUBSTITUTE_REF , 0 , 0 , 0 ) ;
2013-03-15 17:02:14 +00:00
return ret ;
}
return buffer ;
}
2013-01-31 00:08:36 +00:00
void parse_xml_buffer ( const char * url , const char * buffer , int size ,
2014-03-14 18:26:07 +00:00
struct dive_table * table , const char * * params )
2011-08-28 23:58:26 +00:00
{
xmlDoc * doc ;
2013-03-15 17:02:14 +00:00
const char * res = preprocess_divelog_de ( buffer ) ;
2011-08-28 23:58:26 +00:00
2013-01-31 00:08:36 +00:00
target_table = table ;
2013-03-15 17:02:14 +00:00
doc = xmlReadMemory ( res , strlen ( res ) , url , NULL , 0 ) ;
2013-03-17 05:12:23 +00:00
if ( res ! = buffer )
free ( ( char * ) res ) ;
2011-08-28 23:58:26 +00:00
if ( ! doc ) {
2014-03-14 18:26:07 +00:00
report_error ( translate ( " gettextFromC " , " Failed to parse '%s' " ) , url ) ;
2011-08-28 23:58:26 +00:00
return ;
}
2014-04-11 06:17:35 +00:00
set_save_userid_local ( false ) ;
set_userid ( " " ) ;
2011-09-02 18:32:48 +00:00
reset_all ( ) ;
2011-08-30 04:32:27 +00:00
dive_start ( ) ;
2014-03-14 18:26:07 +00:00
doc = test_xslt_transforms ( doc , params ) ;
2011-08-30 04:32:27 +00:00
traverse ( xmlDocGetRootElement ( doc ) ) ;
dive_end ( ) ;
2011-08-28 23:58:26 +00:00
xmlFreeDoc ( doc ) ;
}
2013-03-05 05:10:39 +00:00
extern int dm4_events ( void * handle , int columns , char * * data , char * * column )
{
event_start ( ) ;
2014-01-16 04:50:56 +00:00
if ( data [ 1 ] )
2013-03-05 05:10:39 +00:00
cur_event . time . seconds = atoi ( data [ 1 ] ) ;
2014-01-16 04:50:56 +00:00
if ( data [ 2 ] ) {
2013-03-05 05:10:39 +00:00
switch ( atoi ( data [ 2 ] ) ) {
2014-02-16 23:42:56 +00:00
case 1 :
/* 1 Mandatory Safety Stop */
cur_event . name = strdup ( " safety stop (mandatory) " ) ;
break ;
case 3 :
/* 3 Deco */
/* What is Subsurface's term for going to
2013-03-05 05:10:39 +00:00
* deco ? */
2014-02-16 23:42:56 +00:00
cur_event . name = strdup ( " deco " ) ;
break ;
case 4 :
/* 4 Ascent warning */
cur_event . name = strdup ( " ascent " ) ;
break ;
case 5 :
/* 5 Ceiling broken */
cur_event . name = strdup ( " violation " ) ;
break ;
case 6 :
/* 6 Mandatory safety stop ceiling error */
cur_event . name = strdup ( " violation " ) ;
break ;
case 7 :
/* 7 Below deco floor */
cur_event . name = strdup ( " below floor " ) ;
break ;
case 8 :
/* 8 Dive time alarm */
cur_event . name = strdup ( " divetime " ) ;
break ;
case 9 :
/* 9 Depth alarm */
cur_event . name = strdup ( " maxdepth " ) ;
break ;
case 10 :
/* 10 OLF 80% */
case 11 :
/* 11 OLF 100% */
cur_event . name = strdup ( " OLF " ) ;
break ;
case 12 :
2014-06-22 14:41:44 +00:00
/* 12 High pO₂ */
2014-02-16 23:42:56 +00:00
cur_event . name = strdup ( " PO2 " ) ;
break ;
case 13 :
/* 13 Air time */
cur_event . name = strdup ( " airtime " ) ;
break ;
case 17 :
/* 17 Ascent warning */
cur_event . name = strdup ( " ascent " ) ;
break ;
case 18 :
/* 18 Ceiling error */
cur_event . name = strdup ( " ceiling " ) ;
break ;
case 19 :
/* 19 Surfaced */
cur_event . name = strdup ( " surface " ) ;
break ;
case 20 :
/* 20 Deco */
cur_event . name = strdup ( " deco " ) ;
break ;
case 22 :
/* 22 Mandatory safety stop violation */
cur_event . name = strdup ( " violation " ) ;
break ;
case 257 :
/* 257 Dive active */
/* This seems to be given after surface
2013-03-07 05:23:40 +00:00
* when descending again . Ignoring it . */
2014-02-16 23:42:56 +00:00
break ;
case 258 :
/* 258 Bookmark */
if ( data [ 3 ] ) {
cur_event . name = strdup ( " heading " ) ;
cur_event . value = atoi ( data [ 3 ] ) ;
} else {
cur_event . name = strdup ( " bookmark " ) ;
}
break ;
default :
cur_event . name = strdup ( " unknown " ) ;
cur_event . value = atoi ( data [ 2 ] ) ;
break ;
2013-03-05 05:10:39 +00:00
}
}
event_end ( ) ;
return 0 ;
}
2014-01-09 21:14:50 +00:00
extern int dm4_tags ( void * handle , int columns , char * * data , char * * column )
{
2014-01-16 04:50:56 +00:00
if ( data [ 0 ] )
Get rid of crazy empty tag_list element at the start
So this is totally unrelated to the git repository format, except for
the fact that I noticed it while writing the git saving code.
The subsurface divetag list handling is being stupid, and has a
initial dummy entry at the head of the list for no good reason.
I say "no good reason", because there *is* a reason for it: it allows
code to avoid the special case of empty list and adding entries to
before the first entry etc etc. But that reason is a really *bad*
reason, because it's valid only because people don't understand basic
list manipulation and pointers to pointers.
So get rid of the dummy element, and do things right instead - by
passing a *pointer* to the list, instead of the list. And then when
traversing the list and looking for a place to insert things, don't go
to the next entry - just update the "pointer to pointer" to point to
the address of the next entry. Each entry in a C linked list is no
different than the list itself, so you can use the pointer to the
pointer to the next entry as a pointer to the list.
This is a pet peeve of mine. The real beauty of pointers can never be
understood unless you understand the indirection they allow. People
who grew up with Pascal and were corrupted by that mindset are
mentally stunted. Niklaus Wirth has a lot to answer for!
But never fear. You too can overcome that mental limitation, it just
needs some brain exercise. Reading this patch may help. In particular,
contemplate the new "taglist_add_divetag()".
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2014-03-10 17:18:13 +00:00
taglist_add_tag ( & cur_dive - > tag_list , data [ 0 ] ) ;
2014-01-09 21:14:50 +00:00
return 0 ;
}
2013-03-05 05:10:39 +00:00
extern int dm4_dive ( void * param , int columns , char * * data , char * * column )
{
int i , interval , retval = 0 ;
sqlite3 * handle = ( sqlite3 * ) param ;
float * profileBlob ;
unsigned char * tempBlob ;
int * pressureBlob ;
char * err = NULL ;
char get_events_template [ ] = " select * from Mark where DiveId = %d " ;
2014-01-09 21:14:50 +00:00
char get_tags_template [ ] = " select Text from DiveTag where DiveId = %d " ;
2013-03-05 05:10:39 +00:00
char get_events [ 64 ] ;
dive_start ( ) ;
cur_dive - > number = atoi ( data [ 0 ] ) ;
2013-05-15 05:07:41 +00:00
cur_dive - > when = ( time_t ) ( atol ( data [ 1 ] ) ) ;
2013-03-05 05:10:39 +00:00
if ( data [ 2 ] )
utf8_string ( data [ 2 ] , & cur_dive - > notes ) ;
/*
* DM4 stores Duration and DiveTime . It looks like DiveTime is
* 10 to 60 seconds shorter than Duration . However , I have no
* idea what is the difference and which one should be used .
* Duration = data [ 3 ]
* DiveTime = data [ 15 ]
*/
2013-05-15 01:34:54 +00:00
if ( data [ 3 ] )
cur_dive - > duration . seconds = atoi ( data [ 3 ] ) ;
2013-03-05 05:10:39 +00:00
if ( data [ 15 ] )
2013-05-15 01:34:54 +00:00
cur_dive - > dc . duration . seconds = atoi ( data [ 15 ] ) ;
2013-03-05 05:10:39 +00:00
/*
* TODO : the deviceid hash should be calculated here .
*/
settings_start ( ) ;
dc_settings_start ( ) ;
if ( data [ 4 ] )
utf8_string ( data [ 4 ] , & cur_settings . dc . serial_nr ) ;
if ( data [ 5 ] )
utf8_string ( data [ 5 ] , & cur_settings . dc . model ) ;
cur_settings . dc . deviceid = 0xffffffff ;
dc_settings_end ( ) ;
settings_end ( ) ;
if ( data [ 6 ] )
2013-05-15 01:34:54 +00:00
cur_dive - > dc . maxdepth . mm = atof ( data [ 6 ] ) * 1000 ;
2013-03-05 05:10:39 +00:00
if ( data [ 8 ] )
2013-11-26 22:11:30 +00:00
cur_dive - > dc . airtemp . mkelvin = C_to_mkelvin ( atoi ( data [ 8 ] ) ) ;
2013-03-05 05:10:39 +00:00
if ( data [ 9 ] )
2014-02-16 23:42:56 +00:00
cur_dive - > dc . watertemp . mkelvin = C_to_mkelvin ( atoi ( data [ 9 ] ) ) ;
2013-03-05 05:10:39 +00:00
/*
* TODO : handle multiple cylinders
*/
cylinder_start ( ) ;
2013-03-08 19:44:10 +00:00
if ( data [ 22 ] & & atoi ( data [ 22 ] ) > 0 )
cur_dive - > cylinder [ cur_cylinder_index ] . start . mbar = atoi ( data [ 22 ] ) ;
else if ( data [ 10 ] & & atoi ( data [ 10 ] ) > 0 )
cur_dive - > cylinder [ cur_cylinder_index ] . start . mbar = atoi ( data [ 10 ] ) ;
if ( data [ 23 ] & & atoi ( data [ 23 ] ) > 0 )
cur_dive - > cylinder [ cur_cylinder_index ] . end . mbar = ( atoi ( data [ 23 ] ) ) ;
if ( data [ 11 ] & & atoi ( data [ 11 ] ) > 0 )
2013-03-05 05:10:39 +00:00
cur_dive - > cylinder [ cur_cylinder_index ] . end . mbar = ( atoi ( data [ 11 ] ) ) ;
if ( data [ 12 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . type . size . mliter = ( atof ( data [ 12 ] ) ) * 1000 ;
if ( data [ 13 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . type . workingpressure . mbar = ( atoi ( data [ 13 ] ) ) ;
if ( data [ 20 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . gasmix . o2 . permille = atoi ( data [ 20 ] ) * 10 ;
if ( data [ 21 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . gasmix . he . permille = atoi ( data [ 21 ] ) * 10 ;
cylinder_end ( ) ;
if ( data [ 14 ] )
2013-05-15 01:34:54 +00:00
cur_dive - > dc . surface_pressure . mbar = ( atoi ( data [ 14 ] ) * 1000 ) ;
2013-03-05 05:10:39 +00:00
interval = data [ 16 ] ? atoi ( data [ 16 ] ) : 0 ;
profileBlob = ( float * ) data [ 17 ] ;
tempBlob = ( unsigned char * ) data [ 18 ] ;
pressureBlob = ( int * ) data [ 19 ] ;
2014-02-16 23:42:56 +00:00
for ( i = 0 ; interval & & i * interval < cur_dive - > duration . seconds ; i + + ) {
2013-03-05 05:10:39 +00:00
sample_start ( ) ;
cur_sample - > time . seconds = i * interval ;
if ( profileBlob )
cur_sample - > depth . mm = profileBlob [ i ] * 1000 ;
else
2013-05-15 01:34:54 +00:00
cur_sample - > depth . mm = cur_dive - > dc . maxdepth . mm ;
2013-03-05 05:10:39 +00:00
2014-01-09 21:14:49 +00:00
if ( data [ 18 ] & & data [ 18 ] [ 0 ] )
2013-11-26 22:11:30 +00:00
cur_sample - > temperature . mkelvin = C_to_mkelvin ( tempBlob [ i ] ) ;
2013-03-07 04:40:28 +00:00
if ( data [ 19 ] & & data [ 19 ] [ 0 ] )
2014-02-16 23:42:56 +00:00
cur_sample - > cylinderpressure . mbar = pressureBlob [ i ] ;
2013-03-05 05:10:39 +00:00
sample_end ( ) ;
}
snprintf ( get_events , sizeof ( get_events ) - 1 , get_events_template , cur_dive - > number ) ;
retval = sqlite3_exec ( handle , get_events , & dm4_events , 0 , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , " %s " , translate ( " gettextFromC " , " Database query get_events failed. \n " ) ) ;
2013-03-05 05:10:39 +00:00
return 1 ;
}
2014-01-09 21:14:50 +00:00
snprintf ( get_events , sizeof ( get_events ) - 1 , get_tags_template , cur_dive - > number ) ;
retval = sqlite3_exec ( handle , get_events , & dm4_tags , 0 , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , " %s " , translate ( " gettextFromC " , " Database query get_tags failed. \n " ) ) ;
2014-01-09 21:14:50 +00:00
return 1 ;
}
2013-03-05 05:10:39 +00:00
dive_end ( ) ;
/*
for ( i = 0 ; i < columns ; + + i ) {
fprintf ( stderr , " %s \t " , column [ i ] ) ;
}
fprintf ( stderr , " \n " ) ;
for ( i = 0 ; i < columns ; + + i ) {
fprintf ( stderr , " %s \t " , data [ i ] ) ;
}
fprintf ( stderr , " \n " ) ;
//exit(0);
*/
return SQLITE_OK ;
}
2014-02-15 06:36:50 +00:00
int parse_dm4_buffer ( sqlite3 * handle , const char * url , const char * buffer , int size ,
2014-03-14 18:26:07 +00:00
struct dive_table * table )
2013-03-05 05:10:39 +00:00
{
int retval ;
char * err = NULL ;
target_table = table ;
2013-05-09 17:57:39 +00:00
/* StartTime is converted from Suunto's nano seconds to standard
* time . We also need epoch , not seconds since year 1. */
2013-05-15 05:07:41 +00:00
char get_dives [ ] = " select D.DiveId,StartTime/10000000-62135596800,Note,Duration,SourceSerialNumber,Source,MaxDepth,SampleInterval,StartTemperature,BottomTemperature,D.StartPressure,D.EndPressure,Size,CylinderWorkPressure,SurfacePressure,DiveTime,SampleInterval,ProfileBlob,TemperatureBlob,PressureBlob,Oxygen,Helium,MIX.StartPressure,MIX.EndPressure FROM Dive AS D JOIN DiveMixture AS MIX ON D.DiveId=MIX.DiveId " ;
2013-03-05 05:10:39 +00:00
retval = sqlite3_exec ( handle , get_dives , & dm4_dive , handle , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , translate ( " gettextFromC " , " Database query failed '%s'. \n " ) , url ) ;
2013-03-05 05:10:39 +00:00
return 1 ;
}
return 0 ;
}
2014-02-15 06:36:51 +00:00
extern int shearwater_cylinders ( void * handle , int columns , char * * data , char * * column )
{
cylinder_start ( ) ;
if ( data [ 0 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . gasmix . o2 . permille = atof ( data [ 0 ] ) * 1000 ;
if ( data [ 1 ] )
cur_dive - > cylinder [ cur_cylinder_index ] . gasmix . he . permille = atof ( data [ 1 ] ) * 1000 ;
cylinder_end ( ) ;
return 0 ;
}
extern int shearwater_changes ( void * handle , int columns , char * * data , char * * column )
{
event_start ( ) ;
if ( data [ 0 ] )
cur_event . time . seconds = atoi ( data [ 0 ] ) ;
if ( data [ 1 ] ) {
cur_event . name = strdup ( " gaschange " ) ;
cur_event . value = atof ( data [ 1 ] ) * 100 ;
}
event_end ( ) ;
return 0 ;
}
2014-02-15 06:36:50 +00:00
extern int shearwater_profile_sample ( void * handle , int columns , char * * data , char * * column )
{
sample_start ( ) ;
if ( data [ 0 ] )
cur_sample - > time . seconds = atoi ( data [ 0 ] ) ;
if ( data [ 1 ] )
cur_sample - > depth . mm = metric ? atof ( data [ 1 ] ) * 1000 : feet_to_mm ( atof ( data [ 1 ] ) ) ;
if ( data [ 2 ] )
cur_sample - > temperature . mkelvin = metric ? C_to_mkelvin ( atof ( data [ 2 ] ) ) : F_to_mkelvin ( atof ( data [ 2 ] ) ) ;
2014-02-15 06:36:51 +00:00
if ( data [ 3 ] )
2014-06-03 17:21:41 +00:00
cur_sample - > po2 . mbar = atof ( data [ 3 ] ) * 1000 ;
2014-02-15 06:36:51 +00:00
if ( data [ 4 ] )
cur_sample - > ndl . seconds = atoi ( data [ 4 ] ) * 60 ;
if ( data [ 5 ] )
cur_sample - > cns = atoi ( data [ 5 ] ) ;
if ( data [ 6 ] )
2014-02-16 23:42:56 +00:00
cur_sample - > stopdepth . mm = metric ? atoi ( data [ 6 ] ) * 1000 : feet_to_mm ( atoi ( data [ 6 ] ) ) ;
2014-02-15 06:36:50 +00:00
/* We don't actually have data[3], but it should appear in the
* SQL query at some point .
if ( data [ 3 ] )
cur_sample - > cylinderpressure . mbar = metric ? atoi ( data [ 3 ] ) * 1000 : psi_to_mbar ( atoi ( data [ 3 ] ) ) ;
*/
sample_end ( ) ;
return 0 ;
}
extern int shearwater_dive ( void * param , int columns , char * * data , char * * column )
{
2014-02-16 01:45:22 +00:00
int retval = 0 ;
2014-02-15 06:36:50 +00:00
sqlite3 * handle = ( sqlite3 * ) param ;
char * err = NULL ;
2014-02-15 06:36:51 +00:00
char get_profile_template [ ] = " select currentTime,currentDepth,waterTemp,averagePPO2,currentNdl,CNSPercent,decoCeiling from dive_log_records where diveLogId = %d " ;
char get_cylinder_template [ ] = " select fractionO2,fractionHe from dive_log_records where diveLogId = %d group by fractionO2,fractionHe " ;
char get_changes_template [ ] = " select a.currentTime,a.fractionO2,a.fractionHe from dive_log_records as a,dive_log_records as b where a.diveLogId = %d and b.diveLogId = %d and (a.id - 1) = b.id and (a.fractionO2 != b.fractionO2 or a.fractionHe != b.fractionHe) union select min(currentTime),fractionO2,fractionHe from dive_log_records " ;
char get_buffer [ 1024 ] ;
2014-02-15 06:36:50 +00:00
dive_start ( ) ;
cur_dive - > number = atoi ( data [ 0 ] ) ;
cur_dive - > when = ( time_t ) ( atol ( data [ 1 ] ) ) ;
if ( data [ 2 ] )
utf8_string ( data [ 2 ] , & cur_dive - > location ) ;
if ( data [ 3 ] )
utf8_string ( data [ 3 ] , & cur_dive - > buddy ) ;
if ( data [ 4 ] )
utf8_string ( data [ 4 ] , & cur_dive - > notes ) ;
metric = atoi ( data [ 5 ] ) = = 1 ? 0 : 1 ;
/* TODO: verify that metric calculation is correct */
if ( data [ 6 ] )
cur_dive - > dc . maxdepth . mm = metric ? atof ( data [ 6 ] ) * 1000 : feet_to_mm ( atof ( data [ 6 ] ) ) ;
if ( data [ 7 ] )
cur_dive - > dc . duration . seconds = atoi ( data [ 7 ] ) * 60 ;
if ( data [ 8 ] )
cur_dive - > dc . surface_pressure . mbar = atoi ( data [ 8 ] ) ;
/*
* TODO : the deviceid hash should be calculated here .
*/
settings_start ( ) ;
dc_settings_start ( ) ;
if ( data [ 9 ] )
utf8_string ( data [ 9 ] , & cur_settings . dc . serial_nr ) ;
if ( data [ 10 ] )
utf8_string ( data [ 10 ] , & cur_settings . dc . model ) ;
cur_settings . dc . deviceid = 0xffffffff ;
dc_settings_end ( ) ;
settings_end ( ) ;
2014-02-15 06:36:51 +00:00
snprintf ( get_buffer , sizeof ( get_buffer ) - 1 , get_cylinder_template , cur_dive - > number ) ;
retval = sqlite3_exec ( handle , get_buffer , & shearwater_cylinders , 0 , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , " %s " , translate ( " gettextFromC " , " Database query get_cylinders failed. \n " ) ) ;
2014-02-15 06:36:51 +00:00
return 1 ;
}
snprintf ( get_buffer , sizeof ( get_buffer ) - 1 , get_changes_template , cur_dive - > number , cur_dive - > number ) ;
retval = sqlite3_exec ( handle , get_buffer , & shearwater_changes , 0 , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , " %s " , translate ( " gettextFromC " , " Database query get_changes failed. \n " ) ) ;
2014-02-15 06:36:51 +00:00
return 1 ;
}
snprintf ( get_buffer , sizeof ( get_buffer ) - 1 , get_profile_template , cur_dive - > number ) ;
retval = sqlite3_exec ( handle , get_buffer , & shearwater_profile_sample , 0 , & err ) ;
2014-02-15 06:36:50 +00:00
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , " %s " , translate ( " gettextFromC " , " Database query get_profile_sample failed. \n " ) ) ;
2014-02-15 06:36:50 +00:00
return 1 ;
}
dive_end ( ) ;
return SQLITE_OK ;
}
int parse_shearwater_buffer ( sqlite3 * handle , const char * url , const char * buffer , int size ,
2014-03-14 18:26:07 +00:00
struct dive_table * table )
2014-02-15 06:36:50 +00:00
{
int retval ;
char * err = NULL ;
target_table = table ;
char get_dives [ ] = " select i.diveId,timestamp,location||' / '||site,buddy,notes,imperialUnits,maxDepth,maxTime,startSurfacePressure,computerSerial,computerModel FROM dive_info AS i JOIN dive_logs AS l ON i.diveId=l.diveId " ;
retval = sqlite3_exec ( handle , get_dives , & shearwater_dive , handle , & err ) ;
if ( retval ! = SQLITE_OK ) {
2014-02-16 23:42:56 +00:00
fprintf ( stderr , translate ( " gettextFromC " , " Database query failed '%s'. \n " ) , url ) ;
2014-02-15 06:36:50 +00:00
return 1 ;
}
return 0 ;
}
2011-08-31 01:40:25 +00:00
void parse_xml_init ( void )
2011-08-30 23:28:59 +00:00
{
2011-08-28 23:58:26 +00:00
LIBXML_TEST_VERSION
}
2011-11-05 10:39:17 +00:00
2012-09-18 15:33:55 +00:00
void parse_xml_exit ( void )
{
xmlCleanupParser ( ) ;
}
2011-11-16 05:12:43 +00:00
static struct xslt_files {
const char * root ;
const char * file ;
2014-02-08 12:00:54 +00:00
const char * attribute ;
2011-11-16 05:12:43 +00:00
} xslt_files [ ] = {
2014-02-16 23:42:56 +00:00
{ " SUUNTO " , " SuuntoSDM.xslt " , NULL } ,
{ " Dive " , " SuuntoDM4.xslt " , " xmlns " } ,
{ " Dive " , " shearwater.xslt " , " version " } ,
{ " JDiveLog " , " jdivelog2subsurface.xslt " , NULL } ,
{ " dives " , " MacDive.xslt " , NULL } ,
{ " DIVELOGSDATA " , " divelogs.xslt " , NULL } ,
{ " uddf " , " uddf.xslt " , NULL } ,
{ " UDDF " , " uddf.xslt " , NULL } ,
{ " profile " , " udcf.xslt " , NULL } ,
{ " Divinglog " , " DivingLog.xslt " , NULL } ,
{ " csv " , " csv2xml.xslt " , NULL } ,
{ " sensuscsv " , " sensuscsv.xslt " , NULL } ,
{ " manualcsv " , " manualcsv2xml.xslt " , NULL } ,
{ NULL , }
} ;
2011-11-16 05:12:43 +00:00
2014-03-14 18:26:07 +00:00
static xmlDoc * test_xslt_transforms ( xmlDoc * doc , const char * * params )
2011-11-05 10:39:17 +00:00
{
2011-11-16 05:12:43 +00:00
struct xslt_files * info = xslt_files ;
2011-11-05 10:39:17 +00:00
xmlDoc * transformed ;
xsltStylesheetPtr xslt = NULL ;
xmlNode * root_element = xmlDocGetRootElement ( doc ) ;
2013-02-07 19:20:02 +00:00
char * attribute ;
2011-11-16 05:12:43 +00:00
2014-02-08 12:00:54 +00:00
while ( info - > root ) {
2014-02-09 21:00:33 +00:00
if ( ( strcasecmp ( root_element - > name , info - > root ) = = 0 ) ) {
2014-02-08 12:00:54 +00:00
if ( info - > attribute = = NULL )
break ;
else if ( xmlGetProp ( root_element , info - > attribute ) ! = NULL )
break ;
2014-02-09 21:00:33 +00:00
}
2011-11-16 05:12:43 +00:00
info + + ;
}
if ( info - > root ) {
2013-02-07 19:20:02 +00:00
attribute = xmlGetProp ( xmlFirstElementChild ( root_element ) , " name " ) ;
2013-02-21 01:07:22 +00:00
if ( attribute ) {
if ( strcasecmp ( attribute , " subsurface " ) = = 0 ) {
free ( ( void * ) attribute ) ;
return doc ;
}
free ( ( void * ) attribute ) ;
}
2013-02-07 19:20:02 +00:00
2011-11-05 10:39:17 +00:00
xmlSubstituteEntitiesDefault ( 1 ) ;
2011-11-16 05:12:43 +00:00
xslt = get_stylesheet ( info - > file ) ;
2013-02-22 19:20:46 +00:00
if ( xslt = = NULL ) {
2014-03-14 18:26:07 +00:00
report_error ( translate ( " gettextFromC " , " Can't open stylesheet %s " ) , info - > file ) ;
2011-11-05 10:39:17 +00:00
return doc ;
2013-02-22 19:20:46 +00:00
}
2013-10-16 19:05:19 +00:00
transformed = xsltApplyStylesheet ( xslt , doc , params ) ;
2011-11-05 10:39:17 +00:00
xmlFreeDoc ( doc ) ;
xsltFreeStylesheet ( xslt ) ;
2013-10-16 19:05:19 +00:00
2011-11-05 10:39:17 +00:00
return transformed ;
}
return doc ;
}