X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=strutil.c;h=0ddeae0973690ca2ec6535a7758f56a75d34de34;hb=d713e561814558d9f07c4186aef35a23df8b8445;hp=9ad83d7f7804433cd0ec595aa901520be0a0d919;hpb=31fc1fbce399ac89d07093886301b9b4971f56f9;p=libsigrok.git

diff --git a/strutil.c b/strutil.c
index 9ad83d7f..0ddeae09 100644
--- a/strutil.c
+++ b/strutil.c
@@ -24,6 +24,52 @@
 #include "libsigrok.h"
 #include "libsigrok-internal.h"
 
+/**
+ * Convert a numeric value value to its "natural" string representation.
+ * in SI units
+ *
+ * E.g. a value of 3000000, with units set to "W", would be converted
+ * to "3 MW", 20000 to "20 kW", 31500 would become "31.5 kW".
+ *
+ * @param x The value to convert.
+ * @param unit The unit to append to the string, or NULL if the string
+ *             has no units.
+ *
+ * @return A g_try_malloc()ed string representation of the samplerate value,
+ *         or NULL upon errors. The caller is responsible to g_free() the
+ *         memory.
+ */
+SR_API char *sr_si_string_u64(uint64_t x, const char *unit)
+{
+	if(unit == NULL)
+		unit = "";
+
+	if ((x >= SR_GHZ(1)) && (x % SR_GHZ(1) == 0)) {
+		return g_strdup_printf("%" PRIu64 " G%s", x / SR_GHZ(1), unit);
+	} else if ((x >= SR_GHZ(1)) && (x % SR_GHZ(1) != 0)) {
+		return g_strdup_printf("%" PRIu64 ".%" PRIu64 " G%s",
+				       x / SR_GHZ(1), x % SR_GHZ(1), unit);
+	} else if ((x >= SR_MHZ(1)) && (x % SR_MHZ(1) == 0)) {
+		return g_strdup_printf("%" PRIu64 " M%s",
+				       x / SR_MHZ(1), unit);
+	} else if ((x >= SR_MHZ(1)) && (x % SR_MHZ(1) != 0)) {
+		return g_strdup_printf("%" PRIu64 ".%" PRIu64 " M%s",
+				    x / SR_MHZ(1), x % SR_MHZ(1), unit);
+	} else if ((x >= SR_KHZ(1)) && (x % SR_KHZ(1) == 0)) {
+		return g_strdup_printf("%" PRIu64 " k%s",
+				    x / SR_KHZ(1), unit);
+	} else if ((x >= SR_KHZ(1)) && (x % SR_KHZ(1) != 0)) {
+		return g_strdup_printf("%" PRIu64 ".%" PRIu64 " k%s",
+				    x / SR_KHZ(1), x % SR_KHZ(1), unit);
+	} else {
+		return g_strdup_printf("%" PRIu64 " %s", x, unit);
+	}
+
+	sr_err("strutil: %s: Error creating SI units string.",
+		__func__);
+	return NULL;
+}
+
 /**
  * Convert a numeric samplerate value to its "natural" string representation.
  *
@@ -38,35 +84,7 @@
  */
 SR_API char *sr_samplerate_string(uint64_t samplerate)
 {
-	char *o;
-	uint64_t s = samplerate;
-
-	if ((s >= SR_GHZ(1)) && (s % SR_GHZ(1) == 0)) {
-		o = g_strdup_printf("%" PRIu64 " GHz", s / SR_GHZ(1));
-	} else if ((s >= SR_GHZ(1)) && (s % SR_GHZ(1) != 0)) {
-		o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " GHz",
-				    s / SR_GHZ(1), s % SR_GHZ(1));
-	} else if ((s >= SR_MHZ(1)) && (s % SR_MHZ(1) == 0)) {
-		o = g_strdup_printf("%" PRIu64 " MHz", s / SR_MHZ(1));
-	} else if ((s >= SR_MHZ(1)) && (s % SR_MHZ(1) != 0)) {
-		o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " MHz",
-				    s / SR_MHZ(1), s % SR_MHZ(1));
-	} else if ((s >= SR_KHZ(1)) && (s % SR_KHZ(1) == 0)) {
-		o = g_strdup_printf("%" PRIu64 " kHz", s / SR_KHZ(1));
-	} else if ((s >= SR_KHZ(1)) && (s % SR_KHZ(1) != 0)) {
-		o = g_strdup_printf("%" PRIu64 ".%" PRIu64 " kHz",
-				    s / SR_KHZ(1), s % SR_KHZ(1));
-	} else {
-		o = g_strdup_printf("%" PRIu64 " Hz", s);
-	}
-
-	if (!o) {
-		sr_err("strutil: %s: Error creating samplerate string.",
-		       __func__);
-		return NULL;
-	}
-
-	return o;
+	return sr_si_string_u64(samplerate, "Hz");
 }
 
 /**
@@ -196,24 +214,19 @@ SR_API char **sr_parse_triggerstring(const struct sr_dev_inst *sdi,
 
 	tokens = g_strsplit(triggerstring, ",", max_probes);
 	for (i = 0; tokens[i]; i++) {
-		if (tokens[i][0] < '0' || tokens[i][0] > '9') {
-			/* Named probe */
-			probenum = 0;
-			for (l = sdi->probes; l; l = l->next) {
-				probe = (struct sr_probe *)l->data;
-				if (probe->enabled
-				    && !strncmp(probe->name, tokens[i],
-						strlen(probe->name))) {
-					probenum = probe->index;
-					break;
-				}
+		probenum = -1;
+		for (l = sdi->probes; l; l = l->next) {
+			probe = (struct sr_probe *)l->data;
+			if (probe->enabled
+				&& !strncmp(probe->name, tokens[i],
+					strlen(probe->name))) {
+				probenum = probe->index;
+				break;
 			}
-		} else {
-			probenum = strtol(tokens[i], NULL, 10);
 		}
 
 		if (probenum < 0 || probenum >= max_probes) {
-			sr_err("strutil: Invalid probe (%d).", probenum);
+			sr_err("strutil: Invalid probe.");
 			error = TRUE;
 			break;
 		}