[PATCH v3 07/20] tag-util: do not reset list in parse_tag_command_line
Jani Nikula
jani at nikula.org
Mon Jan 21 10:09:04 PST 2013
On Sun, 20 Jan 2013, Peter Wang <novalazy at gmail.com> wrote:
> No current callers of parse_tag_command_line require that it clear its
> tag list argument. The notmuch 'insert' command will be better served
> if the function modifies a pre-populated list (of new.tags) instead of
> clobbering it outright.
I think I'd like parse_tag_command_line() and parse_tag_line() to behave
similarly, both either resetting or not resetting tag list. I think you
could just parse command line first, and add new.tags afterwards, and
you'd be fine without changing parse_tag_command_line().
> ---
> tag-util.c | 2 --
> tag-util.h | 2 ++
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tag-util.c b/tag-util.c
> index 701d329..3f9da05 100644
> --- a/tag-util.c
> +++ b/tag-util.c
> @@ -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++;
> diff --git a/tag-util.h b/tag-util.h
> index 246de85..4628f16 100644
> --- a/tag-util.h
> +++ b/tag-util.h
> @@ -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
> --
> 1.7.12.1
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list