tag-util: do not reset list in parse_tag_command_line
authorPeter Wang <novalazy@gmail.com>
Sun, 23 Jun 2013 04:23:55 +0000 (14:23 +1000)
committerDavid Bremner <bremner@debian.org>
Sat, 29 Jun 2013 17:36:23 +0000 (14:36 -0300)
The 'insert' command will be better served if parse_tag_command_line
modifies a pre-populated list (of new.tags) instead of clobbering the
list outright.  The sole existing caller, notmuch_tag_command, is
unaffected by this change.

tag-util.c
tag-util.h

index 92e08a1d05030099b6439f0b56568bc6a8b40bc5..3bde4097372ab827e91e23ff963f6c5413652f9b 100644 (file)
@@ -165,8 +165,6 @@ parse_tag_command_line (void *ctx, int argc, char **argv,
 
     int i;
 
-    tag_op_list_reset (tag_ops);
-
     for (i = 0; i < argc; i++) {
        if (strcmp (argv[i], "--") == 0) {
            i++;
index 246de85ccb8f2a936f0adc6c50b6638607063f8d..4628f1630ad6263951b8672d5d43a6699281306a 100644 (file)
@@ -81,6 +81,8 @@ parse_tag_line (void *ctx, char *line,
  * Output Parameters:
  *     ops     contains a list of tag operations
  *     query_str the search terms.
+ *
+ * The ops argument is not cleared.
  */
 
 tag_parse_status_t