setup: Create functions for tag list printing and parsing
authorAustin Clements <amdragon@MIT.EDU>
Mon, 23 Jan 2012 05:50:45 +0000 (06:50 +0100)
committerDavid Bremner <bremner@debian.org>
Mon, 23 Jan 2012 23:22:57 +0000 (19:22 -0400)
This refactors the tag list printing and parsing currently used for
new.tags so that both can be reused for the new search.exclude_tags
option.

notmuch-setup.c

index c3ea9371c13cbc4058922fbdc920a5755e4084f0..f85e0ebd091ba1dafc70c2f9b167d6623c4a471e 100644 (file)
@@ -87,6 +87,38 @@ welcome_message_post_setup (void)
 "have sufficient storage space available now.\n\n");
 }
 
 "have sufficient storage space available now.\n\n");
 }
 
+static void
+print_tag_list (const char **tags, size_t tags_len)
+{
+    unsigned int i;
+    for (i = 0; i < tags_len; i++) {
+       if (i != 0)
+           printf (" ");
+       printf ("%s", tags[i]);
+    }
+}
+
+static GPtrArray *
+parse_tag_list (void *ctx, char *response)
+{
+    GPtrArray *tags = g_ptr_array_new ();
+    char *tag = response;
+    char *space;
+
+    while (tag && *tag) {
+       space = strchr (tag, ' ');
+       if (space)
+           g_ptr_array_add (tags, talloc_strndup (ctx, tag, space - tag));
+       else
+           g_ptr_array_add (tags, talloc_strdup (ctx, tag));
+       tag = space;
+       while (tag && *tag == ' ')
+           tag++;
+    }
+
+    return tags;
+}
+
 int
 notmuch_setup_command (unused (void *ctx),
                       unused (int argc), unused (char *argv[]))
 int
 notmuch_setup_command (unused (void *ctx),
                       unused (int argc), unused (char *argv[]))
@@ -164,30 +196,11 @@ notmuch_setup_command (unused (void *ctx),
     new_tags = notmuch_config_get_new_tags (config, &new_tags_len);
 
     printf ("Tags to apply to all new messages (separated by spaces) [");
     new_tags = notmuch_config_get_new_tags (config, &new_tags_len);
 
     printf ("Tags to apply to all new messages (separated by spaces) [");
-
-    for (i = 0; i < new_tags_len; i++) {
-       if (i != 0)
-           printf (" ");
-       printf ("%s", new_tags[i]);
-    }
-
+    print_tag_list (new_tags, new_tags_len);
     prompt ("]: ");
 
     if (strlen (response)) {
     prompt ("]: ");
 
     if (strlen (response)) {
-       GPtrArray *tags = g_ptr_array_new ();
-       char *tag = response;
-       char *space;
-
-       while (tag && *tag) {
-           space = strchr (tag, ' ');
-           if (space)
-               g_ptr_array_add (tags, talloc_strndup (ctx, tag, space - tag));
-           else
-               g_ptr_array_add (tags, talloc_strdup (ctx, tag));
-           tag = space;
-           while (tag && *tag == ' ')
-               tag++;
-       }
+       GPtrArray *tags = parse_tag_list (ctx, response);
 
        notmuch_config_set_new_tags (config, (const char **) tags->pdata,
                                     tags->len);
 
        notmuch_config_set_new_tags (config, (const char **) tags->pdata,
                                     tags->len);