X-Git-Url: https://sigrok.org/gitweb/?p=sigrok-cli.git;a=blobdiff_plain;f=parsers.c;h=794e4b56c3e73d36dbb0b8ed9fa342fa2f2df982;hp=fdf8562f365f73a04a4099f361eadceade29e191;hb=8ab6aafcf752f6a8b1aedebd64b267eb7b385641;hpb=c2c4a0def11bd2a589aa62f8f501ff20ef5f99e4

diff --git a/parsers.c b/parsers.c
index fdf8562..794e4b5 100644
--- a/parsers.c
+++ b/parsers.c
@@ -22,7 +22,7 @@
 #include <stdint.h>
 #include <string.h>
 #include <glib.h>
-#include <sigrok.h>
+#include <libsigrok/libsigrok.h>
 #include "sigrok-cli.h"
 
 char **parse_probestring(int max_probes, const char *probestring)
@@ -44,41 +44,39 @@ char **parse_probestring(int max_probes, const char *probestring)
 			range = g_strsplit(tokens[i], "-", 2);
 			if (!range[0] || !range[1] || range[2]) {
 				/* Need exactly two arguments. */
-				printf("Invalid probe syntax '%s'.\n",
-				       tokens[i]);
+				g_critical("Invalid probe syntax '%s'.", tokens[i]);
 				error = TRUE;
 				break;
 			}
 
 			b = strtol(range[0], NULL, 10);
 			e = strtol(range[1], NULL, 10);
-			if (b < 1 || e > max_probes || b >= e) {
-				printf("Invalid probe range '%s'.\n",
-				       tokens[i]);
+			if (b < 0 || e >= max_probes || b >= e) {
+				g_critical("Invalid probe range '%s'.", tokens[i]);
 				error = TRUE;
 				break;
 			}
 
 			while (b <= e) {
 				snprintf(str, 7, "%d", b);
-				probelist[b - 1] = g_strdup(str);
+				probelist[b] = g_strdup(str);
 				b++;
 			}
 		} else {
 			tmp = strtol(tokens[i], NULL, 10);
-			if (tmp < 1 || tmp > max_probes) {
-				printf("Invalid probe %d.\n", tmp);
+			if (tmp < 0 || tmp >= max_probes) {
+				g_critical("Invalid probe %d.", tmp);
 				error = TRUE;
 				break;
 			}
 
 			if ((name = strchr(tokens[i], '='))) {
-				probelist[tmp - 1] = g_strdup(++name);
-				if (strlen(probelist[tmp - 1]) > SR_MAX_PROBENAME_LEN)
-					probelist[tmp - 1][SR_MAX_PROBENAME_LEN] = 0;
+				probelist[tmp] = g_strdup(++name);
+				if (strlen(probelist[tmp]) > SR_MAX_PROBENAME_LEN)
+					probelist[tmp][SR_MAX_PROBENAME_LEN] = 0;
 			} else {
 				snprintf(str, 7, "%d", tmp);
-				probelist[tmp - 1] = g_strdup(str);
+				probelist[tmp] = g_strdup(str);
 			}
 		}
 	}
@@ -98,7 +96,7 @@ char **parse_probestring(int max_probes, const char *probestring)
 	return probelist;
 }
 
-GHashTable *parse_generic_arg(const char *arg)
+GHashTable *parse_generic_arg(const char *arg, gboolean sep_first)
 {
 	GHashTable *hash;
 	int i;
@@ -107,10 +105,14 @@ GHashTable *parse_generic_arg(const char *arg)
 	if (!arg || !arg[0])
 		return NULL;
 
-	hash = g_hash_table_new_full(g_str_hash, g_str_equal, g_free, g_free);
+	i = 0;
+	hash = g_hash_table_new_full(g_str_hash, g_str_equal,
+			g_free, g_free);
 	elements = g_strsplit(arg, ":", 0);
-	g_hash_table_insert(hash, g_strdup("sigrok_key"), g_strdup(elements[0]));
-	for (i = 1; elements[i]; i++) {
+	if (sep_first)
+		g_hash_table_insert(hash, g_strdup("sigrok_key"),
+				g_strdup(elements[i++]));
+	for (; elements[i]; i++) {
 		e = strchr(elements[i], '=');
 		if (!e)
 			g_hash_table_insert(hash, g_strdup(elements[i]), NULL);
@@ -124,63 +126,33 @@ GHashTable *parse_generic_arg(const char *arg)
 	return hash;
 }
 
-struct sr_device *parse_devicestring(const char *devicestring)
+char *strcanon(const char *str)
 {
-	struct sr_device *device, *d;
-	struct sr_device_plugin *plugin;
-	GSList *devices, *plugins, *l, *p;
-	int num_devices, device_num, device_cnt;
-	char *tmp;
+	int p0, p1;
+	char *s;
+
+	/* Returns newly allocated string. */
+	s = g_ascii_strdown(str, -1);
+	for (p0 = p1 = 0; str[p0]; p0++) {
+		if ((s[p0] >= 'a' && s[p0] <= 'z')
+				|| (s[p0] >= '0' && s[p0] <= '9'))
+			s[p1++] = s[p0];
+	}
+	s[p1] = '\0';
 
-	if (!devicestring)
-		return NULL;
+	return s;
+}
 
-	device = NULL;
-	device_num = strtol(devicestring, &tmp, 10);
-	if (tmp != devicestring) {
-		/* argument is numeric, meaning a device ID. Make all driver
-		 * plugins scan for devices.
-		 */
-		num_devices = num_real_devices();
-		if (device_num < 0 || device_num >= num_devices)
-			return NULL;
-
-		device_cnt = 0;
-		devices = sr_device_list();
-		for (l = devices; l; l = l->next) {
-			d = l->data;
-			if (sr_device_has_hwcap(d, SR_HWCAP_DEMO_DEVICE))
-				continue;
-			if (device_cnt == device_num) {
-				if (device_num == device_cnt) {
-					device = d;
-					break;
-				}
-			}
-			device_cnt++;
-		}
-	} else {
-		/* select device by driver -- only initialize that driver,
-		 * no need to let them all scan
-		 */
-		device = NULL;
-		plugins = sr_list_hwplugins();
-		for (p = plugins; p; p = p->next) {
-			plugin = p->data;
-			if (strcmp(plugin->name, devicestring))
-				continue;
-			num_devices = sr_init_hwplugins(plugin);
-			if (num_devices == 1) {
-				devices = sr_device_list();
-				device = devices->data;
-			} else if (num_devices > 1) {
-				printf("driver '%s' found %d devices, select by ID instead.\n",
-						devicestring, num_devices);
-			}
-			/* fall through: selected driver found no devices */
-			break;
-		}
-	}
+int canon_cmp(const char *str1, const char *str2)
+{
+	int ret;
+	char *s1, *s2;
+
+	s1 = strcanon(str1);
+	s2 = strcanon(str2);
+	ret = g_ascii_strcasecmp(s1, s2);
+	g_free(s2);
+	g_free(s1);
 
-	return device;
+	return ret;
 }