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

2014-10-24 Thread 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

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

2014-10-24 Thread Mark Walters
Hi I definitely like the idea: some comments below. On Fri, 24 Oct 2014, David Edmondson wrote: > In addition to the :authors attribute of each search result, include > :authors_matched and :authors_non_matched attributes. Both attributes > are always included. If there are no non-matching

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

2014-10-24 Thread David Edmondson
In addition to the :authors attribute of each search result, include :authors_matched and :authors_non_matched attributes. Both attributes are always included. If there are no non-matching authors, the :authors_non_matched attribute is set to the empty string. --- lib/notmuch.h| 34

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

2014-10-24 Thread Austin Clements
Quoth Mark Walters on Oct 24 at 10:23 am: > > Hi > > I definitely like the idea: some comments below. Agreed. > On Fri, 24 Oct 2014, David Edmondson wrote: > > In addition to the :authors attribute of each search result, include > > :authors_matched and :authors_non_matched attributes. Both

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

2014-10-24 Thread David Edmondson
In addition to the :authors attribute of each search result, include :authors_matched and :authors_non_matched attributes. Both attributes are always included. If there are no non-matching authors, the :authors_non_matched attribute is set to the empty string. --- lib/notmuch.h| 34

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

2014-10-24 Thread Mark Walters
Hi I definitely like the idea: some comments below. On Fri, 24 Oct 2014, David Edmondson d...@dme.org wrote: In addition to the :authors attribute of each search result, include :authors_matched and :authors_non_matched attributes. Both attributes are always included. If there are no

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

2014-10-24 Thread 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

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

2014-10-24 Thread Austin Clements
Quoth Mark Walters on Oct 24 at 10:23 am: Hi I definitely like the idea: some comments below. Agreed. On Fri, 24 Oct 2014, David Edmondson d...@dme.org wrote: In addition to the :authors attribute of each search result, include :authors_matched and :authors_non_matched attributes.