[PATCH 01/11] Fixes to parse_options

Bill Spitzak spitzak at gmail.com
Tue Jul 8 21:17:04 PDT 2014


Fail on malformed numbers, such as --width=100mm

Fail on = after booleans, such as --flag=false

Multiple single-letter booleans in one switch allowed, ie
-xyz is the same as -x -y -z. For wayland modules they all have
to belong to the same module.

Previous version could use text after the null at the end of
an argv entry. Now requires the = sign.
---
 shared/option-parser.c |   82 +++++++++++++++++++++++++++++++-----------------
 1 file changed, 53 insertions(+), 29 deletions(-)

diff --git a/shared/option-parser.c b/shared/option-parser.c
index c00349a..22bcf2f 100644
--- a/shared/option-parser.c
+++ b/shared/option-parser.c
@@ -30,53 +30,77 @@
 
 #include "config-parser.h"
 
-static void
+static int
 handle_option(const struct weston_option *option, char *value)
 {
+	char* p;
 	switch (option->type) {
 	case WESTON_OPTION_INTEGER:
-		* (int32_t *) option->data = strtol(value, NULL, 0);
-		return;
+		* (int32_t *) option->data = strtol(value, &p, 0);
+		return p > value && !*p;
 	case WESTON_OPTION_UNSIGNED_INTEGER:
-		* (uint32_t *) option->data = strtoul(value, NULL, 0);
-		return;
+		* (uint32_t *) option->data = strtoul(value, &p, 0);
+		return p > value && !*p;
 	case WESTON_OPTION_STRING:
 		* (char **) option->data = strdup(value);
-		return;
-	case WESTON_OPTION_BOOLEAN:
-		* (int32_t *) option->data = 1;
-		return;
+		return 1;
 	default:
 		assert(0);
 	}
 }
 
+static int
+long_option(const struct weston_option *options, int count, char* arg)
+{
+	int k, len;
+	for (k = 0; k < count; k++) {
+		if (! options[k].name) continue;
+		len = strlen(options[k].name);
+		if (strncmp(options[k].name, arg+2, len) != 0) continue;
+		if (options[k].type == WESTON_OPTION_BOOLEAN) {
+			if (! arg[len+2]) {
+				* (int32_t *) options[k].data = 1;
+				return 1;
+			}
+		} else if (arg[len+2] == '=') {
+			return handle_option(options+k, arg+len+3);
+		}
+	}
+	return 0;
+}
+
+static int
+short_option(const struct weston_option *options, int count, char* arg)
+{
+	int i, k;
+	for (i = 1; arg[i]; i++) {
+		for (k = 0; ; k++) {
+			if (k >= count) return 0;
+			if (options[k].short_name == arg[i]) break;
+		}
+		if (options[k].type == WESTON_OPTION_BOOLEAN)
+			* (int32_t *) options[k].data = 1;
+		else
+			return handle_option(options+k, arg+i+1);
+	}
+	return arg[1]; /* error on just '-' */
+}
+
+
 int
 parse_options(const struct weston_option *options,
 	      int count, int *argc, char *argv[])
 {
-	int i, j, k, len = 0;
-
+	int i, j;
 	for (i = 1, j = 1; i < *argc; i++) {
-		for (k = 0; k < count; k++) {
-			if (options[k].name)
-				len = strlen(options[k].name);
-			if (options[k].name &&
-			    argv[i][0] == '-' &&
-			    argv[i][1] == '-' &&
-			    strncmp(options[k].name, &argv[i][2], len) == 0 &&
-			    (argv[i][len + 2] == '=' || argv[i][len + 2] == '\0')) {
-				handle_option(&options[k], &argv[i][len + 3]);
-				break;
-			} else if (options[k].short_name &&
-				   argv[i][0] == '-' &&
-				   options[k].short_name == argv[i][1]) {
-				handle_option(&options[k], &argv[i][2]);
-				break;
-			}
+		if (argv[i][0] == '-') {
+			if (argv[i][1] == '-') {
+				if (long_option(options, count, argv[i]))
+					continue;
+			} else if (short_option(options, count, argv[i]))
+				continue;
 		}
-		if (k == count)
-			argv[j++] = argv[i];
+		argv[j++] = argv[i];
 	}
 	argv[j] = NULL;
 	*argc = j;
-- 
1.7.9.5



More information about the wayland-devel mailing list