Re: [PATCH] contrib: pick: use the sexp parser

Subject: Re: [PATCH] contrib: pick: use the sexp parser

Date: Sat, 01 Jun 2013 23:49:45 +0300

To: Mark Walters, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Sat, Jun 01 2013, Mark Walters <markwalters1009@gmail.com> wrote:

> This just converts notmuch-pick to use the sexp parser.
> ---

These trivial changes look good -- is sexp already supported in show ?

Tomi

>
>  contrib/notmuch-pick/notmuch-pick.el |    5 ++---
>  1 files changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el
> index 128fabf..33905d6 100644
> --- a/contrib/notmuch-pick/notmuch-pick.el
> +++ b/contrib/notmuch-pick/notmuch-pick.el
> @@ -773,8 +773,7 @@ Complete list of currently available key bindings:
>          (save-excursion
>            (goto-char (point-max))
>            (insert string))
> -	(notmuch-json-parse-partial-list 'notmuch-pick-insert-forest-thread
> -					 'notmuch-pick-show-error
> +	(notmuch-sexp-parse-partial-list 'notmuch-pick-insert-forest-thread
>  					 results-buf)))))
>  
>  (defun notmuch-pick-worker (basic-query &optional query-context target buffer)
> @@ -796,7 +795,7 @@ Complete list of currently available key bindings:
>      (if notmuch-pick-asynchronous-parser
>  	(let ((proc (start-process
>  		     "notmuch-pick" buffer
> -		     notmuch-command "show" "--body=false" "--format=json"
> +		     notmuch-command "show" "--body=false" "--format=sexp"
>  		     message-arg search-args))
>  	      ;; Use a scratch buffer to accumulate partial output.
>                ;; This buffer will be killed by the sentinel, which
> -- 
> 1.7.9.1
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

Thread: