[PATCH] emacs: Proper error string handling in search sentinel

[PATCH] emacs: Proper error string handling in search sentinel

[Austin Clements]

Apparently Emacs provides a function to stringify errors properly.
Use this in the search sentinel where we have to do our own error
messaging, rather than assuming the first error argument will be the
descriptive string.
---
 emacs/notmuch.el |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index e58c51d..4c1a6ca 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -660,7 +660,7 @@ of the result."
 		      ;; things happen if a sentinel signals.  Mimic
 		      ;; the top-level's handling of error messages.
 		      (error
-		       (message "%s" (second err))
+		       (message "%s" (error-message-string err))
 		       (throw 'return nil)))
 		    (if (and atbob
 			     (not (string= notmuch-search-target-thread "found")))
-- 
1.7.10.4

Re: [PATCH] emacs: Proper error string handling in search sentinel

[David Bremner]

Austin Clements <amdragon@MIT.EDU> writes:

> Apparently Emacs provides a function to stringify errors properly.
> Use this in the search sentinel where we have to do our own error
> messaging, rather than assuming the first error argument will be the
> descriptive string.

Pushed.

Mark, this should fix the problem with applying the asynch parser
series you noticed.

d