From 879d526d7ee7ee7a876f284dac76075e7a7267ae Mon Sep 17 00:00:00 2001 From: katherine Date: Mon, 27 May 2019 18:45:21 -0700 Subject: move stores vals to named union for c99 compatibility --- doc/example.c | 14 +++++++------- doc/interface.md | 20 ++++++++++---------- 2 files changed, 17 insertions(+), 17 deletions(-) (limited to 'doc') diff --git a/doc/example.c b/doc/example.c index 324473e..71ce24b 100644 --- a/doc/example.c +++ b/doc/example.c @@ -63,32 +63,32 @@ int main(int argc, char **argv) if (options[i].arg_is_stored) { switch (options[i].type) { case SIMPLE_OPT_BOOL: - printf(", val: %s", options[i].val_bool ? "true" : "false"); + printf(", val: %s", options[i].val.v_bool ? "true" : "false"); break; case SIMPLE_OPT_INT: - printf(", val: %ld", options[i].val_int); + printf(", val: %ld", options[i].val.v_int); break; case SIMPLE_OPT_UNSIGNED: - printf(", val: %lu", options[i].val_unsigned); + printf(", val: %lu", options[i].val.v_unsigned); break; case SIMPLE_OPT_DOUBLE: - printf(", val: %lf", options[i].val_double); + printf(", val: %lf", options[i].val.v_double); break; case SIMPLE_OPT_CHAR: - printf(", val: %c", options[i].val_char); + printf(", val: %c", options[i].val.v_char); break; case SIMPLE_OPT_STRING: - printf(", val: %s", options[i].val_string); + printf(", val: %s", options[i].val.v_string); break; case SIMPLE_OPT_STRING_SET: printf(", val: %s", - options[i].string_set[options[i].val_string_set_idx]); + options[i].string_set[options[i].val.v_string_set_idx]); break; default: diff --git a/doc/interface.md b/doc/interface.md index f8d5886..978ecb3 100644 --- a/doc/interface.md +++ b/doc/interface.md @@ -52,21 +52,21 @@ the fields which are set by `simple_opt_parse` are: bool arg_is_stored; union { - bool val_bool; - long val_int; - unsigned long val_unsigned; - double val_double; - char val_char; - char val_string[SIMPLE_OPT_OPT_ARG_MAX_WIDTH]; - int val_string_set_idx; - }; + bool v_bool; + long v_int; + unsigned long v_unsigned; + double v_double; + char v_char; + char v_string[SIMPLE_OPT_OPT_ARG_MAX_WIDTH]; + int v_string_set_idx; + } val; ``` `was_seen` indicates if this option was encountered during parsing, -`arg_is_stored` if an argument was passed to the option, and the `val_` +`arg_is_stored` if an argument was passed to the option, and the `val.v_` fields contain the value passed (with the correct field to set being determined by the `type` field shown above) for all but `SIMPLE_OPT_STRING_SET`, for which -`val_string_set_idx` is set, an index into the `string_set` field's array, +`val.v_string_set_idx` is set, an index into the `string_set` field's array, indicating which possibility was matched. options of the following types: -- cgit v1.2.3