[PATCH v3 0/6] Make Emacs search use sexp format

Austin Clements amdragon at MIT.EDU
Fri May 31 17:40:02 PDT 2013


This is v3 of id:1369934016-22308-1-git-send-email-amdragon at mit.edu.
This tweaks the shell invocation as suggested by Tomi and fixes two
comment typos pointed out by Mark.  It also adds a NEWS patch.  I'm
going to go ahead and mark this ready because of Tomi's and Mark's
reviews of v2.

The diff from v2 follows

diff --git a/NEWS b/NEWS
index 80abd97..2a4bde6 100644
--- a/NEWS
+++ b/NEWS
@@ -66,6 +66,17 @@ notmuch-vim, but of course that is their decision.
 Emacs Interface
 ---------------
 
+Better handling of errors in search buffers
+
+  Instead of interleaving errors in search result buffers, search mode
+  now reports errors in the minibuffer.
+
+Faster search results
+
+  Communication between search mode and the notmuch CLI is now more
+  efficient because it uses the CLI's S-expression support.  As a
+  result, search mode should now fill search buffers faster.
+
 No Emacs 22 support
 
   The Emacs 22 support added late 2010 was sufficient only for a short
diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el
index 180f63d..a206808 100644
--- a/emacs/notmuch-lib.el
+++ b/emacs/notmuch-lib.el
@@ -497,8 +497,8 @@ status."
 	 (command (or (executable-find notmuch-command)
 		      (error "command not found: %s" notmuch-command)))
 	 (proc (apply #'start-process name buffer
-		      "sh" "-c"
-		      "ERR=\"$1\"; shift; exec \"$0\" \"$@\" 2>\"$ERR\""
+		      "/bin/sh" "-c"
+		      "exec 2>\"$1\"; shift; exec \"$0\" \"$@\""
 		      command err-file args)))
     (process-put proc 'err-file err-file)
     (process-put proc 'sub-sentinel sentinel)
@@ -533,8 +533,8 @@ status."
 	    (when warnings
 	      (notmuch-logged-error (car warnings) (cdr warnings)))))
       (error
-       ;; Emacs behaves strangely if error an error escapes from a
-       ;; sentinel, so turns errors into messages.
+       ;; Emacs behaves strangely if an error escapes from a sentinel,
+       ;; so turn errors into messages.
        (message "%s" (error-message-string err))))
     (ignore-errors (delete-file err-file))))
 




More information about the notmuch mailing list