aboutsummaryrefslogtreecommitdiffhomepage
path: root/emacs
diff options
context:
space:
mode:
authorGravatar Carl Worth <cworth@cworth.org>2011-03-10 16:53:46 -0800
committerGravatar Carl Worth <cworth@cworth.org>2011-03-10 16:53:46 -0800
commit708c4f46caae76d4366a6a8de369943a3adef443 (patch)
tree1ffe5201a5b4f9efb97e9478f4120b3257e827f6 /emacs
parent8a534dc60d5f4bf579eabda9ae482a1982de3e4e (diff)
emacs: Don't drop error messages from "notmuch search"
With the previous commit, unexpected output before or between search results would be displayed. However, trailing junk from the "notmuch search" output would still be silently swallowed. The most common case for an error message from "notmuch search" would be an invalid command-line, and in that case, there would be no search results and the trailing error message would get swallowed. We fix the process sentinel to check for leftover data and add it to the final buffer. We also add a test case to ensure this works.
Diffstat (limited to 'emacs')
-rw-r--r--emacs/notmuch.el2
1 files changed, 2 insertions, 0 deletions
diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index dbf269a5..d9a6b67a 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -580,6 +580,8 @@ This function advances the next thread when finished."
(insert "Incomplete search results (search process was killed).\n"))
(if (eq status 'exit)
(progn
+ (if notmuch-search-process-filter-data
+ (insert (concat "Error: Unexpected output from notmuch search:\n" notmuch-search-process-filter-data)))
(insert "End of search results.")
(if (not (= exit-status 0))
(insert (format " (process returned %d)" exit-status)))