[Patch V4] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-11 Thread Mark Walters
Hi > Here is the latest version of this patch set. I think I have fixed most > of the problems raised in review but there are some remaining issues > detailed below. I know this patch set needs rebasing on top of Jani's notmuch-show command line parsing patch: should I do that now or wait for

Re: [Patch V4] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-11 Thread Mark Walters
Hi Here is the latest version of this patch set. I think I have fixed most of the problems raised in review but there are some remaining issues detailed below. I know this patch set needs rebasing on top of Jani's notmuch-show command line parsing patch: should I do that now or wait for

[Patch V4] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-02 Thread Mark Walters
Here is the latest version of this patch set. I think I have fixed most of the problems raised in review but there are some remaining issues detailed below. Changes and queries: 1) Changed --do-not-exclude option to --no-exclude 2) The api notmuch_query_set_omit_excluded_messages remains:

[Patch V4] Add NOTMUCH_MESSAGE_FLAG_EXCLUDED flag

2012-02-02 Thread Mark Walters
Here is the latest version of this patch set. I think I have fixed most of the problems raised in review but there are some remaining issues detailed below. Changes and queries: 1) Changed --do-not-exclude option to --no-exclude 2) The api notmuch_query_set_omit_excluded_messages remains: