X-Git-Url: https://git.ozlabs.org/?p=ccan;a=blobdiff_plain;f=ccan%2Fopt%2Fopt.c;h=bae3c665c93e7594bb4c5e7e32880f5e73b13952;hp=e7ebc3dd0550e2181b13c41438ddeb9e8f244dff;hb=d6daa78732b7f67a30283d9cad7c98cbcbdb7414;hpb=f4b1f445a7b21b1599530afb897ef54efe15479d

diff --git a/ccan/opt/opt.c b/ccan/opt/opt.c
index e7ebc3dd..bae3c665 100644
--- a/ccan/opt/opt.c
+++ b/ccan/opt/opt.c
@@ -32,7 +32,7 @@ static const char *next_name(const char *names, unsigned *len)
 static const char *first_opt(unsigned *i, unsigned *len)
 {
 	for (*i = 0; *i < opt_count; (*i)++) {
-		if (opt_table[*i].flags == OPT_SUBTABLE)
+		if (opt_table[*i].type == OPT_SUBTABLE)
 			continue;
 		return first_name(opt_table[*i].names, len);
 	}
@@ -42,7 +42,7 @@ static const char *first_opt(unsigned *i, unsigned *len)
 static const char *next_opt(const char *p, unsigned *i, unsigned *len)
 {
 	for (; *i < opt_count; (*i)++) {
-		if (opt_table[*i].flags == OPT_SUBTABLE)
+		if (opt_table[*i].type == OPT_SUBTABLE)
 			continue;
 		if (!p)
 			return first_name(opt_table[*i].names, len);
@@ -83,7 +83,7 @@ static const char *next_lopt(const char *p, unsigned *i, unsigned *len)
 const char *first_sopt(unsigned *i)
 {
 	const char *p;
-	unsigned int len;
+	unsigned int len = 0 /* GCC bogus warning */;
 
 	for (p = first_opt(i, &len); p; p = next_opt(p, i, &len)) {
 		if (p[0] != '-')
@@ -107,26 +107,44 @@ static void check_opt(const struct opt_table *entry)
 	const char *p;
 	unsigned len;
 
-	assert(entry->flags == OPT_HASARG || entry->flags == OPT_NOARG);
+	if (entry->type != OPT_HASARG && entry->type != OPT_NOARG)
+		errx(1, "Option %s: unknown entry type %u",
+		     entry->names, entry->type);
+
+	if (!entry->desc)
+		errx(1, "Option %s: description cannot be NULL", entry->names);
+
+
+	if (entry->names[0] != '-')
+		errx(1, "Option %s: does not begin with '-'", entry->names);
 
-	assert(entry->names[0] == '-');
 	for (p = first_name(entry->names, &len); p; p = next_name(p, &len)) {
 		if (*p == '-') {
-			assert(len > 1);
+			if (len == 1)
+				errx(1, "Option %s: invalid long option '--'",
+				     entry->names);
 			opt_num_long++;
 		} else {
-			assert(len == 1);
-			assert(*p != ':');
+			if (len != 1)
+				errx(1, "Option %s: invalid short option"
+				     " '%.*s'", entry->names, len+1, p-1);
+			if (*p == ':')
+				errx(1, "Option %s: invalid short option '-:'",
+				     entry->names);
 			opt_num_short++;
-			if (entry->flags == OPT_HASARG) {
+			if (entry->type == OPT_HASARG) {
 				opt_num_short_arg++;
-				/* FIXME: -? with ops breaks getopt_long */
-				assert(*p != '?');
+				if (*p == '?')
+					errx(1, "Option %s: '-?' cannot take"
+					     " an argument", entry->names);
 			}
 		}
 		/* Don't document args unless there are some. */
-		if (entry->flags == OPT_NOARG)
-			assert(p[len] != ' ' && p[len] != '=');
+		if (entry->type == OPT_NOARG) {
+			if (p[len] == ' ' || p[len] == '=')
+				errx(1, "Option %s: does not take arguments"
+				     "'%s'", entry->names, p+len+1);
+		}
 	}
 }
 
@@ -136,7 +154,7 @@ static void add_opt(const struct opt_table *entry)
 	opt_table[opt_count++] = *entry;
 }
 
-void _opt_register(const char *names, enum opt_flags flags,
+void _opt_register(const char *names, enum opt_type type,
 		   char *(*cb)(void *arg),
 		   char *(*cb_arg)(const char *optarg, void *arg),
 		   void (*show)(char buf[OPT_SHOW_LEN], const void *arg),
@@ -144,7 +162,7 @@ void _opt_register(const char *names, enum opt_flags flags,
 {
 	struct opt_table opt;
 	opt.names = names;
-	opt.flags = flags;
+	opt.type = type;
 	opt.cb = cb;
 	opt.cb_arg = cb_arg;
 	opt.show = show;
@@ -162,8 +180,8 @@ void opt_register_table(const struct opt_table entry[], const char *desc)
 		struct opt_table heading = OPT_SUBTABLE(NULL, desc);
 		add_opt(&heading);
 	}
-	for (i = 0; entry[i].flags != OPT_END; i++) {
-		if (entry[i].flags == OPT_SUBTABLE)
+	for (i = 0; entry[i].type != OPT_END; i++) {
+		if (entry[i].type == OPT_SUBTABLE)
 			opt_register_table(subtable_of(&entry[i]),
 					   entry[i].desc);
 		else {
@@ -186,7 +204,7 @@ static char *make_optstring(void)
 	str[num++] = ':';
 	for (p = first_sopt(&i); p; p = next_sopt(p, &i)) {
 		str[num++] = *p;
-		if (opt_table[i].flags == OPT_HASARG)
+		if (opt_table[i].type == OPT_HASARG)
 			str[num++] = ':';
 	}
 	str[num++] = '\0';
@@ -197,7 +215,7 @@ static char *make_optstring(void)
 static struct option *make_options(void)
 {
 	struct option *options = malloc(sizeof(*options) * (opt_num_long + 1));
-	unsigned int i, num = 0, len;
+	unsigned int i, num = 0, len = 0 /* GCC bogus warning */;
 	const char *p;
 
 	for (p = first_lopt(&i, &len); p; p = next_lopt(p, &i, &len)) {
@@ -205,7 +223,7 @@ static struct option *make_options(void)
 		memcpy(buf, p, len);
 		buf[len] = 0;
 		options[num].name = buf;
-		options[num].has_arg = (opt_table[i].flags == OPT_HASARG);
+		options[num].has_arg = (opt_table[i].type == OPT_HASARG);
 		options[num].flag = NULL;
 		options[num].val = 0;
 		num++;
@@ -278,7 +296,7 @@ bool opt_parse(int *argc, char *argv[], void (*errlog)(const char *fmt, ...))
 	while ((ret = getopt_long(*argc, argv, optstring, options, &longidx))
 	       != -1) {
 		char *problem;
-		const char *name;
+		const char *name = NULL; /* GCC bogus warning */
 
 		/* optopt is 0 if it's an unknown long option, *or* if
 		 * -? is a valid short option. */
@@ -300,7 +318,7 @@ bool opt_parse(int *argc, char *argv[], void (*errlog)(const char *fmt, ...))
 		else
 			e = find_long(longidx, &name);
 
-		if (e->flags == OPT_HASARG)
+		if (e->type == OPT_HASARG)
 			problem = e->cb_arg(optarg, e->arg);
 		else
 			problem = e->cb(e->arg);