Subject: bug#5342: 23.1; mairix-widget-search always shows whole threads

From: David Engster <[email-masked]>
Date: Sun, 10 Jan 2010 14:51:04 +0100

David Abrahams <[email-masked]> writes:
> Please write in English if possible, because the Emacs maintainers
> `M-x mairix-widget-search'
> search on body containing "foobar"
> do NOT check "Show full threads"
> finds all messages in threads that mention foobar in the body of any message.
>
> Tracked it down to 
>
>   (defun mairix-widget-send-query (widgets)
>     "Send query from WIDGETS to mairix binary."
>     (mairix-search
>      (mairix-widget-make-query-from-widgets widgets)
>      (if (widget-value (cadr (assoc "Threads" widgets)))
>          t
>        -1))
>   ;    ^^---------this should be nil
>     (kill-buffer mairix-customize-query-buffer))

I can confirm this change fixes the bug.  Thank you for tracking it
down.

I attached a bzr bundle containing the above patch.  Could please
someone commit this as a tiny change by Dave Abrahams and close this
bug?

Thanks,
David

# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: void@void-20100110134124-hfbk3j5r5xns33kr
# target_branch: http://bzr.savannah.gnu.org/srv/bzr/emacs/trunk
# testament_sha1: 474d00d991bef2c435a2bd6a1eb45039f742af62
# timestamp: 2010-01-10 14:47:44 +0100
# base_revision_id: [email-masked]\
#   6c004mpy2oj0fkey
# 
# Begin patch
=== modified file 'lisp/net/mairix.el'
--- lisp/net/mairix.el	2009-10-31 02:38:34 +0000
+++ lisp/net/mairix.el	2010-01-10 13:41:24 +0000
@@ -597,7 +597,7 @@
    (mairix-widget-make-query-from-widgets widgets)
    (if (widget-value (cadr (assoc "Threads" widgets)))
        t
-     -1))
+     nil))
   (kill-buffer mairix-customize-query-buffer))
 
 (defun mairix-widget-save-search (widgets)

# Begin bundle