1f/aa9a16387c17071f43b5c0c2f8a567fe3da853

   1 Return-Path: <bremner@tethera.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id B18ED429E32\r
   6         for <notmuch@notmuchmail.org>; Wed,  7 Dec 2011 04:34:58 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id gU9+8lnZ4FEe for <notmuch@notmuchmail.org>;\r
  16         Wed,  7 Dec 2011 04:34:57 -0800 (PST)\r
  17 Received: from tempo.its.unb.ca (tempo.its.unb.ca [131.202.1.21])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 7BF96429E26\r
  21         for <notmuch@notmuchmail.org>; Wed,  7 Dec 2011 04:34:57 -0800 (PST)\r
  22 Received: from zancas.localnet\r
  23         (fctnnbsc36w-156034079193.pppoe-dynamic.High-Speed.nb.bellaliant.net\r
  24         [156.34.79.193]) (authenticated bits=0)\r
  25         by tempo.its.unb.ca (8.13.8/8.13.8) with ESMTP id pB7CYrUg007122\r
  26         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO);\r
  27         Wed, 7 Dec 2011 08:34:53 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.77)\r
  29         (envelope-from <bremner@tethera.net>)\r
  30         id 1RYGi4-0007n2-SG; Wed, 07 Dec 2011 08:34:52 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: notmuch@notmuchmail.org\r
  33 Subject: [PATCH 4/4] notmuch-search: convert to command-line-arguments\r
  34 Date: Wed,  7 Dec 2011 08:34:40 -0400\r
  35 Message-Id: <1323261280-29907-4-git-send-email-david@tethera.net>\r
  36 X-Mailer: git-send-email 1.7.7.3\r
  37 In-Reply-To: <1323261280-29907-1-git-send-email-david@tethera.net>\r
  38 References: <87iplso9c9.fsf@zancas.localnet>\r
  39         <1323261280-29907-1-git-send-email-david@tethera.net>\r
  40 Cc: David Bremner <bremner@debian.org>\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Wed, 07 Dec 2011 12:34:59 -0000\r
  54 \r
  55 From: David Bremner <bremner@debian.org>\r
  56 \r
  57 The switch on format_sel is slightly clunky, but it doesn't seem worth\r
  58 special casing argument processing for function pointers, when I think\r
  59 the function pointer approach will be modified/abandoned.\r
  60 ---\r
  61  notmuch-search.c |  109 ++++++++++++++++++++---------------------------------\r
  62  1 files changed, 41 insertions(+), 68 deletions(-)\r
  63 \r
  64 diff --git a/notmuch-search.c b/notmuch-search.c\r
  65 index 36686d1..2ae7597 100644\r
  66 --- a/notmuch-search.c\r
  67 +++ b/notmuch-search.c\r
  68 @@ -415,81 +415,54 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
  69      notmuch_database_t *notmuch;\r
  70      notmuch_query_t *query;\r
  71      char *query_str;\r
  72 -    char *opt;\r
  73      notmuch_sort_t sort = NOTMUCH_SORT_NEWEST_FIRST;\r
  74      const search_format_t *format = &format_text;\r
  75 -    int i, ret;\r
  76 +    int opt_index, ret;\r
  77      output_t output = OUTPUT_SUMMARY;\r
  78      int offset = 0;\r
  79      int limit = -1; /* unlimited */\r
  80  \r
  81 -    argc--; argv++; /* skip subcommand argument */\r
  82 -\r
  83 -    for (i = 0; i < argc && argv[i][0] == '-'; i++) {\r
  84 -       if (strcmp (argv[i], "--") == 0) {\r
  85 -           i++;\r
  86 -           break;\r
  87 -       }\r
  88 -        if (STRNCMP_LITERAL (argv[i], "--sort=") == 0) {\r
  89 -           opt = argv[i] + sizeof ("--sort=") - 1;\r
  90 -           if (strcmp (opt, "oldest-first") == 0) {\r
  91 -               sort = NOTMUCH_SORT_OLDEST_FIRST;\r
  92 -           } else if (strcmp (opt, "newest-first") == 0) {\r
  93 -               sort = NOTMUCH_SORT_NEWEST_FIRST;\r
  94 -           } else {\r
  95 -               fprintf (stderr, "Invalid value for --sort: %s\n", opt);\r
  96 -               return 1;\r
  97 -           }\r
  98 -       } else if (STRNCMP_LITERAL (argv[i], "--offset=") == 0) {\r
  99 -           char *p;\r
 100 -           opt = argv[i] + sizeof ("--offset=") - 1;\r
 101 -           offset = strtol (opt, &p, 10);\r
 102 -           if (*opt == '\0' || p == opt || *p != '\0') {\r
 103 -               fprintf (stderr, "Invalid value for --offset: %s\n", opt);\r
 104 -               return 1;\r
 105 -           }\r
 106 -       } else if (STRNCMP_LITERAL (argv[i], "--limit=") == 0) {\r
 107 -           char *p;\r
 108 -           opt = argv[i] + sizeof ("--limit=") - 1;\r
 109 -           limit = strtoul (opt, &p, 10);\r
 110 -           if (*opt == '\0' || p == opt || *p != '\0') {\r
 111 -               fprintf (stderr, "Invalid value for --limit: %s\n", opt);\r
 112 -               return 1;\r
 113 -           }\r
 114 -       } else if (STRNCMP_LITERAL (argv[i], "--format=") == 0) {\r
 115 -           opt = argv[i] + sizeof ("--format=") - 1;\r
 116 -           if (strcmp (opt, "text") == 0) {\r
 117 -               format = &format_text;\r
 118 -           } else if (strcmp (opt, "json") == 0) {\r
 119 -               format = &format_json;\r
 120 -           } else {\r
 121 -               fprintf (stderr, "Invalid value for --format: %s\n", opt);\r
 122 -               return 1;\r
 123 -           }\r
 124 -       } else if (STRNCMP_LITERAL (argv[i], "--output=") == 0) {\r
 125 -           opt = argv[i] + sizeof ("--output=") - 1;\r
 126 -           if (strcmp (opt, "summary") == 0) {\r
 127 -               output = OUTPUT_SUMMARY;\r
 128 -           } else if (strcmp (opt, "threads") == 0) {\r
 129 -               output = OUTPUT_THREADS;\r
 130 -           } else if (strcmp (opt, "messages") == 0) {\r
 131 -               output = OUTPUT_MESSAGES;\r
 132 -           } else if (strcmp (opt, "files") == 0) {\r
 133 -               output = OUTPUT_FILES;\r
 134 -           } else if (strcmp (opt, "tags") == 0) {\r
 135 -               output = OUTPUT_TAGS;\r
 136 -           } else {\r
 137 -               fprintf (stderr, "Invalid value for --output: %s\n", opt);\r
 138 -               return 1;\r
 139 -           }\r
 140 -       } else {\r
 141 -           fprintf (stderr, "Unrecognized option: %s\n", argv[i]);\r
 142 -           return 1;\r
 143 -       }\r
 144 +    enum { NOTMUCH_FORMAT_JSON, NOTMUCH_FORMAT_TEXT }\r
 145 +       format_sel = NOTMUCH_FORMAT_TEXT;\r
 146 +\r
 147 +    notmuch_opt_desc_t options[] = {\r
 148 +       { "sort", 's', NOTMUCH_OPT_KEYWORD,\r
 149 +         (notmuch_keyword_t []){ { "oldest-first", NOTMUCH_SORT_OLDEST_FIRST },\r
 150 +                                 { "newest-first", NOTMUCH_SORT_NEWEST_FIRST },\r
 151 +                                 {0, 0} },\r
 152 +         &sort },\r
 153 +       { "format", 'f', NOTMUCH_OPT_KEYWORD,\r
 154 +         (notmuch_keyword_t []){ { "json", NOTMUCH_FORMAT_JSON },\r
 155 +                                 { "text", NOTMUCH_FORMAT_TEXT },\r
 156 +                                 {0, 0} },\r
 157 +         &format_sel },\r
 158 +       { "output", 'o', NOTMUCH_OPT_KEYWORD,\r
 159 +         (notmuch_keyword_t []){ { "summary", OUTPUT_SUMMARY },\r
 160 +                                 { "threads", OUTPUT_THREADS },\r
 161 +                                 { "messages", OUTPUT_MESSAGES },\r
 162 +                                 { "files", OUTPUT_FILES },\r
 163 +                                 { "tags", OUTPUT_TAGS },\r
 164 +                                 {0, 0} },\r
 165 +         &output },\r
 166 +       { "offset", 'O', NOTMUCH_OPT_INT, 0, &offset },\r
 167 +       { "limit", 'L', NOTMUCH_OPT_INT, 0, &limit },\r
 168 +       { 0, 0, 0, 0, 0 }\r
 169 +    };\r
 170 +\r
 171 +    opt_index = parse_arguments (argc, argv, options, 1);\r
 172 +\r
 173 +    if (opt_index < 0) {\r
 174 +       exit(1);\r
 175      }\r
 176  \r
 177 -    argc -= i;\r
 178 -    argv += i;\r
 179 +    switch (format_sel) {\r
 180 +    case NOTMUCH_FORMAT_TEXT:\r
 181 +       format = &format_text;\r
 182 +       break;\r
 183 +    case NOTMUCH_FORMAT_JSON:\r
 184 +       format = &format_json;\r
 185 +       break;\r
 186 +    }\r
 187  \r
 188      config = notmuch_config_open (ctx, NULL, NULL);\r
 189      if (config == NULL)\r
 190 @@ -500,7 +473,7 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
 191      if (notmuch == NULL)\r
 192         return 1;\r
 193  \r
 194 -    query_str = query_string_from_args (notmuch, argc, argv);\r
 195 +    query_str = query_string_from_args (notmuch, argc-opt_index, argv+opt_index);\r
 196      if (query_str == NULL) {\r
 197         fprintf (stderr, "Out of memory.\n");\r
 198         return 1;\r
 199 -- \r
 200 1.7.7.3\r
 201 \r