Mark Walters writes:
> This patch just renames the internal variables for the JSON parser now
> it is no longer specific to search mode. It also fixes up the white
> space after the previous patch. There should be no functional changes.
This series looks very good to me. I still have a couple
Quoth Jani Nikula on Oct 22 at 12:22 am:
> Add a custom value range processor to enable date and time searches of
> the form date:since..until, where "since" and "until" are expressions
> understood by the previously added date/time parser, to restrict the
> results to messages within a particular
Quoth Jani Nikula on Oct 22 at 12:22 am:
> Test the date/time parser module directly, independent of notmuch,
> using the parse-time test tool.
>
> Credits to Michal Sojka for writing most of the
> tests.
> ---
> test/notmuch-test |1 +
> test/parse-time-string | 71
>
This series splits the async json parser into its own function. The
previous version is at id:87k3xlyoek@qmul.ac.uk.
This version splits that patch into three bits: the first contains all
the actual changes, the second changes some variable names and fixes
the whitespace, and the third just
This patch splits out the incremental json parser into its own
function.
It moves the main logic of the parser to happen inside the parse
buffer rather than inside the results buffer, but makes sure all
results and all errors are displayed in the results buffer.
It also changes the local parser
This patch just renames the internal variables for the JSON parser now
it is no longer specific to search mode. It also fixes up the white
space after the previous patch. There should be no functional changes.
---
emacs/notmuch.el | 46 +++---
1 files
This just moves the notmuch-search-process-filter after the newly
split out incremental json parser. I think this removes a warning in
some versions of emacs.
There should be no functional change.
---
emacs/notmuch.el | 34 +-
1 files changed, 17 insertions(+),
Mark Walters markwalters1...@gmail.com writes:
This patch just renames the internal variables for the JSON parser now
it is no longer specific to search mode. It also fixes up the white
space after the previous patch. There should be no functional changes.
This series looks very good to me. I