count: Convert to new-style argument parsing
authorAustin Clements <amdragon@MIT.EDU>
Fri, 13 Jan 2012 23:07:02 +0000 (18:07 -0500)
committerDavid Bremner <bremner@debian.org>
Sat, 14 Jan 2012 01:43:54 +0000 (21:43 -0400)
notmuch-count.c

index 20ce33426c7072cf3bf6688a07083d5c89b1f328..0982f99f3d9ffb2e8e6801bb43208175cb52d5a0 100644 (file)
 
 #include "notmuch-client.h"
 
 
 #include "notmuch-client.h"
 
+enum {
+    OUTPUT_THREADS,
+    OUTPUT_MESSAGES,
+};
+
 int
 notmuch_count_command (void *ctx, int argc, char *argv[])
 {
 int
 notmuch_count_command (void *ctx, int argc, char *argv[])
 {
@@ -28,34 +33,22 @@ notmuch_count_command (void *ctx, int argc, char *argv[])
     notmuch_database_t *notmuch;
     notmuch_query_t *query;
     char *query_str;
     notmuch_database_t *notmuch;
     notmuch_query_t *query;
     char *query_str;
-    int i;
-    notmuch_bool_t output_messages = TRUE;
+    int opt_index;
+    int output = OUTPUT_MESSAGES;
 
 
-    argc--; argv++; /* skip subcommand argument */
+    notmuch_opt_desc_t options[] = {
+       { NOTMUCH_OPT_KEYWORD, &output, "output", 'o',
+         (notmuch_keyword_t []){ { "threads", OUTPUT_THREADS },
+                                 { "messages", OUTPUT_MESSAGES },
+                                 { 0, 0 } } },
+       { 0, 0, 0, 0, 0 }
+    };
 
 
-    for (i = 0; i < argc && argv[i][0] == '-'; i++) {
-       if (strcmp (argv[i], "--") == 0) {
-           i++;
-           break;
-       }
-       if (STRNCMP_LITERAL (argv[i], "--output=") == 0) {
-           const char *opt = argv[i] + sizeof ("--output=") - 1;
-           if (strcmp (opt, "threads") == 0) {
-               output_messages = FALSE;
-           } else if (strcmp (opt, "messages") == 0) {
-               output_messages = TRUE;
-           } else {
-               fprintf (stderr, "Invalid value for --output: %s\n", opt);
-               return 1;
-           }
-       } else {
-           fprintf (stderr, "Unrecognized option: %s\n", argv[i]);
-           return 1;
-       }
-    }
+    opt_index = parse_arguments (argc, argv, options, 1);
 
 
-    argc -= i;
-    argv += i;
+    if (opt_index < 0) {
+       return 1;
+    }
 
     config = notmuch_config_open (ctx, NULL, NULL);
     if (config == NULL)
 
     config = notmuch_config_open (ctx, NULL, NULL);
     if (config == NULL)
@@ -66,7 +59,7 @@ notmuch_count_command (void *ctx, int argc, char *argv[])
     if (notmuch == NULL)
        return 1;
 
     if (notmuch == NULL)
        return 1;
 
-    query_str = query_string_from_args (ctx, argc, argv);
+    query_str = query_string_from_args (ctx, argc-opt_index, argv+opt_index);
     if (query_str == NULL) {
        fprintf (stderr, "Out of memory.\n");
        return 1;
     if (query_str == NULL) {
        fprintf (stderr, "Out of memory.\n");
        return 1;
@@ -82,10 +75,14 @@ notmuch_count_command (void *ctx, int argc, char *argv[])
        return 1;
     }
 
        return 1;
     }
 
-    if (output_messages)
+    switch (output) {
+    case OUTPUT_MESSAGES:
        printf ("%u\n", notmuch_query_count_messages (query));
        printf ("%u\n", notmuch_query_count_messages (query));
-    else
+       break;
+    case OUTPUT_THREADS:
        printf ("%u\n", notmuch_query_count_threads (query));
        printf ("%u\n", notmuch_query_count_threads (query));
+       break;
+    }
 
     notmuch_query_destroy (query);
     notmuch_database_close (notmuch);
 
     notmuch_query_destroy (query);
     notmuch_database_close (notmuch);