sr_parse_(period|voltage): deprecate struct sr_rational

[libsigrok.git] / strutil.c

diff --git a/strutil.c b/strutil.c
index 5ac9b71912a7ef1fa4a03f6fef784f5e6727313d..f9299ca6a08eec2219b9044c9e0657d8a1d18f24 100644 (file)
--- a/strutil.c
+++ b/strutil.c
@@ -165,8 +165,8 @@ SR_API char *sr_period_string(uint64_t frequency)
  */
 SR_API char *sr_voltage_string(struct sr_rational *voltage)
 {
-       char *o;
        int r;
+       char *o;
 
        if (!(o = g_try_malloc0(30 + 1))) {
                sr_err("%s: o malloc failed", __func__);
@@ -178,7 +178,7 @@ SR_API char *sr_voltage_string(struct sr_rational *voltage)
        else if (voltage->q == 1)
                r = snprintf(o, 30, "%" PRIu64 "V", voltage->p);
        else
-               r = -1;
+               r = snprintf(o, 30, "%gV", (float)voltage->p / (float)voltage->q);
 
        if (r < 0) {
                /* Something went wrong... */
@@ -216,6 +216,7 @@ SR_API char **sr_parse_triggerstring(const struct sr_dev_inst *sdi,
                                     const char *triggerstring)
 {
        GSList *l;
+       GVariant *gvar;
        struct sr_probe *probe;
        int max_probes, probenum, i;
        char **tokens, **triggerlist, *trigger, *tc;
@@ -230,11 +231,11 @@ SR_API char **sr_parse_triggerstring(const struct sr_dev_inst *sdi,
                return NULL;
        }
 
-       if (sdi->driver->info_get(SR_DI_TRIGGER_TYPES,
-                       (const void **)&trigger_types, sdi) != SR_OK) {
+       if (sdi->driver->config_list(SR_CONF_TRIGGER_TYPE, &gvar, sdi) != SR_OK) {
                sr_err("%s: Device doesn't support any triggers.", __func__);
                return NULL;
        }
+       trigger_types = g_variant_get_string(gvar, NULL);
 
        tokens = g_strsplit(triggerstring, ",", max_probes);
        for (i = 0; tokens[i]; i++) {
@@ -269,6 +270,7 @@ SR_API char **sr_parse_triggerstring(const struct sr_dev_inst *sdi,
                }
        }
        g_strfreev(tokens);
+       g_variant_unref(gvar);
 
        if (error) {
                for (i = 0; i < max_probes; i++)
@@ -391,12 +393,12 @@ SR_API gboolean sr_parse_boolstring(const char *boolstr)
        return FALSE;
 }
 
-SR_API int sr_parse_period(const char *periodstr, struct sr_rational *r)
+SR_API int sr_parse_period(const char *periodstr, uint64_t *p, uint64_t *q)
 {
        char *s;
 
-       r->p = strtoull(periodstr, &s, 10);
-       if (r->p == 0 && s == periodstr)
+       *p = strtoull(periodstr, &s, 10);
+       if (*p == 0 && s == periodstr)
                /* No digits found. */
                return SR_ERR_ARG;
 
@@ -404,17 +406,17 @@ SR_API int sr_parse_period(const char *periodstr, struct sr_rational *r)
                while (*s == ' ')
                        s++;
                if (!strcmp(s, "fs"))
-                       r->q = 1000000000000000ULL;
+                       *q = 1000000000000000ULL;
                else if (!strcmp(s, "ps"))
-                       r->q = 1000000000000ULL;
+                       *q = 1000000000000ULL;
                else if (!strcmp(s, "ns"))
-                       r->q = 1000000000ULL;
+                       *q = 1000000000ULL;
                else if (!strcmp(s, "us"))
-                       r->q = 1000000;
+                       *q = 1000000;
                else if (!strcmp(s, "ms"))
-                       r->q = 1000;
+                       *q = 1000;
                else if (!strcmp(s, "s"))
-                       r->q = 1;
+                       *q = 1;
                else
                        /* Must have a time suffix. */
                        return SR_ERR_ARG;
@@ -424,12 +426,12 @@ SR_API int sr_parse_period(const char *periodstr, struct sr_rational *r)
 }
 
 
-SR_API int sr_parse_voltage(const char *voltstr, struct sr_rational *r)
+SR_API int sr_parse_voltage(const char *voltstr, uint64_t *p, uint64_t *q)
 {
        char *s;
 
-       r->p = strtoull(voltstr, &s, 10);
-       if (r->p == 0 && s == voltstr)
+       *p = strtoull(voltstr, &s, 10);
+       if (*p == 0 && s == voltstr)
                /* No digits found. */
                return SR_ERR_ARG;
 
@@ -437,9 +439,9 @@ SR_API int sr_parse_voltage(const char *voltstr, struct sr_rational *r)
                while (*s == ' ')
                        s++;
                if (!strcasecmp(s, "mv"))
-                       r->q = 1000L;
+                       *q = 1000L;
                else if (!strcasecmp(s, "v"))
-                       r->q = 1;
+                       *q = 1;
                else
                        /* Must have a base suffix. */
                        return SR_ERR_ARG;