diff options
author | Jani Nikula <jani@nikula.org> | 2017-10-01 23:53:11 +0300 |
---|---|---|
committer | David Bremner <david@tethera.net> | 2017-10-04 22:00:42 -0300 |
commit | 4a6721970a42a9f86149fb5d395d1001fed2d305 (patch) | |
tree | 7ee198f5d38f841a580f9b9e30cce5999e8e74cd /notmuch-dump.c | |
parent | d57da17fcd30a5cbe191ccd42906e489dd2bd6a3 (diff) |
cli: use designated initializers for opt desc
Several changes at once, just to not have to change the same lines
several times over:
- Use designated initializers to initialize opt desc arrays.
- Only initialize the needed fields.
- Remove arg_id (short options) as unused.
- Replace opt_type and output_var with several type safe output
variables, where the output variable being non-NULL determines the
type. Introduce checks to ensure only one is set. The downside is
some waste of const space per argument; this could be saved by
retaining opt_type and using a union, but that's still pretty
verbose.
- Fix some variables due to the type safety. Mostly a good thing, but
leads to some enums being changed to ints. This is pedantically
correct, but somewhat annoying. We could also cast, but that defeats
the purpose a bit.
- Terminate the opt desc arrays using {}.
The output variable type safety and the ability to add new fields for
just some output types or arguments are the big wins. For example, if
we wanted to add a variable to set when the argument is present, we
could do so for just the arguments that need it.
Beauty is in the eye of the beholder, but I think this looks nice when
defining the arguments, and reduces some of the verbosity we have
there.
Diffstat (limited to 'notmuch-dump.c')
-rw-r--r-- | notmuch-dump.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/notmuch-dump.c b/notmuch-dump.c index 5cc3b2f6..03e64d60 100644 --- a/notmuch-dump.c +++ b/notmuch-dump.c @@ -369,7 +369,7 @@ notmuch_dump_command (notmuch_config_t *config, int argc, char *argv[]) notmuch_exit_if_unmatched_db_uuid (notmuch); - char *output_file_name = NULL; + const char *output_file_name = NULL; int opt_index; int output_format = DUMP_FORMAT_BATCH_TAG; @@ -377,18 +377,18 @@ notmuch_dump_command (notmuch_config_t *config, int argc, char *argv[]) notmuch_bool_t gzip_output = 0; notmuch_opt_desc_t options[] = { - { NOTMUCH_OPT_KEYWORD, &output_format, "format", 'f', + { .opt_keyword = &output_format, .name = "format", .keywords = (notmuch_keyword_t []){ { "sup", DUMP_FORMAT_SUP }, { "batch-tag", DUMP_FORMAT_BATCH_TAG }, { 0, 0 } } }, - { NOTMUCH_OPT_KEYWORD_FLAGS, &include, "include", 'I', + { .opt_flags = &include, .name = "include", .keywords = (notmuch_keyword_t []){ { "config", DUMP_INCLUDE_CONFIG }, { "properties", DUMP_INCLUDE_PROPERTIES }, { "tags", DUMP_INCLUDE_TAGS} } }, - { NOTMUCH_OPT_STRING, &output_file_name, "output", 'o', 0 }, - { NOTMUCH_OPT_BOOLEAN, &gzip_output, "gzip", 'z', 0 }, - { NOTMUCH_OPT_INHERIT, (void *) ¬much_shared_options, NULL, 0, 0 }, - { 0, 0, 0, 0, 0 } + { .opt_string = &output_file_name, .name = "output" }, + { .opt_bool = &gzip_output, .name = "gzip" }, + { .opt_inherit = notmuch_shared_options }, + { } }; opt_index = parse_arguments (argc, argv, options, 1); |