ee/ddb78b32e39df508b8d007275c25a0a653abdc

   1 Return-Path: <amdragon@mit.edu>\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 B62EA429E54\r
   6         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 21:08:07 -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: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 66RznvDQNUI8 for <notmuch@notmuchmail.org>;\r
  16         Sun, 22 Jan 2012 21:08:07 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-8.mit.edu (DMZ-MAILSEC-SCANNER-8.MIT.EDU\r
  18         [18.7.68.37])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 268E3429E40\r
  20         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 21:08:07 -0800 (PST)\r
  21 X-AuditID: 12074425-b7f4a6d0000008e0-93-4f1ceb36bfd1\r
  22 Received: from mailhub-auth-4.mit.edu ( [18.7.62.39])\r
  23         by dmz-mailsec-scanner-8.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 4E.4A.02272.63BEC1F4; Mon, 23 Jan 2012 00:08:06 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-4.mit.edu (8.13.8/8.9.2) with ESMTP id q0N585G7023710; \r
  27         Mon, 23 Jan 2012 00:08:06 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q0N584kS001140\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Mon, 23 Jan 2012 00:08:05 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1RpC81-0002NU-Dj; Mon, 23 Jan 2012 00:07:37 -0500\r
  37 Date: Mon, 23 Jan 2012 00:07:37 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Pieter Praet <pieter@praet.org>\r
  40 Subject: Re: [PATCH v2 4/6] setup: move tag printing and parsing into\r
  41         separate functions\r
  42 Message-ID: <20120123050737.GT16740@mit.edu>\r
  43 References: <87obtvaw8h.fsf@praet.org>\r
  44         <1327292557-7701-4-git-send-email-pieter@praet.org>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: text/plain; charset=us-ascii\r
  47 Content-Disposition: inline\r
  48 In-Reply-To: <1327292557-7701-4-git-send-email-pieter@praet.org>\r
  49 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  50 X-Brightmail-Tracker:\r
  51  H4sIAAAAAAAAA+NgFmpileLIzCtJLcpLzFFi42IRYrdT1zV7LeNvMPGpjsX1mzOZLX6/vsHs\r
  52         wOTxbNUtZo+OfZdZA5iiuGxSUnMyy1KL9O0SuDL2/P/DUvBLrOLxi5AGxrmCXYycHBICJhJH\r
  53         Dk5kgrDFJC7cW8/WxcjFISSwj1Fi2r3NrBDOBkaJjwsPs0M4J5kkDn+6wAThLGGUOLN3MgtI\r
  54         P4uAqsTGVd/AZrEJaEhs27+cEcQWEVCWOP3kJzuIzSygJbF14wewuLBAlMTbRa+AVnBw8Aro\r
  55         SLy97wQSFhKIkbix8wUziM0rIChxcuYTFpjWG/9eMoGUMwtISyz/xwES5hRwkPg15zhYuaiA\r
  56         isSUk9vYJjAKzULSPQtJ9yyE7gWMzKsYZVNyq3RzEzNzilOTdYuTE/PyUot0LfRyM0v0UlNK\r
  57         NzGCg9pFdQfjhENKhxgFOBiVeHgjlsr4C7EmlhVX5h5ilORgUhLl3foCKMSXlJ9SmZFYnBFf\r
  58         VJqTWnyIUYKDWUmE1+gxUI43JbGyKrUoHyYlzcGiJM6rqfXOT0ggPbEkNTs1tSC1CCYrw8Gh\r
  59         JMF7/RVQo2BRanpqRVpmTglCmomDE2Q4D9DwJpAa3uKCxNzizHSI/ClGRSlx3mUgCQGQREZp\r
  60         HlwvLOm8YhQHekWYtxmkigeYsOC6XwENZgIazJEnBTK4JBEhJdXAuK93osnjXflOC/Mf6L/8\r
  61         maTYqnbsjtdzA7nIbfzrrcpjayNXHcwT3rtupkgHb6CgKptlulL7o6N2298snGsnwM9nOP3a\r
  62         XU2zvIeaPnvTHN9+7Y45tf7q4iWWyakxeUs5kp7uPLbIbXdx9I2N5U88Dx3f+fxQ8d5zzjkH\r
  63         b3g9Xxs2hUtcrGmDsBJLcUaioRZzUXEiABwQh48VAwAA\r
  64 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  65 X-BeenThere: notmuch@notmuchmail.org\r
  66 X-Mailman-Version: 2.1.13\r
  67 Precedence: list\r
  68 List-Id: "Use and development of the notmuch mail system."\r
  69         <notmuch.notmuchmail.org>\r
  70 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  72 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  73 List-Post: <mailto:notmuch@notmuchmail.org>\r
  74 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  75 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  76         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  77 X-List-Received-Date: Mon, 23 Jan 2012 05:08:07 -0000\r
  78 \r
  79 Quoth Pieter Praet on Jan 23 at  5:22 am:\r
  80 > From: Austin Clements <amdragon@MIT.EDU>\r
  81 > \r
  82 > * notmuch-setup.c (notmuch_setup_command):\r
  83 >   Break tag printing and response parsing out into separate functions\r
  84 >   called `print_tag_list' respectively `parse_tag_list', for reuse\r
  85 >   with the 'search.exclude_tags' option.\r
  86 \r
  87 Since I'm revising this patch a little bit anyway, how about\r
  88 \r
  89 setup: Create functions for tag list printing and parsing\r
  90 \r
  91 This refactors the tag list printing and parsing currently used for\r
  92 new.tags so that both can be reused for the new search.exclude_tags\r
  93 option.\r
  94 \r
  95 > \r
  96 > ---\r
  97 >  notmuch-setup.c |   55 ++++++++++++++++++++++++++++++++++---------------------\r
  98 >  1 files changed, 34 insertions(+), 21 deletions(-)\r
  99 > \r
 100 > diff --git a/notmuch-setup.c b/notmuch-setup.c\r
 101 > index c3ea937..dcfa607 100644\r
 102 > --- a/notmuch-setup.c\r
 103 > +++ b/notmuch-setup.c\r
 104 > @@ -87,6 +87,38 @@ welcome_message_post_setup (void)\r
 105 >  "have sufficient storage space available now.\n\n");\r
 106 >  }\r
 107 >  \r
 108 > +static void\r
 109 > +print_tag_list (const char **tags, size_t tags_len)\r
 110 > +{\r
 111 > +    unsigned int i;\r
 112 > +    for (i = 0; i < tags_len; i++) {\r
 113 > +     if (i != 0)\r
 114 > +         printf (" ");\r
 115 > +     printf ("%s", tags[i]);\r
 116 > +    }\r
 117 > +}\r
 118 > +\r
 119 > +static GPtrArray *\r
 120 > +parse_tag_list (void *ctx, char *response)\r
 121 > +{\r
 122 > +    GPtrArray *tags = g_ptr_array_new ();\r
 123 > +    char *tag = response;\r
 124 > +    char *space;\r
 125 > +\r
 126 > +    while (tag && *tag) {\r
 127 > +     space = strchr (tag, ' ');\r
 128 > +     if (space)\r
 129 > +         g_ptr_array_add (tags, talloc_strndup (ctx, tag, space - tag));\r
 130 > +     else\r
 131 > +         g_ptr_array_add (tags, talloc_strdup (ctx, tag));\r
 132 > +     tag = space;\r
 133 > +     while (tag && *tag == ' ')\r
 134 > +         tag++;\r
 135 > +    }\r
 136 > +\r
 137 > +    return tags;\r
 138 > +}\r
 139 > +\r
 140 >  int\r
 141 >  notmuch_setup_command (unused (void *ctx),\r
 142 >                      unused (int argc), unused (char *argv[]))\r
 143 > @@ -164,30 +196,11 @@ notmuch_setup_command (unused (void *ctx),\r
 144 >      new_tags = notmuch_config_get_new_tags (config, &new_tags_len);\r
 145 >  \r
 146 >      printf ("Tags to apply to all new messages (separated by spaces) [");\r
 147 > -\r
 148 > -    for (i = 0; i < new_tags_len; i++) {\r
 149 > -     if (i != 0)\r
 150 > -         printf (" ");\r
 151 > -     printf ("%s", new_tags[i]);\r
 152 > -    }\r
 153 > -\r
 154 > +    print_tag_list(new_tags, new_tags_len);\r
 155 \r
 156 Missing space before paren.  (Sorry, my fault.)\r
 157 \r
 158 >      prompt ("]: ");\r
 159 >  \r
 160 >      if (strlen (response)) {\r
 161 > -     GPtrArray *tags = g_ptr_array_new ();\r
 162 > -     char *tag = response;\r
 163 > -     char *space;\r
 164 > -\r
 165 > -     while (tag && *tag) {\r
 166 > -         space = strchr (tag, ' ');\r
 167 > -         if (space)\r
 168 > -             g_ptr_array_add (tags, talloc_strndup (ctx, tag, space - tag));\r
 169 > -         else\r
 170 > -             g_ptr_array_add (tags, talloc_strdup (ctx, tag));\r
 171 > -         tag = space;\r
 172 > -         while (tag && *tag == ' ')\r
 173 > -             tag++;\r
 174 > -     }\r
 175 > +     GPtrArray *tags = parse_tag_list (ctx, response);\r
 176 >  \r
 177 >       notmuch_config_set_new_tags (config, (const char **) tags->pdata,\r
 178 >                                    tags->len);\r