Re: [PATCH v1 1/3] search: Seperately report matching and non-matching authors.

Subject: Re: [PATCH v1 1/3] search: Seperately report matching and non-matching authors.

Date: Fri, 24 Oct 2014 11:21:54 +0100

To: Mark Walters, notmuch@notmuchmail.org

Cc:

From: David Edmondson


On Fri, Oct 24 2014, Mark Walters wrote:
> What about having both authors_matched and authors_not_matched as lists
> of authors (ie one string for each author)?

That's a sensible idea. I will look into it.

> Then emacs, for example, wouldn't try and parse the string back into
> authors before splitting.

It doesn't really do that other than to decide where to truncate the
visible string. That whole chunk of code is horrible.

>>  	first_non_matched_author = 0;
>
> I think I would prefer to make this look like the matched case and drop
> the first_non_matched_author stuff.

Agreed, will do.

Thread: