FlapJackApps/libdc
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Suunto EON Steel: add 'varargs' version of 'add_string()' function

Browse Source 
.. and use it for the cases that used to do this by hand (Personal
Adjustment and Desaturation Time).

There will be more users coming up.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Linus Torvalds 2016-07-21 19:36:51 +09:00 committed by Dirk Hohndel
parent 69d26e00b0
commit 4599405078
1 changed files with 27 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
src/suunto_eonsteel_parser.c
View File

@ -24,10 +24,12 @@
#include <stdio.h> #include <stdio.h>
#include <ctype.h> #include <ctype.h>
#include <math.h> #include <math.h>
#include <stdarg.h>
/* Wow. MSC is truly crap */ /* Wow. MSC is truly crap */
#ifdef _MSC_VER #ifdef _MSC_VER
#define snprintf _snprintf #define snprintf _snprintf
#define vsnprintf _vsnprintf
#endif #endif
#include <libdivecomputer/suunto_eonsteel.h> #include <libdivecomputer/suunto_eonsteel.h>
@ -1198,6 +1200,26 @@ static int add_string(suunto_eonsteel_parser_t *eon, const char *desc, const cha
return 0; return 0;
} }
static int add_string_fmt(suunto_eonsteel_parser_t *eon, const char *desc, const char *fmt, ...)
{
char buffer[256];
va_list ap;
/*
* We ignore the return value from vsnprintf, and we
* always NUL-terminate the destination buffer ourselves.
*
* That way we don't have to worry about random bad legacy
* implementations.
*/
va_start(ap, fmt);
buffer[sizeof(buffer)-1] = 0;
(void) vsnprintf(buffer, sizeof(buffer)-1, fmt, ap);
va_end(ap);
return add_string(eon, desc, buffer);
}
static float get_le32_float(const unsigned char *src) static float get_le32_float(const unsigned char *src)
{ {
union { union {
@ -1325,11 +1347,9 @@ static int traverse_diving_fields(suunto_eonsteel_parser_t *eon, const struct ty
/* Signed byte of conservatism (-2 .. +2) */ /* Signed byte of conservatism (-2 .. +2) */
if (!strcmp(name, "Conservatism")) { if (!strcmp(name, "Conservatism")) {
char buffer[10];
int val = *(signed char *)data; int val = *(signed char *)data;
snprintf(buffer, sizeof(buffer), "P%d", val); return add_string_fmt(eon, "Personal Adjustment", "P%d", val);
return add_string(eon, "Personal Adjustment", buffer);
} }
if (!strcmp(name, "LowSetPoint")) { if (!strcmp(name, "LowSetPoint")) {
@ -1344,14 +1364,11 @@ static int traverse_diving_fields(suunto_eonsteel_parser_t *eon, const struct ty
return 0; return 0;
} }
// Time recoded in seconds.
// Let's just agree to ignore seconds
if (!strcmp(name, "DesaturationTime")) { if (!strcmp(name, "DesaturationTime")) {
char desat[16]; unsigned int time = array_uint32_le(data) / 60;
unsigned int time = array_uint32_le(data); // In seconds! return add_string_fmt(eon, "Desaturation Time", "%d:%02d", time / 60, time % 60);
// Let's just agree to ignore seconds
time /= 60;
snprintf(desat, sizeof(desat), "%d:%02d", time / 60, time % 60);
return add_string(eon, "Desaturation Time", desat);
} }
return 0; return 0;