[PATCH v3 07/20] tag-util: do not reset list in parse_tag_command_line

2013-01-21 Thread Jani Nikula
On Sun, 20 Jan 2013, Peter Wang 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

Re: [PATCH v3 07/20] tag-util: do not reset list in parse_tag_command_line

2013-01-21 Thread Jani Nikula
On Sun, 20 Jan 2013, Peter Wang noval...@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.

[PATCH v3 07/20] tag-util: do not reset list in parse_tag_command_line

2013-01-20 Thread Peter Wang
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. --- tag-util.c | 2 -- tag-util.h | 2 ++ 2 files changed, 2

[PATCH v3 07/20] tag-util: do not reset list in parse_tag_command_line

2013-01-19 Thread Peter Wang
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. --- tag-util.c | 2 -- tag-util.h | 2 ++ 2 files changed, 2