[PATCH v2 7/8] lib: add NOTMUCH_EXCLUDE_FLAG to notmuch_exclude_t

David Bremner david at tethera.net
Sun May 12 04:20:04 PDT 2013


Mark Walters <markwalters1009 at gmail.com> writes:

> -	    notmuch_query_set_omit_excluded (query, NOTMUCH_EXCLUDE_FALSE);
> +	    notmuch_query_set_omit_excluded (query, NOTMUCH_EXCLUDE_FLAG);

Shouldn't the documentation be updated to cover NOTMUCH_EXCLUDE_FLAG?  I
realize it is not _wrong_ as written, but it is a bit confusing not to
cover all possible values of the enum.

Other than that, the series LGTM.

d


More information about the notmuch mailing list